エントリーの編集
エントリーの編集は全ユーザーに共通の機能です。
必ずガイドラインを一読の上ご利用ください。
記事へのコメント0件
- 注目コメント
- 新着コメント
このエントリーにコメントしてみましょう。
注目コメント算出アルゴリズムの一部にLINEヤフー株式会社の「建設的コメント順位付けモデルAPI」を使用しています
- バナー広告なし
- ミュート機能あり
- ダークモード搭載
関連記事
リリースフラグによってプルリクを小さくする方法
先にまとめ 変更は追加とリリースと削除が混同するので、レビューコストが高い リリースフラグを用い、... 先にまとめ 変更は追加とリリースと削除が混同するので、レビューコストが高い リリースフラグを用い、旧実装を残すことで、新実装を追加行のみで表現できる リリースフラグの反転だけで新実装への切り替えができ、切り戻しなどが容易になりやすい 解説しようとしていること 「変更」を「追加」と「削除」に分ける リファクタリングと仕様変更はプルリクとしては分けた方がレビューコストが下がる、というのは有名な話ですし、実際にレビューをしたことがある方々なら実感できるでしょう。 参考: レビューをもらいやすい細かいプルリクの切り分け方 - Software engineering from east direction 本記事では加えて、「変更」を「追加」と「削除」に分けた方が良い、という話をします。 追加・削除・変更のプルリクを見たときに、レビュアは以下のような観点でそれぞれレビューするかと思います。 追加: