質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

新規登録して質問してみよう
ただいま回答率
85.50%
Git

Gitはオープンソースの分散バージョン管理システム(DVCS)です。

GitHub

GitHubは、Gitバージョン管理システムを利用したソフトウェア開発向けの共有ウェブサービスです。GitHub商用プランおよびオープンソースプロジェクト向けの無料アカウントを提供しています。

Q&A

解決済

1回答

5310閲覧

Github pull request後マージ前の作業フローについて

t-book

総合スコア29

Git

Gitはオープンソースの分散バージョン管理システム(DVCS)です。

GitHub

GitHubは、Gitバージョン管理システムを利用したソフトウェア開発向けの共有ウェブサービスです。GitHub商用プランおよびオープンソースプロジェクト向けの無料アカウントを提供しています。

0グッド

2クリップ

投稿2015/07/27 07:43

いつもお世話になっています。
Githubのpull requestにおける、作業の流れについて質問させていただきます。

##状況

    1. ローカルの作業用ブランチで修正作業を行う
    1. forkした自分のGithubの作業用ブランチに、1.をpush
    1. Githubからpull requestを送信
    1. pull requestに対して修正指示を貰う
    1. 再度ローカルの作業用ブランチで修正する
    1. 気づいたらmasterが更新されていた!

この後にすべきこととして、ローカル作業用ブランチにmasterの差分を取り込んで修正し、再pushすれば良いかと思いますが、その際以下の認識に間違いはありますでしょうか。
※ローカルのmasterは再pull済み
※conflictは一旦考えない

##自分の認識

###その1

  • 作業ブランチへcheckoutし、rebase master でベースを置き換える
  • 修正後pushしようとすると、過去の履歴が変わっているのでforceしなければならない

##その2

  • 作業ブランチへcheckoutし、merge master で新たにマージコミットを作成する
  • 過去の履歴は変わらないので、そのままpushできる

##質問事項

上記の認識があっているという前提ですが...

Q1. pull requestフローにおいて push -f は行うべきではないのか、 = rebase は初回のみした方が良いのか
実際に試したことはないのですが、過去が変わることで初回push時の差分がおかしくなる?かどうかは分かりませんが、何となくよろしくない気がします。この考えは間違っていますでしょうか

Q2. マージコミットを作成した場合、修正コミットと、全てまとめても良いのか
プロジェクトのポリシーによるものかもしれませんが、pull requestのフィードバック後にマージを含む、複数コミットをすべて一つにまとめて(rebase -i)pushしても、問題はないでしょうか。

結局はどうするのが一般的な作法なのでしょうか、というところです。
以上、よろしくお願いします。

気になる質問をクリップする

クリップした質問は、後からいつでもMYページで確認できます。

またクリップした質問に回答があった際、通知やメールを受け取ることができます。

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

guest

回答1

0

ベストアンサー

投稿2015/07/27 08:52

sho_cs

総合スコア3541

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

t-book

2015/07/27 09:31 編集

ご回答ありがとうございます! なるほど、どちらも一般的なのですね。追加の質問になるのですが、もしかしたら`rebase`して`push -f`しなおしても、githubの方でいい感じに前回のプルリクからの差分を表示してくれるものなのでしょうか。 もしそうなのであれば、自分しか触らない作業ブランチは`push -f`、他の人も触るあるいは派生ブランチがある場合は`merge`する(まとめるかどうかはともかく)、と使い分けるのが個人的にGoodかなと思いました
t-book

2015/08/07 09:18

再度rebaseしてもgithubの方で、以前の履歴は消えずに辿れることが確認できました。
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

15分調べてもわからないことは
teratailで質問しよう!

ただいまの回答率
85.50%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問