レビュー依頼前のコミット整理方法 - 弥生開発者ブログ

Misoca開発チームのmzpです。 新しいMisocaステッカーが完成したので、いろいろな場所で配りはじめました。 今日は、Misoca内でレビュー依頼をする前にやっているコミットの整理について紹介しようと思います。 Misocaの開発の話ですので、GitHubのpull requestベースでのレビューが前提です。 要約 pull requestをレビューしやすくするために、コミットを整理しよう。 方針 最終的な結果だけを残し、途中の試行錯誤の跡を消す 無関係の内容は別のコミットにする 具体的な作戦 自動生成と非自動生成のものは分ける scaffoldで生成されたものや Gemfile.lock 等の自動で更新されるものは、特にレビューする必要はありません。 そのため、それ以外の手で書いた変更とは別のコミットとは別にしておくとよいです。 試行錯誤の跡を消す PR内で発生したバグの修正コ

[SWIMATH2]

探してたやつだ

[ktanaka117]

コミット整理します、ｽﾏｾﾝ

[masutaka26]

必要以上に整理する必要はないけど、ある程度はね “GitHubのcommitsとコミット順を揃える git rebase --ignore-date origin/master”

[ttskch]

githubのおかげで大枠の論理的説明はPRに任せられるので、各コミットは試行錯誤も含めた現実をログするものとして使ってもいいかなと思ってます

[innx_hidenori]

ここまでやるんですね

[ktkrme]

これやって先輩にフルボッコにされるまでが定番の流れ　http://qiita.com/takasek/items/433931ca5e9b70edbb64

[ledsun]

良い