Как стать автором
Обновить

Комментарии 8

По хорошему, вся эта информация может быть сразу в тексе коммита.
Соглашусь, каждый коммит должен сопровождаться содержательным сообщением, а не чем-то вроде «fix». Но не стоит пренебрегать описанием при создании пулл-реквеста, где можно добавить дополнительной информации о решаемой проблеме. Это особенно полезно, если команда — распределенная, и ревьюер не может «дойти ногами» до автора реквеста и задать ему какие-то уточняющие вопросы.
На мой взгляд, зависит от флоу.
Если в реквесте коммитов несколько (много), то удобно, когда есть краткое содержание в описании.
Если же один реквест ~= один коммит, и в нём всё подробно описано, то да, в реквесте можно практически ничего не писать.

Соглашусь. Особенно больно переезжать с одного гит-сервера на другой, когда обычный максимум информации — это коммиты. В лучшем случае ещё и ишузы.

Pair programming, mob programming уже непопулярны в слак?) Зачем я буду включать коллегу в ревьюеры если он не в курсе, что я делаю. Значит изначально работа велась неверно.

Можно быть в курсе задачи, которую ты делаешь, но не быть в курсе возникших проблем и причин почему выбрано то или иное решение.


В практике бывало такое, ревьюишь кого-то, пишешь мол зачем ты тут эту дрянь сделал, в ответ на что узнаешь, что это из-за кучи проблем и невозможности использовать другое более хорошее решение (которое было проверено). Не пришлось бы задавать эти вопросы будь это описано сразу в пиаре.

Можно в комментариях в исходниках писать причину использования костыля. Тогда ни у кого не возникнет такого вопроса, ни у ревьюера, ни у кого-либо ещё в будущем.

Только полноправные пользователи могут оставлять комментарии. Войдите, пожалуйста.