Dev Easy Notes

В дополнение к предыдущему, сильная педантичность к проверке изменений приводит к другой проблеме. Большие изменения ревьются крайне долго или вообще забивается на ревью. Это конечно отчасти проблема автора МР, но порой никак без больших изменений. Так устроена наша психика, когда привыкаете докапываться до каждой строчки. Потом смотришь на изменения в 1к строчек и твоей мозг думает, ууу это на долго, давай сначала с мелочью разберемся чтобы ничего не отвлекало. Ну а дальше сами понимаете. У нас в команде вообще есть рофл для таких МРв, если он висит несколько дней и без комментариев, мы считаем что МР достаточно "настоялся" и в нем точно нет косяков.

Ну и еще момент, для всех правил есть исключение. Правила которые я описал выше применимы когда у вас зонеркоманд из сеньоров и крепких мидлов. Когда проводим ревью джунов, можно добавить чуть больше духоты, но аккуратно и с объяснением почему нужно делать так, и так.

www.tgoop.com/dev_easy_notes/279

4.3K viewsNikita, Jan 22, 2024 at 10:00

В дополнение к предыдущему, сильная педантичность к проверке изменений приводит к другой проблеме. Большие изменения ревьются крайне долго или вообще забивается на ревью. Это конечно отчасти проблема автора МР, но порой никак без больших изменений. Так устроена наша психика, когда привыкаете докапываться до каждой строчки. Потом смотришь на изменения в 1к строчек и твоей мозг думает, ууу это на долго, давай сначала с мелочью разберемся чтобы ничего не отвлекало. Ну а дальше сами понимаете. У нас в команде вообще есть рофл для таких МРв, если он висит несколько дней и без комментариев, мы считаем что МР достаточно "настоялся" и в нем точно нет косяков.

Ну и еще момент, для всех правил есть исключение. Правила которые я описал выше применимы когда у вас зонеркоманд из сеньоров и крепких мидлов. Когда проводим ревью джунов, можно добавить чуть больше духоты, но аккуратно и с объяснением почему нужно делать так, и так.

BY Dev Easy Notes