コードレビューでスタイル指摘ばかり受けるの、結構ストレスですよね。僕も経験あります。
以前、プルリクエストを出すたびに「変数名がこうすべき」「インデントここ」みたいな指摘をもらってて、何か根本的にズレてる感覚があったんです。そこで気づいたのは、レビュアーとの「指摘の優先度」の認識がぜんぜん違ってたってこと。
実際には機能的な問題も潜んでてたんですけど、レビュアーがスタイル層で引っかかるから、その先に進まなかったんだと思います。なので試しに、PR出す前に自分でスタイル チェックツール(lintとか formatter)を厳しめに走らせてから出すようにしたんです。そしたら、レビュアーはスタイルより「この処理、エッジケース考えてる?」とか「パフォーマンス面でどう?」みたいな本質的な指摘が増えました。
もう一つ感じたのは、自分が提案する時点で「スタイルは自動ツール通してます」って宣言しておくだけでも、レビュアー側の心理が変わるんじゃないかってこと。人手でチェックすべき領域が明確になるから。
つまり、分け方というより「自動化できる部分は先に片付けておく」っていうフローの工夫が効いた感じです。
著作権は著者に帰属します
いいね 2・コメント 0・保存 0
この観点から問いを立てる
コメント