コミットは作業ログではない！ - Qiita

いきなりだけど、こんなコミット、よく見ない？ （例１とする） 見た瞬間に腕まくりして修正依頼と理由を書き出したくなっちゃう感じ！ これが良くない理由を、持論だけど挙げてみるよ このコミットについて コミットは作業単位でするものではない プルリクエスト（以下 PR）をレビューする側に立って考えるとよくわかるのだけど、 PRレビューをする人が欲しているのは(= masterブランチが欲している、チームが欲していると言っても良い）作業者の作業ログではない。 レビューする人にとって、そのPR作成者が「どれくらいの時間をかけた」とか「休日もコミットを重ねてた」とか 「何ファイルも直さなくちゃあならなくて大変だった」とか「いつどこでブランチを切り換えて自分の出したPRを見てくれていた」とか、 そんなことはぶっちゃけ全く興味がないんだ。 欲しいのは「生成物と成果」であって、それを「どの様な作業工程」で作

[k12u]

コミットは作業ログにも使えるし、別にそれは悪ではない。全てはワークフロー次第。ワンフレーズで雑に大きいことを言うのは割と悪。

[tmatsuu]

作業ログとしてコミットしてもいいと個人的には思う。PRする際にそのような作業ログはsquashすると良い。逆に作業ログ以外はsquashするべからず。きれいなPRを心がけたい。

[restartr]

気持ちはわかる。けど、PRで簡潔に箇条書きしてもらうだけでレビューコストは十分に下げられる。

[mattn]

「コミットメッセージはお前のツイッターじゃあねーから（怒）」

[haru-s]

クラス名を変えたくなるのは変更の途中だったりするからなあ。一旦途中の変更を元に戻して…は、カオスになるし。

[src256]

全部 fix bug

[tkmoteki]

ごもっとも。コミットは作業ログではない via @nuzzel

[koogawa]

「レビュー直した」も困る