правильный способ включить запрошенные изменения в запросе на вытягивание

Question

Каким будет правильный способ включения запрошенных изменений в PR?

Предположите, что рецензент запросил некоторые изменения в моем PR, например. метод переименования, отмена зависимостей и т. д.

Могу ли я внести изменения и совершить новые коммиты?
или do I change past commit to include the requested changes?

or reset and change

Answer 1

Я хотел бы сказать, что это сильно зависит от вашей организации команды.

В моей команде, например, если у меня есть какая-то просьба, измененная в моем PR, я попытаюсь сгруппировать их в сообщении PR, начиная свой комментарий с Rework after PR: <my changes here>.

Некоторые хорошие ресурсы здесь:
- Understanding Github flow
- git-flow cli

Вообще, я рекомендую вам не commit --amend изменения (после PR), это самый лучший способ потерять след вещей, которые вы, возможно, потребуется позже ,

Answer 2

Нет, вам не нужно изменять свою последнюю фиксацию. Вам просто нужно следить за нормальным процессом, т. Е. Изменять файлы, делать git commit -am "Bug_id", а затем делать git push. Он будет автоматически указан ниже изменить после комментария комментатор.

Answer 3

Как говорили другие, это во многом зависит от практики, которую сопровождающие придерживаются/просят принять другие.

Обычно я исправляю изменения, заданные рецензентами, на отдельную фиксацию вместо изменения предыдущей, поскольку это облегчает отслеживание изменений рецензентами. Когда он/она удовлетворен вашим PR, они обычно просят вас раздавить/исправить коммиты, чтобы они имели больше смысла для кого-то, кто смотрел на историю Git. Используйте замечательную интерактивную функцию переадресации git для этого: https://git-scm.com/book/en/v2/Git-Tools-Rewriting-History. Сначала это может показаться пугающим, но это чрезвычайно универсальный инструмент, как только вы его поняли и освоили.

Однако, если ваши изменения незначительны и несущественны и не нуждаются в отдельной фиксации, лучше просто добавить изменения и внести поправки в предыдущую фиксацию. Не забудьте использовать git push -f, чтобы внести изменения после того, как вы это сделали, потому что git требует по умолчанию, чтобы ваши изменения были потомками (то есть изменениями, совершаемыми поверх определенного коммита) последнего совершения, который вы нажали.