Size: a a a

2021 April 06

O

Oleg in atinfo chat
Если у фичи будет выключатель, то количество кода точно не уменьшится. Как это может уменьшить количество конфликтов мне не ясно.
Вот например минимальный вариант - в одной ветке есть фича и тесты. Хочу отдельно ревью на фичу, отдельно на тесты. Мержить тесты без фичи - никак.
источник

ЕГ

Евгений Горбоконенко... in atinfo chat
А зачем их ревьюить по отдельности?
источник

O

Oleg in atinfo chat
Сделаны в разное время, разными людьми, но в одной фича-ветке
источник

ЕГ

Евгений Горбоконенко... in atinfo chat
В разное время, разными людьми - стало быть сначала был МР с фичей, её отревьюили. Потом был МР с тестами, их отревьюили. Не увидел проблемы)
источник

O

Oleg in atinfo chat
Так они пишутся параллельно, в первый МР тесты могут попасть, во втором тогда части тестового кода не будет
источник

ЕГ

Евгений Горбоконенко... in atinfo chat
Это звучит будто тут что-то сильно с процессом не так, и не надо выдумывать как решить эту проблему, а просто делать всё правильно и последовательно)
источник

O

Oleg in atinfo chat
Ну ок, давай искать проблему.
В ветке может быть код разных людей?
источник

O

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

ЕГ

Евгений Горбоконенко... in atinfo chat
Сначала разбить задачу на подзадачи так, чтобы у каждого разработчика была своя таска, и он в ней был один. И ветка была для этой таски отдельная. Затем открывать отдельные MR'ы в feature-ветку.
источник

ЕГ

Евгений Горбоконенко... in atinfo chat
И стараться декомпозировать и делить в команде задачи так, чтобы не было такого, что "код этого разработчика нужен в задаче вот того, ещё до тех пор, пока он будет вмержен".
источник

ЕГ

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

O

Oleg in atinfo chat
Я не вижу смысла стараться делать так, что б код был независим относительно людей
источник

O

Oleg in atinfo chat
Отдельная таска на тесты, это же нормально*
источник

ЕГ

Евгений Горбоконенко... in atinfo chat
Не всегда, но может такое быть, да
источник

O

Oleg in atinfo chat
А они не могут работать без кода фичи, которую они тестируют
источник

ЕГ

Евгений Горбоконенко... in atinfo chat
Почему нет? Это не так сложно, как вы думаете, а профит гигантский. Позволяет просто брать и писать код, и не заморачиваться
источник

ЕГ

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

ЕГ

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

O

Oleg in atinfo chat
Задачи ОБЫЧНО разделяются, но ИНОГДА, особенно в середине процесса, оказывается, что они связаны. Если делать одну задачу ПОСЛЕ другой, то можно застрять, поэтому обычно все делается параллельно.
источник

ИС

Игорь Середа... in atinfo chat
Нет.
источник