テクノロジー

コードレビュー指摘300件を3ヶ月分類したら効いていたのは2種類だけだった

1: hkdn 2026/07/06 04:20

なるほど、正しいかも。

2: fa11enprince 2026/07/06 05:14

“RefactorとArchitectureは、PRレビューの場では遅すぎる。” それはその通りかも!。 他の検証内容も興味深い。

3: taguch1 2026/07/06 06:36

とりあえず動くからマージして後からリファクタリングみたいな対応はメンテナンスの難易度を数倍にするので好きじゃない。なんとなくでかいプルリクな感じがして嫌な予感もする。

4: sirobu 2026/07/06 06:40

styleは人力レビューだと好みの問題やコーディング規約読んでない愚か者が出てくるから機械的に指摘、修正させるのに賛成

5: nguyen-oi 2026/07/06 07:10

linterで自動化できる指摘に時間使うのはマジでリソースの無駄やな

6: chikoshoot 2026/07/06 07:50

どちらかと言えば、設計工程で流出していたことに気がついた、だな。

7: clairvy 2026/07/06 07:54

イマイチ状況がわからん。何を評価したいんだろ?

8: lenore 2026/07/06 09:07

スタイルは強制的に適用しとくし潜在バグやある程度の命名規約は静的解析にかける。人間やAIはそれ以外の不具合のレビューに注力するのでいいのでは。アーキテクチャは設計の段階で指摘漏れしたヤツかな

9: choota 2026/07/06 09:21

nits指摘、案件入場直後のタイミングなんかでへーそうなんや!みたいな知識を授けてもらえたりしたなぁ。PR外でそういうレベル合わせの場を持ちたい。

10: tettekete37564 2026/07/06 12:05

“仮説はこうだ。レビュアーの認知容量は有限で、nit:に30件コメントしたPRでは、Bug候補の検出感度が落ちる。書く側も「全部直さなきゃ」で疲れ、設計判断への思考が浅くなる。” < 説得力のある考察

11: morimarii 2026/07/06 12:38

コードが良くなったか?ってどうやって根拠は何? 開発速度も微妙すぎてなにも言えない

12: north_korea 2026/07/06 12:43

フックとかフォーマッターの話ってAI開発黎明期によくあったやつだな。今そこで悩んでいる人はもういないから懐かしい