Кодревью. Нужно или нет?
В последнее время случилось много обсуждений полезности и бесполезности кодревью.
Самые интересные можно послушать у подкаста SDCast https://sdcast.ksdaemon.ru/2020/07/sdcast-121/ и у ребят из Podlodka Teamlead Crew https://www.youtube.com/watch?v=IDj3x__YZgE
Если не хотите слушать, то частые два противоположных мнения таковы:
Стул №1. Кодревью нужно
Надо делать обязательно, а иначе наговнокодят, будет плохая архитектура, наберем техдолга, потом концов не сыщешь в том, кто прав, кто виноват, что с этим теперь делать.
Стул №2. Кодревью не нужно
Что мы тут взрослым людям сопли подтираем, релиз задач блокируем, время тратим. Нормально делай – нормально будет.
Между стульями
В связи с приходом в команду новых джунов и в целом расслоением команды на новичков, не знающих проекты, середнячков, знающих часть проектов и опытных ребят, делавших большинство (или все) этих проектов своими руками, я решил рассказать о своем видении кодревью.
На мой взгляд, кодревью для джунов строго обязательно. Пока апрува нет – в прод не уходит. Каждый вздох, чих, взгляд, букву (утрирую немножко) надо отсмотреть и проверить. Тут без строгого контроля можно не просто плохую архитектуру заиметь, а еще и упавший прод.
Кодревью для мидлов – ну пусть оно будет не такое строгое, не блокирующее релиз в большинстве задач, кроме каких-то критических, развесистых, сложных. Однако глазами надо внимательно за ними проверять более опытному товарищу.
За матерыми и опытными ребятами по-хорошему можно и не следить. Это и правда в большинстве случаев трата времени и замедление релиза.
Тем не менее я за то, чтобы по возможности пробегаться по коду даже матерых товарищей. Больше даже не с целью баги найти, а просто быть в курсе, что как делается. Чтобы потом не офигевать, когда коллега в отпуске, а вы совсем не в курсе, что творится в его куске кода, который надо поправить (привет всем тем, кто не пишет документацию).
Итог
Я не думаю, что надо однозначно решать делать кодревью для всего на свете, или наоборот ни для чего не делать. Смотрите по сотрудникам, их компетенциям, ответственности, внимательности, уровню задач.
В последнее время случилось много обсуждений полезности и бесполезности кодревью.
Самые интересные можно послушать у подкаста SDCast https://sdcast.ksdaemon.ru/2020/07/sdcast-121/ и у ребят из Podlodka Teamlead Crew https://www.youtube.com/watch?v=IDj3x__YZgE
Если не хотите слушать, то частые два противоположных мнения таковы:
Стул №1. Кодревью нужно
Надо делать обязательно, а иначе наговнокодят, будет плохая архитектура, наберем техдолга, потом концов не сыщешь в том, кто прав, кто виноват, что с этим теперь делать.
Стул №2. Кодревью не нужно
Что мы тут взрослым людям сопли подтираем, релиз задач блокируем, время тратим. Нормально делай – нормально будет.
Между стульями
В связи с приходом в команду новых джунов и в целом расслоением команды на новичков, не знающих проекты, середнячков, знающих часть проектов и опытных ребят, делавших большинство (или все) этих проектов своими руками, я решил рассказать о своем видении кодревью.
На мой взгляд, кодревью для джунов строго обязательно. Пока апрува нет – в прод не уходит. Каждый вздох, чих, взгляд, букву (утрирую немножко) надо отсмотреть и проверить. Тут без строгого контроля можно не просто плохую архитектуру заиметь, а еще и упавший прод.
Кодревью для мидлов – ну пусть оно будет не такое строгое, не блокирующее релиз в большинстве задач, кроме каких-то критических, развесистых, сложных. Однако глазами надо внимательно за ними проверять более опытному товарищу.
За матерыми и опытными ребятами по-хорошему можно и не следить. Это и правда в большинстве случаев трата времени и замедление релиза.
Тем не менее я за то, чтобы по возможности пробегаться по коду даже матерых товарищей. Больше даже не с целью баги найти, а просто быть в курсе, что как делается. Чтобы потом не офигевать, когда коллега в отпуске, а вы совсем не в курсе, что творится в его куске кода, который надо поправить (привет всем тем, кто не пишет документацию).
Итог
Я не думаю, что надо однозначно решать делать кодревью для всего на свете, или наоборот ни для чего не делать. Смотрите по сотрудникам, их компетенциям, ответственности, внимательности, уровню задач.