githubでpull request毎にブランチを切るべき理由

うえちょこ🐤🔤 @uechoco

最近githubのpull request(PR)をブランチ切って送ってる人をよく見かけるんだけど、ブランチの名前がPRの事象名になってて、このPRのためだけにブランチ切ってるの？って思ったんだけど、あれはどういう意味なのか、なにかの作法なのか、よくわかってない @

shishi @shishi4tw

@uechoco 私はそう教わりましたので毎回ブランチ切ってますよ！

ooharabucyou @ooharabucyou

@uechoco http://d.hatena.ne.jp/kunit/touch/20090917%231253191889 http://progit.org/book/ja/ch3-4.html こういうことかな？

うえちょこ🐤🔤 @uechoco

@shishi4tw 精進します。。。

うえちょこ🐤🔤 @uechoco

@ooharabucyou トピックブランチという手法なんですね　やったことないので実感はわかないんですが、試してみます！

nojimage @nojimage

@uechoco ブランチ切ってないと、PR送った後でもそのブランチにコミットした内容全部リクエストしたことになっちゃうんですよ

うえちょこ🐤🔤 @uechoco

@nojimage PR後にコミットしたことなかったので、初めて知りました　それは・・・切ったほうが絶対いいですね！

Hidenori Goto | COO and CTO at KabuK Style @hidenorigoto

@uechoco Symfonyのドキュメントでもブランチを作ってね、っていう風に書かれてはいます。 http://bit.ly/lxv20Y upstream/masterの最新コミットから分岐した状態にしておくと、マージする人が楽なんですよね。

nojimage @nojimage

@uechoco という失敗をちょっと前にやらかしたばっかりですたい！

うえちょこ🐤🔤 @uechoco

@hidenorigoto マージする人の目線はあんまり体験したことがなくて、そういうことなんですね

うえちょこ🐤🔤 @uechoco

皆さんのおかげで謎が解けました！　あとは実践あるのみですね

Sotaro Karasawa🍺 @sotarok

@uechoco 基本的には人が作ったブランチに(コミッターじゃない人が)勝手にコミットすんなよ、って事だと思う。いくつかのコミットがあった場合なんか目も当てられないw たいていsquashでmergeするしね。

Sotaro Karasawa🍺 @sotarok

なぞがとけたあとの蛇足

Sotaro Karasawa🍺 @sotarok

まぁでも本質的にそういうことだよねーコミットがローカルにできるってことは。

りょう（カロリー不足） @ryokanuma

今はgithub使いではないけれど、subversionから乗り換える動機としては十分に便利そう。 / .@uechoco さんの「githubでpull request毎にブランチを切るべき理由」をお気に入りにしま.. http://togetter.com/li/140708

ooharabucyou @ooharabucyou

@uechoco OpenPNE でも初期はトピックブランチという考え方を知らなかったので、マージがカオスになっている時があったり。