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

まとめました。
3
うえちょこ🐤PUI PUI🐹 @uechoco

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

2011-05-27 08:52:53
shishi @shishi4tw

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

2011-05-27 08:58:36
うえちょこ🐤PUI PUI🐹 @uechoco

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

2011-05-27 09:03:31
nojimage @nojimage

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

2011-05-27 09:06:06
うえちょこ🐤PUI PUI🐹 @uechoco

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

2011-05-27 09:07:02
Hidenori Goto @hidenorigoto

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

2011-05-27 09:07:34
nojimage @nojimage

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

2011-05-27 09:09:31
うえちょこ🐤PUI PUI🐹 @uechoco

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

2011-05-27 09:12:43
うえちょこ🐤PUI PUI🐹 @uechoco

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

2011-05-27 09:13:51
Sotaro Karasawa🍺 @sotarok

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

2011-05-27 09:23:48
Sotaro Karasawa🍺 @sotarok

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

2011-05-27 09:26:30
りょう(カロリー不足) @ryokanuma

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

2011-05-27 09:36:06
ooharabucyou @ooharabucyou

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

2011-05-27 09:45:16

コメント

コメントがまだありません。感想を最初に伝えてみませんか?