tokuhirom's blog

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

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

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