GitHubへpull requestする際のベストプラクティス - hnwの日記

みなさん、Git使ってますか？僕はまだメインのVCSがSubversionなのもあって、なかなか慣れません。せっかくGitを使っているのに、ちょっと不便なSubversionくらいの位置づけです。でも、同じような理解度の人って多いんじゃないでしょうか。 一方で、最近はGitHub管理のオープンソースプロジェクトが増えてきました。バグレポートを送るにしてもpull request*1が前提のような空気があり、Git初心者には少し敷居が高い印象があります。 そんな僕も先日初pull requestをしてみたんですが、色々な失敗の積み重ねで残念なpull requestになってしまいました。その反省を元に、本稿ではpull requestする際のベストプラクティスを紹介します。これは「Git Workflow」をベースにコマンド例などを加筆したものです。 概要 pull requestする際は、

[miau]

「過去のcommitを1つにまとめる」んじゃなくて、変更点がわかりやすい単位でcommitをまとめたほうがいいような気がするけど、どうなんだろう。

[uasi]

自分でも試行錯誤してこのプラクティスに行き着いた、これホント苦労なくて済むので万人が実践すべき

[taka_2]

"面倒すぎるだろ！と感じた人がいるかもしれません。"／うん、めんどくさい。。

[PoohKid]

こちらはオープンソースプロジェクト向けなので読者の9割(決めつけ)の社内開発では同一リポジトリでブランチ切る方がオススメ http://blog.qnyp.com/2013/05/28/pull-request-for-github-beginners/

[ledsun]

丁寧に説明してもらってるはずなのに、よくわかりません。／（3年後）ようやく、息をするように、理解できた。

[serihiro]

人生初のpull reqを送る前に

[iaskell]

なるほどわからん

[yysaki]

GitHubへpull requestする際のベストプラクティス

[machupicchubeta]

"git remote add upstream"

[nkuri]

参考になります。

[crist18]

fork

[kjw_junichi]

ここら辺読んで作業するべきだった。。

[hashrock]

git rebaseでコミットをまとめてPR作る

[netoresama]

覚える

[kgrock]

GitHubへpull requestする際のベストプラクティス

[kikeda1104]

fork先のcommitをrebaseする方法 [git][github]

[tksmd]

pr のやり方

[puhitaku]

プルリク送ったった

[h5y1m141]

これ読んで大分理解出来てきた気がする。

[Itisango]

“masterブランチで作業してはいけません。また、masterブランチからpull requestを送るのもいけません。必ずpull requestのための別ブランチから送るようにしましょう。これにはいくつか理由がある”

[tvsk]

branchからpull request

[takeshich]

sourceforgeへのRequestMergeの参考にしてみる

[azihsoyn]

“作業ブランチに移動 $ git checkout myFeatureSpike Switched to branch 'myFeatureSpike' $ 作業ブランチを元にpull request用のブランチを作成・移動 $ git checkout -b myFeature Switched to branch 'myFeature' $ 作業ブランチ上のfork元からの差分を1 commitにま

[tohokuaiki]

なるほど。

[minamiyama1994]

そういえばブクマってなかったので今更ながらブクマっておく、非常によくまとまっているのでpullreq分らん各位は読んだらいい気がする(ほんまか)

[sechiro]

“git remote add upstream”

[re_guzy]

ものすごく参考にした

[d_animal141]

GitHubへpull requestする際のベストプラクティス

[skit_n]

プルリクベースの開発スタイルをこれから学ぶ身にとってとても参考になった記事。

[Nyoho]

fork したあとに元のレポジトリに追従する方法も含めて丁寧に書いてある

[DrFaust]

pull request.

[atsumo]

pull reqベストプラクティス

[mtrock]

“git stash”

[hidehara]

あ、masterのままpull reqやってもた。。。早く読んで修正しないと！

[Mrk1869]

これを参考に初pull request送った。かなり緊張した...!!