[解決済み】コードレビュー後のプルリクエストを更新するための望ましいGithubワークフロー
質問
Github上のオープンソースプロジェクトに変更を提出し、コアチームメンバーの一人からコードレビューコメントを受け取りました。
レビューコメントを考慮してコードを更新し、再提出してみたいと思います。この場合、どのようなワークフローが最適でしょうか?私の限られたgit/githubの知識からすると、以下のいずれかを行うことができます。
-
コードを新しいコミットとして更新し、最初のコミットと更新したコミットの両方を私のプルリクエストに追加します。
-
どうにかして(?)私のリポジトリから古いコミットをロールバックし、すべてを含む単一の新しいコミットを作成し、そのためのプルリクエストを上げる?
-
git commit
にはamend機能がありますが、ローカルリポジトリの外にコミットをプッシュした後は使ってはいけないと聞いたことがあるのですが?この場合、私はローカルのPCで変更を行い、プロジェクトのgithubブランチにプッシュしました。この場合、'amend'を使ってもいいのでしょうか? -
他に何か?
2/3 のオプションは、オープンソースプロジェクトがすべてを実装する履歴を 1 回のコミットで済ませることができるので、いいような気がします。
注:回答に影響するかどうかわかりませんが、私は別のブランチで変更を行わず、masterの上にコミットしただけです。
解決方法は?
プルリクエストで使用しているブランチに新しいコミットを追加し、そのブランチをGitHubにプッシュするだけです。プルリクエストは自動的に追加されたコミットで更新されます。
#2と3は不要です。もし、あなたのブランチがマージされた場所だけを見たい (そして追加コミットを見たくない) 場合は、次のようにします。
git log --first-parent
を使えば、マージコミットだけをログに表示させることができます。
関連
-
[解決済み] Git のアンプッシュされたコミットを表示する
-
[解決済み] Git のリモートブランチを作成する方法を教えてください。
-
[解決済み] Git リポジトリで削除されたファイルを検索して復元する方法
-
[解決済み] Git を使ってタグをリモートリポジトリにプッシュするには?
-
[解決済み] Gitで落としたスタッシュを復元する方法とは?
-
[解決済み] 現在のGitブランチをmasterブランチにする
-
[解決済み] Git はシンボリックリンクをどのように扱うのですか?
-
[解決済み] Git のブランチの一覧を最新のコミット順に表示するにはどうしたらよいですか?
-
[解決済み] detached HEADとmaster/originの連携はどうすればいいですか?
-
[解決済み】"git pull" でローカルファイルを強制的に上書きするには?
最新
-
nginxです。[emerg] 0.0.0.0:80 への bind() に失敗しました (98: アドレスは既に使用中です)
-
htmlページでギリシャ文字を使うには
-
ピュアhtml+cssでの要素読み込み効果
-
純粋なhtml + cssで五輪を実現するサンプルコード
-
ナビゲーションバー・ドロップダウンメニューのHTML+CSSサンプルコード
-
タイピング効果を実現するピュアhtml+css
-
htmlの選択ボックスのプレースホルダー作成に関する質問
-
html css3 伸縮しない 画像表示効果
-
トップナビゲーションバーメニュー作成用HTML+CSS
-
html+css 実装 サイバーパンク風ボタン
おすすめ
-
[解決済み】Gitマージで「すでに最新」と報告されるが、違いはある
-
[解決済み] [email protected] へのいくつかの参考文献のプッシュに失敗しました。
-
[解決済み] [Solved] 作業ツリーのディレクトリ 'example.com' を作成できませんでした。パーミッションが拒否されました
-
git commit リモートエラー [rejected] master -> master (フェッチファースト)
-
[解決済み] git が「Pull is not possible because you have unmerged files」と表示するのはなぜですか?
-
[解決済み] git status に fatal: bad object HEAD と表示される。
-
[解決済み] 既にリベースを開始している場合、2つのコミットを1つにマージするにはどうすればよいですか?
-
[解決済み] フォークされたレポからプルリクエストを更新する方法は?
-
[解決済み】GitHubのプルリクエストを修正する方法は?[重複あり]
-
[解決済み] GitHubです。マージされたプルリクエストを再開する