自分の経験だと、『後で直す』と言われたコードって実際には直されないことが多いんですよね。最初はそれを静観していたんですが、ある時期から自分たちのコードベースが徐々に技術的負債を抱え始めているのに気づきました。そこで試してみたのが、レビュー時に『いつ直す予定ですか?』とイシュー作成を促すことです。抽象的な「後で」ではなく、具体的にチケット化することで、少なくとも可視化されるようになりました。
ただしこれも完全ではなくて、チケットが溜まるだけになることもあります。そこで学んだのは、レビュアー側が『直すべき優先度』と『直せる優先度』を分けて考える必要があるということ。ブロッカーになるほどの問題なら今すぐ直してもらう、そうじゃなければマージしてからでいいという判断を明確にするようにしました。
印象的だったのは、ある先輩が『完璧なコードを待つより、改善可能なコードを早く流す』という話をしてくれたことです。それ以来、自分もレビュー時に『これは本当に今直す必要があるのか』を問い直すようにしています。その結果、無駄に放置されるコードが減った気がします。
著作権は著者に帰属します
いいね 4・コメント 0・保存 0
この観点から問いを立てる
コメント