За выходные захерачил пулреквест на 2000 новых линий и больше 1000 удаленных.

Это конечно же не повод для гордости, так как ревьювить этот PR будет сплошным кошмаром для Арнаба…

В идеале нужно использовать правило маленьких коммитов и делать небольшие, изолированные изменения. Но я был в ударе и решил зарефакторить весь сайт, пока Арнаб отдыхал на озерах в своей Канаде.

Нефиг ходить в отпуска 😀

В итоге 3000 строчек изменений заревьювилось супер быстро.

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

В то время как в маленьких коммитах разобраться несложно и очень легко задушнить из-за мелочей.

Мораль: маленькие коммиты лучше, так как это ведет к более качественной кодовой базе, но только при условиях, что а) PR ревьювятся максимально быстро и б) участники процесса договорились о принципах заранее и нет препираний по мелочам в каждом пулреквесте.

Иначе будут а) задержки и б) пустод.очерство.