Мерж откладывается по эстетическим причинам

Открываешь MR, а там: «Я бы назвал эту переменную по-другому».

Автор объясняет свой выбор, ревьюер предлагает альтернативу, третий разработчик не согласен с обоими. Через двадцать сообщений выясняется, что все варианты нормальные, но MR всё ещё не замержен, а время уже потеряно.

Код-ревью легко превращается в спор о вкусах. Ранний return или вложенный if? Тернарник или обычное условие? Само по себе обсуждение читаемости и поддерживаемости полезно. Проблема начинается, когда «мне так больше нравится» становится причиной блокировать MR.

Чтобы избежать этого, мы с командой договорились помечать необязательные замечания префиксом nit (от nitpick — придирка к мелочам, а не от nit - гнида 😁)

Например: nit: я бы переименовал userData в userProfile

А если название действительно вводит в заблуждение или нарушает соглашения команды — это уже полноценное замечание, которое стоит аргументировать, а споры о форматировании и прочих вещах, которые можно формализовать, лучше вообще отдать линтерам и форматтерам.

#ks_разработка