俺がレビューコメントでよくつけるやつ

基本的にメンバーは良いコードを書くという前提で暮らしています。

• 良いと思います。テストがあるともっと良いと思います。
• 変更が軽微で、まあテストなくてもいいけど、あったほうがより良いよねえ、という時に使う
• 動いてるんならいいと思います
• 処理が複雑なので、実際に動かしてみないとうまく動くかどうかコードをパッと見ただけだったらよくわからないけど、動いてるんならちゃんと動いてるんだろうな、という気がするときに使います。
• lgtm
• シフトキーをオスのもめんどくさいなって時には小文字で書きます。
• コメントが無いとあとでなんだかわからなくなっちゃうかも?
• ちょっと複雑な処理なのにコメントないときとか
• wiki へのリンクを貼ってくだされ~
• 仕様がちょっと複雑だったり、後から経緯おえないと死ぬな、という時に言います
• 日本語でOK
• 英語でコメントついてるけど、コメントの内容が端折り過ぎで説明不足なときにいいます

Published: 2016-08-04(Thu) 02:48