Все коммиты привязывать к задаче
Во многих командах есть требование — каждый коммит в репозитории должен быть привязан к задаче в трекере. Однако есть ребята, которые не понимают, зачем это нужно, и считают это пустым формализмом. Попробую рассказать.
Во-первых, задача в трекере — это как тест до кода: прочитав\поставив задачу, ты уже определил конечный результат. Ты понимаешь, что и для чего ты делаешь, а не работаешь в режиме свободного художника.
Во-вторых, когда код не привязан к задаче, его сложнее смотреть — в глазах ревьюера он превращается из решения задачи в бессмысленные строчки на экране. В коде, который смотрели без привязки к задаче, никогда не найдут серьезных смысловых ошибок — только стилевые и другие очевидные вещи, которые вообще-то должен искать статический анализатор.
Ну и в-третьих, это блейм — тебе же самому через полгода захочется понять, зачем ты написал ту или иную строку. Ссылка на задачу в этом случае прибавит контекста. А что-нибудь вроде «changed layout» только добавит желания выкинуть весь старый код и написать новый.
Во многих командах есть требование — каждый коммит в репозитории должен быть привязан к задаче в трекере. Однако есть ребята, которые не понимают, зачем это нужно, и считают это пустым формализмом. Попробую рассказать.
Во-первых, задача в трекере — это как тест до кода: прочитав\поставив задачу, ты уже определил конечный результат. Ты понимаешь, что и для чего ты делаешь, а не работаешь в режиме свободного художника.
Во-вторых, когда код не привязан к задаче, его сложнее смотреть — в глазах ревьюера он превращается из решения задачи в бессмысленные строчки на экране. В коде, который смотрели без привязки к задаче, никогда не найдут серьезных смысловых ошибок — только стилевые и другие очевидные вещи, которые вообще-то должен искать статический анализатор.
Ну и в-третьих, это блейм — тебе же самому через полгода захочется понять, зачем ты написал ту или иную строку. Ссылка на задачу в этом случае прибавит контекста. А что-нибудь вроде «changed layout» только добавит желания выкинуть весь старый код и написать новый.