Насколько душным должно быть код-ревью?

Код-ревью легко превращается в источник раздражения: придирки к стилю, бесконечные итерации, долгое ожидание и неясный фидбек. Но ту же проверку можно сделать полезной для обеих сторон — если договориться о правилах и убрать рутину.

Сабина (DataArt) и Жанара (Ozon) в совместном докладе разбирают боли ревьюера и ревьюируемого и показывают, как давать и принимать фидбек так, чтобы он приносил пользу. Они опираются на собственный опрос разработчиков о том, что делает код-ревью полезным.

В докладе: — как отдать рутину машинам: тесты (pytest), форматирование (black, isort), линтинг (flake8, pylint, mypy, bandit) и Ruff — pre-commit и автоматическое ограничение размера больших пулл-реквестов — как определять стандарты кода для проекта и фиксировать их в документации и чек-листах — как готовить пулл-реквест: ревью самому себе, понятное описание (What? Why?), не смешивать рефакторинг с логикой, уважать время ревьюеров — как принимать фидбек без перфекционизма и уточнять неясные замечания вместо защиты — как давать фидбек: примеры кода, запросы вместо команд, отказ от «ты», отмечать не только недостатки — culture map: почему «okay» у американцев и голландцев значит разное