АС
АПК в МР? А у вас есть пример пайпа для этого?
Size: a a a
2021 November 12
PZ
Эт само собой
КЧ
Угораешь? Я манал сутки ждать пока эта тварь оклемается.
Но слышал что так делают
Но слышал что так делают
Д
Вот как раз по такой и надо. Так как пользовательская постановка есть, а техническое решение - только в коде. Если его нужно поменять, то нужно проверить актуальность всех кусков (и выкинуть к этой самой матери)
c
Надо писать регламенты так же как и комментарии, кратко и по делу, и чтобы это не оставляли на выходные почитать.
АС
Да я даже делал https://infostart.ru/public/1117485/ . Но чтоб полноценные МР с запуском статанализа и тестов, и чтоб потом только влить, а не бегать с cf и с хранилищем - это прям какое то недостижимое будущее.
PZ
Но реально, 2-3 ревью с побольше кода.
И все ок.
Я уже давно не тешу себя надеждами, что все прочитают регламент и с первой попытки будут без ошибок делать.
Объясняешь почему так и люди переучиваются.
И все ок.
Я уже давно не тешу себя надеждами, что все прочитают регламент и с первой попытки будут без ошибок делать.
Объясняешь почему так и люди переучиваются.
АС
Ваш % кода, который был дважды прочитан (аудит, ревью в МР/ПР, парное)
Анонимный опрос
Проголосовало: 32A
Всмысле читают ли мой? или читаю ли я?
АС
Ты положил 1000 строк кода, коллега положил 1000 строк кода. Если ты его прочитал, а он твой нет, то 50% (1000/2000*100%)
АС
Покрытие кода аудитором, вот
A
я прочитал 10 из 50 коммитов. Остальные 20 человек прочитали 0 коммитов.
АС
ты молодец
A
в какой процент тыкать? =)
АС
ну видимо дважды прочитано оказалось 10/50 строк, то есть 20%. Если ты взял поменьше коммиты, то 0-20, если побольше, то 20-50
КЧ
А, не, у меня все это есть, только без АПК
АС
и тесты со статанализом на МР?
OT
в нафиг такие MR )
OT
Они только вредят. На крайний случай тестировать в АПК ветку по запросу и увсе
OT
тестировать -> проверять