NK
эт я Павлу говорил ))
сорри, не заметил))
Size: a a a
2019 July 09
P
пробовали код декомпозировать, чтобы конфликты не решать?
да с этим более менее хорошо было, я бы сказал. ну то есть конфликты были когда действительно связанный функционал правился
P
просто было много людей и очень интенсивная разработка
P
и медленные код ревью
NK
и медленные код ревью
нереально печет, когда ты поправил все комменты после код-ревью, и в это время другой девелопер смерджил свое
NK
и потом твой пулл реквест помирает от конфликтов
P
ID:671739633
и потом твой пулл реквест помирает от конфликтов
ну тут либо он либо ты, выживает сильнейший :)
NK
ну тут либо он либо ты, выживает сильнейший :)
угу)
АО
ID:671739633
и потом твой пулл реквест помирает от конфликтов
особенно если он тоже поправил те же комменты 😂
NK
особенно если он тоже поправил те же комменты 😂
лид поправил свои же комменты?
NK
какой-то нетерпеливый лид прям
АО
ID:671739633
лид поправил свои же комменты?
если лид ревьюит, то да
NK
зачем их оставлять тогда
АО
ну я не знаю, в каких ещё ситуациях два человека будут исправлять одно и то же
Dv
почему не мержаться?
ну у нас было обязательное код ревью, отпачковка веток всегда шла от develop, мержилась в develop только после code-review, а в мастер ветки мержились обычно раньше, как только пройдут тестирование.
в итоге если у тебя конфликт с мастером, то ты разрешаешь его, когда твои правки вольются в девелоп, и потом девелоп будет мержиться в мастер можно встретить тот же конфликт.
а еще при мерже в девелоп может быть конфликт, и если этот конфликт разрешить иначе чем при мерже ветки в мастер, то при мерже девелопа в мастер можно получить что угодно, например один и тот же кусок кода дважды в разных местах кода
ну у нас было обязательное код ревью, отпачковка веток всегда шла от develop, мержилась в develop только после code-review, а в мастер ветки мержились обычно раньше, как только пройдут тестирование.
в итоге если у тебя конфликт с мастером, то ты разрешаешь его, когда твои правки вольются в девелоп, и потом девелоп будет мержиться в мастер можно встретить тот же конфликт.
а еще при мерже в девелоп может быть конфликт, и если этот конфликт разрешить иначе чем при мерже ветки в мастер, то при мерже девелопа в мастер можно получить что угодно, например один и тот же кусок кода дважды в разных местах кода
Мне не кажется правильной такая организация работы, при которой одни и те же изменения мержатся в одну и ту же ветку по два раза из разных мест.
P
Мне не кажется правильной такая организация работы, при которой одни и те же изменения мержатся в одну и ту же ветку по два раза из разных мест.
мне тоже, точнее мне она кажется очень неправильной
NK
мне тоже многое кажется очень неправильным
AS
Лочить файлы для того, чтобы кто-то не мог запушить без меня
В гитлаб есть codeowers чтоб ты в ревью всегда добавить
Dv
В гитлаб есть codeowers чтоб ты в ревью всегда добавить
У нас используют upsource для ревью, там тоже такое есть. И он сам умеет вычислять овнеров.
Dv
А ещё там можно поставить вотч на определённые файлы или определённые слова в коде.