СА
Но это при условии что код ревью замещается дизайн ревью
А как же QA? Только для релизной ветки проводится?
Size: a a a
2021 March 02
СА
у нас были правила “вечером не должно быть висящих ревью, если они есть - то задача их провести становится первоочередной и блокирует остальные на следующий день”, работало
Хороший вариант, спасибо!
f🤔
Хороший вариант, спасибо!
это еще очень легко мониторится (у тебя вечером на все команды все МРы либо с WIP:, либо с аппрувами и ждут выкатки)
СА
некоторые команды еще бота юзают, которые смотрит открытые MR и в зависимости от давности пишет в чат команды всякие сообщения (он лежит на github, не знаю можно ли сюда ссылки кидать, чтобы не сочлось за рекламу)
У нас такой пишет конкретным ревьюверам, а не всей команде.
СА
это еще очень легко мониторится (у тебя вечером на все команды все МРы либо с WIP:, либо с аппрувами и ждут выкатки)
У нас ещё есть шаг QA между аппрувом и выкаткой. Это отдельная боль)
PV
У меня что-то IPsec-туннель перестал подниматься на телефоне с Tele2, не знаете, режут там чего-нибудь?
в логе сервера ретрансмиты фиксируются
в логе сервера ретрансмиты фиксируются
f🤔
У нас ещё есть шаг QA между аппрувом и выкаткой. Это отдельная боль)
ну тогда три состояния - WIP, аппрув и тег ready_for_acceptance, два аппрува (один от QA) и тег ready_for_release
Y
А как же QA? Только для релизной ветки проводится?
Автотесты и qa в проде, либо qa тестят релизы
Y
У нас ещё есть шаг QA между аппрувом и выкаткой. Это отдельная боль)
Ну вот у вас в этом процессе две блокирующих синхронных операции. По сути это всегда будет горлышком, как не старайся.
СА
Автотесты и qa в проде, либо qa тестят релизы
Тестить релизы кажется сложным, потому что надо перед этим разобраться, что именно тестить, и всё равно получается тест отдельных задач, но с влиянием других, попавших в этот же релиз
СА
Ну вот у вас в этом процессе две блокирующих синхронных операции. По сути это всегда будет горлышком, как не старайся.
Это правда, но это обеспечивает качество. В больших кодбазах это очень важно.
Y
Это правда, но это обеспечивает качество. В больших кодбазах это очень важно.
Альтернатива - канарейка и жертвуете качеством на проценте
Y
Тут всегда будет этот выбор: быстро vs качественно
А крутилка за вами
А крутилка за вами
Y
При этом сам факт код ревью не несет пользы
Y
Sast + дизайн ревью закрывают эту потребность более чем
СА
Альтернатива - канарейка и жертвуете качеством на проценте
Канарейка тоже есть, но на 15 минут перед основной раскаткой)
СА
При этом сам факт код ревью не несет пользы
Конечно несёт, когда есть инженерная культура.
Y
Конечно несёт, когда есть инженерная культура.
Не соглашусь
СА
Не соглашусь
А как ещё обеспечить выполнение правил и практик, соблюдение принципов и вот это вот всё?
Y
А как ещё обеспечить выполнение правил и практик, соблюдение принципов и вот это вот всё?
Дизайн ревью + автоматический анализ