Как отразить результат просмотра на GitHub

Question

В общем, должен ли я (= участник) объединить два коммита, прежде чем отправлять код, который указан в обзоре на GitHub и исправлен мной?

$ git commit -am "Reflect the result of review"
$ git rebase -i HEAD~2
・・・merge two commits and delete the above commit message by squash option
$ git push -f origin [branch for this pull request]

Другими словами, второй шаг (git rebase ...) нужен?

Редактировать: Чтобы избежать недоразумений, я добавил "(= contributor)" к вышеприведенному. Таким образом, в этом вопросе «я» означает вкладчик, а не рецензент.

Answer 1

Вы имеете в виду, чтобы сжать х количество ревизий в одну? Это одна из возможностей ... но я предпочитаю делать это так:

git reset --soft HEAD~2 # take branch pointer two revisions back... it can be n changes, all changes are saved into the index ready to be committed
git commit -m "My feature"

Тогда вы можете нажать или все, что считаете нужным сделать с веткой.