Closed IkumaTadokoro closed 2 years ago
住民税の会計年度(徴収)の開始は6月であるため、Fiscaliの判定ロジックを無理やり前後させて判定させていた。 月の前後の判定がインラインとなっており、意図が掴みにくいため、別途メソッドとして抽出した。
なおrefinementをFiscaliに適用することも検討したが、refineのスコープの制限ゆえに大量のメソッドオーバーライドが発生する。変更範囲を最小限にするためにActiveSupport::TimeWithZoneに対してrefineを行う方針とした
Closes #119
PR提出前のチェックリスト:
main
bin/lint
bin/rspec
住民税の会計年度(徴収)の開始は6月であるため、Fiscaliの判定ロジックを無理やり前後させて判定させていた。 月の前後の判定がインラインとなっており、意図が掴みにくいため、別途メソッドとして抽出した。
なおrefinementをFiscaliに適用することも検討したが、refineのスコープの制限ゆえに大量のメソッドオーバーライドが発生する。変更範囲を最小限にするためにActiveSupport::TimeWithZoneに対してrefineを行う方針とした
Closes #119
PR提出前のチェックリスト:
main
ブランチに追随している (そうでなければrebaseすること)bin/lint
とbin/rspec
を実行した