コードレビューをなくして見えたもの

この記事の内容をざっくり1分20秒でまとめた動画を作っているので、時間がない方はこちらをご覧ください。 はじめに 半年ほど前に、こんな記事を書きました。 人間によるコードレビューをやめて、実装の前に「コンセプト」をチームで確認するワークフローに移行したい、という内容でした。その記事の最後にこんなことを書いていました。 ここまで書いといてアレですが、上記コンセプト立案型ワークフローはまだ実践できてません。実践する中での気づきとか、やっぱり無理とか全然あると思います。それ含めて改善できるといいですね。 上記記事を書いてから時が経ち、実際にチームで半年ちょっとくらいコードレビューをなくしてみたのでその振り返りをしたいと思います。そして見えてきた課題を元に、改善された次のフローをどうするかを考えていきます。 先に結論 人間のコードレビューをやめてもリリース数は落ちずに、リリースまでの時間が減った

[tsukarukatamade]

いままで5分で判断できてたことが、これからは30分かかる。なんかこれをマイナスに感じてしまう錯覚があるね。えらいめにあってだいぶ遠回りさせられたのに、時計見ると時間余ってんだよ

[misshiki]

人間のコードレビューを半年超やめ、週のマージ数は5.1→5.5件、24時間以内のマージ率は65％→77％に。一方、代替の意図確認は形骸化。次はAIが下書きし、人間が下した判断を記録・確認するフローへ。

[noko_wt]

コードレビューをしなくても良い修正としないといけない修正はあるので、担当しているシステムや職責でレビューに対する考えはだいぶ変わる

[unmarshal]

システムアーキテクチャと同じで、テンプレみたいのはあれど正解は無いと思う。プロジェクトやメンバーによって変動するから抽象化しすぎると詰む

[securecat]

yoshitsugumiさん 動画はNotebook LMかなと

[peppers_white]

クライアントでやるのがめんどくさいから一旦確認した後にmuseみたいな自動で催促を送れるエージェントシステムに〆切ぐらいにもうすぐ締め切りだよと警告出してまとまったことを決めるみたいにしてる

[ee83jaige]

「何もしないと人間による関与はゼロに収束していくと思います」っていうのは、それで問題ないからそうなるんだよね。開発においてはもはやAIがあれば人間は解雇でOKってだけなのでは。

[yoshitsugumi]

冒頭の動画どうやって作ってるんだろう？