АГ
вот я по себе скажу - когда на ревью попадает портянка из 30 файлов на 5000 строк правок - то это нереально проревьюить. пролистываешь и аппрувишь. если правка небольшая - ее можно воспринимать.
Size: a a a
2021 June 02
АГ
можно даже практиковать делать на одну задачу 3-4-... реквестов, каждый из которых мелкий и делает какую-то одну вполне конкретную и понятную вещь
АГ
но это если есть культ ревью. идею отказа от ревью - люто плюсую.
PD
Ну, review-on-demand можно оставить, там портянок не бывает)
S
что мешает ввести правило: не более Н изменений?(бывают и правда исключения, да)
PD
Цель какая?
АГ
невозможность автоматизированного контроля в первую очередь
АГ
все, что опирается на "дисциплину" - не работает. не будут люди это соблюдать. по крайней мере иногда. а то и часто.
АГ
а специальные инструменты под это искать прикручивать / делать - как то сильно излишним выглядит
S
уменьшить время на прохождение ревью, уменьшить сложность и обьем контекста для ревьюера
АГ
еще кстати ревьюить мешает авторитет автора. увидел серьезного чувака в авторах - аппрувнул не глядя.
АГ
хотя все ошибаются.
PD
Смотри доклад, практики review-on-demand. И выкинь code review (если не софт для АЭС пишешь)
S
> Линтеры/форматтеры/тесты/sonarqube - лишь верхушка.
Код может быть хорошо написан, но сама структура/архитектура реализована плохо для чего-то в будущем/вылезет в большую проблему в будущем. Так что нужен взгляд человека, и донесение до ревьюера контекста(через акцентирование)
Код может быть хорошо написан, но сама структура/архитектура реализована плохо для чего-то в будущем/вылезет в большую проблему в будущем. Так что нужен взгляд человека, и донесение до ревьюера контекста(через акцентирование)
S
Про не смешивать реализацию и рефакторинг я не говорю, потому что очевидно
АГ
что является очень большой тратой времени и вообще сложно.
АГ
тут ошибиться сильно проще, и в итоге получить то, что это не будет работать.
АГ
т.е. у вас в компании принцип бой скаута порицается.
АГ
"вынеси рефакторинг в отдельный реквест" == "выкинь свой рефакторинг и больше так не делай"
PD
А вот это - про design review как раз