Когда остановиться и проверить — self-review перед push
До сих пор мы проверяли PR, который уже открыт, после того как CI уже отработал. Этот урок делает шаг назад, на одну стадию раньше, к моменту перед тем, как вы вообще сделаете push: к моменту, когда агент закончил редактировать код, а вы всё ещё наедине с diff, до того как он станет чьей-то чужой пр
Перед тем как выйти из дома с собранной сумкой, вы открываете её снова и сами проверяете, что на самом деле внутри. Недостаточно, что молния застёгнута и снаружи всё выглядит аккуратно — нужно убедиться, что туда не попало ничего, что вы не положили намеренно.
- self-review перед push
- Привычка читать полный diff и запускать локальные проверки перед тем, как отправить код через push, чтобы поймать проблему до того, как она станет публичным PR, который кто-то другой должен будет вскрыть.
- git diff --stat
- Быстрый обзор каждого изменённого файла и того, сколько строк изменилось в каждом — первый шаг self-review, до того как погружаться в детали какого-либо отдельного файла.
- тихий побочный эффект
- Правка, которая появляется в том же diff, но не связана с запрошенной задачей; она тихо проходит CI, потому что ни один существующий тест её не проверяет, и только чтение самого diff её обнаруживает.
- локальная проверка перед push
- Запуск lint и тестов на своей машине перед push, чтобы поймать проблемы в самый дешёвый для их исправления момент — до того как они станут публичными замечаниями к открытому PR.
