2019-06-08
Как поступать с душными комментарии во время код-ревью?
Часто бывает что инженер, проверяющий твой код, оставляет какие-то бестолковые комментарии. Иногда они даже верны, но смещают фокус с основной задачи.
Раньше я вздыхал, и исправлял все замечания, что конечно замедляло время до закрытия задачи, и вообще бесило страшно.
Став старше, я пытался вступать в спор, рассказывая, что такие комментарии отнимают уйму времени, размазывают фокус и конечно же вносят больше проблем (редко кто тщательно тестирует код после внесения таких правок). Опять же такие споры затягивали ревью и изрядно трепали нервы.
Сейчас на многие замечания, если они не относятся к решению текущей задачи, я создаю тикеты в джире.
Хорошее замечание, вот я создал тикет, чтобы не потерялось, обязательно поправим.
- Во-первых это обезоруживает комментатора: его замечания учтены, и вроде как неудобно заставлять тебя включать в PR другую задачу.
- Во-вторых, есть возможность привлечь команду к оценке важности этих изменений (когда тикет будут включать в спринт)
- И на последок, это просто ускоряет процесс, и наглядно показывает цену таким "мелким" замечаниям и предложениям.
Как всегда важен баланс, и если замечание толковое, имеет непосредственное отношение к задаче или улучшает предложенные тобой изменения, то так делать не стоит.