Комментарии 3
Не могу сказать, чтобы здесь были описаны бестпрактис, а вот как описание выстроенного процесса - нормас, спасибо
Забыли упомянуть Линтер, который пытались внедрить и который ставил в тупик даже опытных разрабов и не прижился, уступив место грамотно созданному (не сразу) регламенту.
Про новеньких:
Бывало, что два "относительно новеньких" встают на ревью в сетке двойного ревью, которые апрувят и сливают сразу свой код с явными ошибками. Такую ситуацию подсвечивали обоим и в дальнкйшнм было меньше ошибок (но по началу приходилось поглядывать за ними).
По поводу частичного ревью и конфликтов ревьюера и разработчика согласен, но, каюсь, пару раз так делал, чтоб разработчик сам весь код подправил по начальным замечаниям. Процесс долгий и реально приводит к конфликтам, поэтому отказался.
Как всегда остаётся дилемма - качество кода (только согласно регламента) против скорости отдачи задачи в тест. Весы шатаются и ревьюеру важно проявлять стойкость и приводить доводы своих замечаний. Желательно с ссылками на регламент. Бывало, что хотят обойти регламент фразой "работает же", упираясь в непоколебимость регламентных требований и "твёрдого" ревьюера :)
Ну и, естественно, я также был и есть со стороны разработчика, которому подсвечивают лишние пробелы, пропущенные точки с запятой и т.д. Вызывает негодование из-за лишней итерации задачи, но с годами маты сменились на "хмм, точно, ща подправлю".
Про рефакторинг полностю согласен. Если разраб не хочет править код вне своего, но видно, что его правки выделяются, решение остаётся за разработчиком, но в задаче желательно отписать, что форма требует дальнейшего рефакторинга. Такие комменты особо никто не воспринимает, но зато разраб все свои мосты грамотно уничтожил и к нему не подкопаться потом.
Про "противоположные рекомендации" первый раз увидел, такого не должно быть, но если есть, нужно реально вычишать "серые зоны" общением. Благо все открыты для диалога. И тут реально будет спасать регламент написания кода (который после обсуждений реально подправить и всем подсветить, такое было).
Статья хорошая, я бы добавил пару матов ещё в ней 😂
Спасибо большое за то, что прочитали)
Кстати, линтер сейчас есть, некоторые вещи позволяет на ревью не пускать сразу. Его, конечно, тоже надо внедрять осторожно. Если включить все и сразу - то первые PR'ы после этого пройдут его примерно через никогда. Один из вариантов - идти по пути наименьшего срабатывания. Включили одну проверку: упало в 5 местах - терпимо, правим, оставляем включенной. Включили вторую: упало в 5000 мест - к ней мы не готовы, зайдем попозже:-)
Жизнь во время ревью: как помирить Лешу и Славу и при этом не обидеть Колю…