Обновить
3
Олег Климаков@olegklimakov

Frontend разработчик

6
Подписчики
Отправить сообщение

А для видимости прогресса совершенно ни к чему делать PR, достаточно пушить фичеветку.

Да, я про это и говорю, что она не всегда пушится валидным кодом, который доживет до PR.

Поинт про интерактивный ребейз я понял, это хорошая мысль. Главное чтобы в компании не было политики - все PR сквошим, тогда в этом отпадает смысл

Согласен, подход интересный, попробую в следующий раз на огромных PR :)

Историю безусловно стоит оформлять, но не в рамках PR. Зачем? Он же все равно будет сквошиться в 1 коммит, в котором будет и описание и номера задач в которых можно посмотреть контекст.

Мне кажется тезиса, что история коммитов не важна в принципе в комментариях не было. Обсуждаем исключительно в рамках пулл реквеста. А там уже каждый сам себе хозяин. Я например могу черновые разработки коммитить, чтобы появилась активность в задаче и ПМ видел что на задачу не забили. Потом вообще переписать этот код и декомпозировать. Поэтому эти коммиты сами по себе никакой ценности не несут. А целиком PR со всеми изменения - несет.

Всё зависит от того как принято проводить ревью в команде.
Компоновать красивые коммиты может быть затратно по времени и через историю коммитов мало кто проверяет пул реквест (по моему опыту).

Чаще всё же через просмотр изменений по всему реквесту.

Информация

В рейтинге
Не участвует
Дата рождения
Зарегистрирован
Активность