Pull Request 時のレビューについて
コメントの残し方
レビューをいうのは意図と違った受け取り方をされやすいものなので、温度感をコメントで補うようにしています。
- [MUST]
- 絶対直してほしいなというときに付けます。強い指摘になるので、言葉遣いには少し気をつけます。
- [IMO] (In my openion の略です)
- 自分ならこうするかな〜?、自分はこう思うのだけどどうだろう?というときに使います。
- [NITS] (Nits Pick の略)
- 強制ではないが細かい指摘や、軽い修正をして欲しいときに使います。
- [Q] (Question の略)
- ここわからないでの教えて!という時に付けます。実装で不明瞭だった点や、そもそも自分の知らないことについて聞いたりします。