Комментарии 8
По хорошему, вся эта информация может быть сразу в тексе коммита.
0
Соглашусь, каждый коммит должен сопровождаться содержательным сообщением, а не чем-то вроде «fix». Но не стоит пренебрегать описанием при создании пулл-реквеста, где можно добавить дополнительной информации о решаемой проблеме. Это особенно полезно, если команда — распределенная, и ревьюер не может «дойти ногами» до автора реквеста и задать ему какие-то уточняющие вопросы.
0
На мой взгляд, зависит от флоу.
Если в реквесте коммитов несколько (много), то удобно, когда есть краткое содержание в описании.
Если же один реквест ~= один коммит, и в нём всё подробно описано, то да, в реквесте можно практически ничего не писать.
Если в реквесте коммитов несколько (много), то удобно, когда есть краткое содержание в описании.
Если же один реквест ~= один коммит, и в нём всё подробно описано, то да, в реквесте можно практически ничего не писать.
0
Соглашусь. Особенно больно переезжать с одного гит-сервера на другой, когда обычный максимум информации — это коммиты. В лучшем случае ещё и ишузы.
+1
промахнулся уровнем. это было к первому комментарию
0
Pair programming, mob programming уже непопулярны в слак?) Зачем я буду включать коллегу в ревьюеры если он не в курсе, что я делаю. Значит изначально работа велась неверно.
0
Можно быть в курсе задачи, которую ты делаешь, но не быть в курсе возникших проблем и причин почему выбрано то или иное решение.
В практике бывало такое, ревьюишь кого-то, пишешь мол зачем ты тут эту дрянь сделал, в ответ на что узнаешь, что это из-за кучи проблем и невозможности использовать другое более хорошее решение (которое было проверено). Не пришлось бы задавать эти вопросы будь это описано сразу в пиаре.
0
Зарегистрируйтесь на Хабре, чтобы оставить комментарий
Пулл-реквесты с эмпатией