intermediate

Code review

Ревьюйте behavior, risk, boundaries, tests, maintainability и operational clarity вместо feedback только по style.

Эффективный code review смотрит на behavior, risk, boundaries, tests, maintainability и operability — не только на style. Спросите: соответствует требованиям? Что ломается при scale? Ошибки обработаны? Следующий читатель поймёт? Для рискованных изменений — чеклисты security и migrations.

Обратная связь конкретная и уважительная; blockers отдельно от suggestions. Автор даёт контекст, скриншоты и доказательства тестов.

На интервью: как вы ревьюите PR с auth и database migration иначе, чем UI tweak.

Типовые ошибки: bikeshedding форматирования; rubber-stamp; споры без конкретных правок.

Компромисс — тщательность review и latency против дефектов в проде.

Чеклист:

Behavior, tests и rollback plan.
Boundaries и error paths.
Глубина review по риску изменения.