サクッとレビューができる 小さなPull Requestを作るには - LIVESENSE ENGINEER BLOG

大きなPull Requestのレビューがつらい 修正ファイル数が多いこと自体が問題なのではない 1つの内容に集中する 小さなPull Requestの作り方 リファクタリングの修正は気になっても別で出す Web API 1つに着目して実装を切り分ける 小さなPull Requestで作ったときのリリースの仕方 featureブランチを作って、そこから更にブランチを作っていく フィーチャートグルを使う 小さいPull Requestで小さくフィードバックをもらおう 大きなPull Requestのレビューがつらい 転職ドラフトでWebアプリケーションエンジニアをしている @iwtn です。 この記事ではチーム開発では当たり前になったレビューにおいて、修正されたファイルがたくさんあるとつらいよね、というお話と、その解決策を提示してみたいと思います。 昨今のWebアプリケーションなどのチーム開

[iwtn]

セルクマ。こんなこと考えてコード書いてます。

[kimihiro031]

でっかいPRを過去に出してしまって後悔が絶えない。結局小さいほうがレビュアーにも優しいし、早く終わるんだよな...

[shun9167]

同じイメージがある。小さくしてくれたらすぐレビューするのにって思うことが幾度となくある。 “個人的にレビューの負荷というものは、そのPull Requestに含まれる修正量の2乗に比例するぐらいのイメージがあります。”

[atico]

レビューで大事なのは実装内容を理解できるかできないかなので、見た感じ良さそうで済ますのではなくて、一通り実装者にざっくり説明してもらうのがいいと思う。簡単なプルリクを除いて。

[sheep_san_white]

ついついモリモリにしがちなので、スコープを切り出すのは大事という話。