タグ

関連タグで絞り込む (1)

タグの絞り込みを解除

PRとgitに関するbraitomのブックマーク (1)

  • commit をどう分割すべきか 〜コードレビューの観点から〜

    普段の開発の中で git の commit の単位に気を遣っている人もいると思うんですが、どういう単位で commit すべきかみたいな話をあまり見かけない気がします。自分自身 GitHub の Pull Request(以降 PR)ベースのチーム開発を何年か経験してみて、「こうすると良さそう」というものが見えてきた気がするのでまとめてみました。 なお、小さい単位で PR を出す方針にしている場合は、以降の内容に出てくる commit を適宜 PR で読み替えてもらうと良いかもしれません。 そもそも commit の単位に気を遣った方が良いのか? コードレビューを行う場合など、他者がコードを読む場合はある程度気を遣った方が良いと思っています。理由は次のとおりです。 コードレビューにおいてレビュアーのレビュー時間が短縮できる コードレビューの精度が上がる 変更の経緯を追いやすい 自分にとって

    commit をどう分割すべきか 〜コードレビューの観点から〜
    braitom
    braitom 2018/01/09
    コミットの粒度について。コミットの粒度に気を使った方よい理由、どう分割するとよいのか、アンチパターンがまとめられている。
  • 1