Size: a a a

2021 April 06

O

Oleg in atinfo chat
или что б это один человек делал
источник

O

Oleg in atinfo chat
это уже не принципиально
источник

НБ

Никита Белов... in atinfo chat
без разницы. разраб дает запрос на мерж из ветки в ветку. если ему надо дать запрос в две ветки - он делает два запроса. если надо пройти ревью один раз, при двух запросах, я бы делал вебхуки и кодил . типа придумать как выглядит запрос на мр, и тянуть его куда нить в общую панель
источник

O

Oleg in atinfo chat
не понял :)
источник

O

Oleg in atinfo chat
собственно проблемы получается две - больше одного разраба в одной ветке или больше одного мерж реквеста
источник

O

Oleg in atinfo chat
пример самый простой - параллельно два человека пишут код и тесты на него
источник

ЕГ

Евгений Горбоконенко... in atinfo chat
Да не надо писать двум людям одновременно код и тесты на него
источник

ЕГ

Евгений Горбоконенко... in atinfo chat
Это фиговая идея
источник

НБ

Никита Белов... in atinfo chat
ну если надо удешевить время ревьюера, чтобы он два раза не ревьюил одно и тоже, то можно придумать чо нить типа моя/последовтаельность/в/пуше , сделать хуки на мр. и сделать скрипт, который скрыжит общие "моя/последовтаельность/в/пуше" в одну, чтобы посмотреть одни раз. но это тупо.
источник

O

Oleg in atinfo chat
почему?
источник

ЕГ

Евгений Горбоконенко... in atinfo chat
Потому что возникают все вышеописанные вами проблемы с синхронизацией)
источник

O

Oleg in atinfo chat
но они возникают не у меня, а у гита. Без гита проблем нет
источник

ЕГ

Евгений Горбоконенко... in atinfo chat
Потому что у тестирующего нет под рукой кода, который пишет второй, соответственно он пишет непонятные тесты, которые он не может запускать в процессе
источник

НБ

Никита Белов... in atinfo chat
а без гита как? из архива разворачиваете?
источник

O

Oleg in atinfo chat
Без гита они пишут код в одной ветке
источник

O

Oleg in atinfo chat
один написал классик - другой сразу тестик на него. И так инкрементально
источник

НБ

Никита Белов... in atinfo chat
простите, но вам нужен девопс , который объяснит то , что вы хотите. я опять нихрена не понял )
источник

НБ

Никита Белов... in atinfo chat
куча разрабов? в одной ветке? ну поздравляю
источник

O

Oleg in atinfo chat
два
источник

EB

Evgenii B in atinfo chat
Я сейчас попробую сделать хорошо и понятно. дайте знать если я делаю сильно понятнее, но все еще не понятно, и я добавлю кое-чего.


Есть ветка MAIN (она же мастер, она же - фактически отражает состояние продакшена)

когда ты хочешь зарелизить какую-то большую фичу, ты создаешь в Джире Сторю. на сторе привязаны сабтаски для разработчиков.

представь, что у тебя есть 2 разработчика. Один делает работу в vasya-17785-add-endpoint-cart-items, другой в masha-17801-add-template-cart.


Маша и Вася создали Пул реквесты своих веток в Ветку релиз, который по сути привязан к юзерсторе. пусть будет ветка release-new-cart-page.


2 PR которые открыты на ревью и будут закрыты (вмержены) когда все вопросы будут решены.

Какие вопросы могут быть?
- мердж конфликт
- красные тесты
- проблемы, которые тесты не покрывают

Предположим,  что ветки очень сырые и разработчики чтобы разблокировать себя именно должны делиться кодом, как ты сказал. У них должна быть какая-то точка синхронизации. Что можно сделать?

Маша может от своей текущей фиче ветки сделать ветку, в которую смерджит код Васи. Проверит, что ей нужно. даст обратную связь Васе. Вася обновит свою ветку.

Так решилась проблема неявная. точка синхронизации - кто-то взял и вмерджил себе локально без всяких разрешений ветку коллеги.

Далее уже каждый из Мердж реквестов должен будет удовлетворять тем критериям качества и проверок, которые есть у вас на проекте.

Следующий вопрос:
вот кто-то уже фичу сделал. ветка прошла ревью и была смерджена в релиз. но появились доработки. Что с ними делать? Создай отдельную ветку с доработками! повтори ревью. Если хочется не терять все мердж реквесты -- называйте коммиты во всех ветках с префиксами тикетов в джире. так у Васи например будет один тикет. но в нем будут отображены линки на 3 мердж реквеста к этому тикету: 1 изначальный (большой) и 2 маленьких (правки)
источник