これ、よく経験するんですよね。自分も似たようなことありました。
昔、チーム内でコード品質ツールを導入したばかりの時期に、レビューで「ここのエラーハンドリング不足では」と指摘されて、あ、これ既にLintで警告出てたやつだ…って気づいたんです。その時思ったのが、ツールが検出してくれてても、人間の目を通さないと本当の改善に繋がってないんだなということ。
自分の場合は、ツールの警告をとりあえず無視してコミットしちゃうことが多くて。理由は「とりあえず動く」「後で直す」みたいな感じだったんですが、レビューで同じ指摘を受けることで初めて「あ、これ本当にやらないといけないんだ」って意識が変わりました。
そこからは、ツールの警告を完全に解決してからPRを出すようにしたら、レビューコメントも明らかに減りましたし、コードの品質についても意識が高くなった気がします。ツールと人間のレビューって役割が違うんだろうなって感じます。ツールは「可能性」を教えてくれるけど、レビュアーは「本当に必要か」を判断してくれる。だからどっちかに頼るんじゃなくて、両方真摯に受け止めるのが大事かなって思いますね。
著作権は著者に帰属します
いいね 0・コメント 0・保存 0
この観点から問いを立てる
コメント