Как провести код-ревью: советы по применению
Разбираемся, кому и когда нужна (и не нужна!) дополнительная проверка кода, и каких ошибок избежать на старте.
Разбираемся, кому и когда нужна (и не нужна!) дополнительная проверка кода, и каких ошибок избежать на старте.
Весь кипиш вокруг код ревью построен на идее что другой программист посмотрит кучу диффов файлов, правильно поймёт что имел в виду другой человек, посмотрит в джире описание таска, почитает документацию, сможет понять задачу лучше основного программиста и указать ему, программисту на ошибки.
И всё это без ущерба своим задачам.
Типичный итог, митинг:
- Что ты сделал сегодня?
- Сделал кодревью пяти таскам от Миши.
- А что ты сделал своего?
- Только вот этот маленький успел.
- ..., Премию получает Миша, так как он сделал 5 тасков, а ты один и маленький..
- ....
Грустный итог! В такой ситуации я бы открыто обсудила политику оценки результатов и рабочую загрузку с лидом. Потому что делать код-ревью системно в ущерб себе, естественно, не интересно: это отразится на работе, моральном духе и команде. В итоге в минусе все.