1. ホーム
  2. ギット

[解決済み】コードレビュー後のプルリクエストを更新するための望ましいGithubワークフロー

2022-03-25 08:38:30

質問

Github上のオープンソースプロジェクトに変更を提出し、コアチームメンバーの一人からコードレビューコメントを受け取りました。

レビューコメントを考慮してコードを更新し、再提出してみたいと思います。この場合、どのようなワークフローが最適でしょうか?私の限られたgit/githubの知識からすると、以下のいずれかを行うことができます。

  1. コードを新しいコミットとして更新し、最初のコミットと更新したコミットの両方を私のプルリクエストに追加します。

  2. どうにかして(?)私のリポジトリから古いコミットをロールバックし、すべてを含む単一の新しいコミットを作成し、そのためのプルリクエストを上げる?

  3. git commit にはamend機能がありますが、ローカルリポジトリの外にコミットをプッシュした後は使ってはいけないと聞いたことがあるのですが?この場合、私はローカルのPCで変更を行い、プロジェクトのgithubブランチにプッシュしました。この場合、'amend'を使ってもいいのでしょうか?

  4. 他に何か?

2/3 のオプションは、オープンソースプロジェクトがすべてを実装する履歴を 1 回のコミットで済ませることができるので、いいような気がします。

注:回答に影響するかどうかわかりませんが、私は別のブランチで変更を行わず、masterの上にコミットしただけです。

解決方法は?

プルリクエストで使用しているブランチに新しいコミットを追加し、そのブランチをGitHubにプッシュするだけです。プルリクエストは自動的に追加されたコミットで更新されます。

#2と3は不要です。もし、あなたのブランチがマージされた場所だけを見たい (そして追加コミットを見たくない) 場合は、次のようにします。 git log --first-parent を使えば、マージコミットだけをログに表示させることができます。