Распространенным сценарием в разработке программного обеспечения является проверка кода чужого кода. Распространенным инструментом для этого является открытие запроса на извлечение.
Мой вопрос, когда проблемы обнаружены в обзоре, должны ли изменения
- быть зафиксирован отдельно (новый коммит)
- или если существующий коммит будет изменен (при условии, что никто не выходит из вашего предыдущего коммита ... поскольку переписывание истории из общей ветки - плохая новость).
Для первого сценария легко отслеживать инкрементные изменения, хотя это добавляет некоторый шум в историю фиксации. Второй вариант имеет обратные плюсы и минусы.
git
code-reviews
repository
loeschg
источник
источник
Ответы:
Вы предполагаете, что исправление не представляет никаких новых проблем и исправляет старые, завершенные. Но многие исправления заслуживают отдельного обзора, и это, вероятно, намного проще, когда дополнительные изменения могут быть рассмотрены отдельно.
источник