НБ
давай по буквам. есть сущность разраба. пусть будет Р. есть сущность тимлида, который мержит, пусть будет Т. кто по вашему делает ревью - Т или Р?
Size: a a a
2021 April 06
ЕГ
Дальше вы уже делаете это в том порядке, в каком будет комфортно команде. Хотите - сделайте фича-ветку и отдельные ветки для бэкера и фронтера, которые потом МРятся в фичу и в этот момент происходит ревью, а фича мержится уже без ревью в дев или релиз
O
В данном случае, Р делает ревью, а Т это ревью ревьюит.
EB
открыть PR/MR другими словами
O
ревью в данной терминологии сущность, а не процесс
O
просто MR означает "мерж", а не "ревью"
НБ
разраб обычно не мержит. но если у вас разрабы сами по себе могут мержить, то какое ревью ? делайте хуки на коммиты и смотрите кто что запушит
ЕГ
МР означает Мерж Реквест
ЕГ
То есть запрос на мерж одной ветки в другую)
O
я так и сказал
O
мы делаем ревью в фишай. Это вообще никак не связано с мержами
НБ
Евгений общается в понятиях системы контроля версий, у вас какая то своя терминология, потому у вас путаница и непонимание между собой
O
ОК, я хочу, что бы после того как разработчик сделал сабтаску, его код можно было каким-то образом посмотреть и оставить комментарии.
НБ
когда разраб делает запрос на МР, т.е. до апрува, должен быть кто то, кто поставит апрув, а тот кто мержит - пусть смотрит по апрувам . авпрув есть - мержит
ЕГ
Ну и можно, естественно, поставить ограничение, что без n апрувов мерж просто недоступен
НБ
да, именно так. но там можно и вкатить люлей, кто мержит без апрувов
O
Ну так вот, если ОДИН разработчик пишет код в своей ветке и мержит ее ОДИН раз - то получается именно то, что надо
ЕГ
Возможно даже, но это не точно, надеюсь знающие люди поправят, можно залочить мерж без апрува конкретного человека
ЕГ
Можно) Но можно и проморгать такой момент случайно и в одном из скольки-то случаев это может стрельнуть неприятно, поэтому лучше всё-таки лочить, ИМХО
O
ну так можно не давать прав на апрув другим людям. И можно автоматизировать мерж после апрува