Changesets + GitHub Actionsの連携を調べているがまだよくわからないことが多い。




pre, postをつけると前後で実行するコマンドになる。

今日読んで気になった記事のメモ。


だから、基本的には良かれと思って既存コードとコーディングスタイルを合わせようとする。あるいは、これは善し悪しがある、まあ悪い方が多いんですけれど、機能追加の際などにレビュワーの負担を下げようとしてコードのDiffを小さくしようとするんです。
本当は、このインデントの構造を変えた方がいいんだけど、それをするとDiffが大きくなるから止めよう、とか、本当はメソッドに抽出したほうがいいんだけど、それだとDiffが大きくなるからこのif文を追加で、みたいな感じでDiffを小さくしようとする。
(GitHub上などで)Diffでコードレビューするようになったことで、Diffの小ささがよしとされてしまう。これをやや俯瞰で見ると、構造的複雑さとか命名的な悪さというのが、じわじわと累積されていく形になるんですね。
そういうのが何年も積み重なると、現場のレガシーコードが出来上がる。
レガシーコードの「命名的問題」「構造的問題」のどちらを優先してリファクタリングすると保守性や可読性が高くなるかの調査のレポート記事。引用はレガシーコードが出来上がる過程のあるあるを丁寧に文章にしてあって好きだった部分。