Insurance::BaseのAPIの位置付けを変更 - Githubissues

IkumaTadokoro / quitcost

「無職になったらいくらかかる？」を簡単に計算できるサービス

https://quitcost.herokuapp.com/

25 stars 2 forks source link

Insurance::BaseのAPIの位置付けを変更 #313

Closed IkumaTadokoro closed 2 years ago

IkumaTadokoro commented 2 years ago

Closes: #311

やったこと

実装意図が分かりづらい（実質的には意味がない）ラッパーメソッド（privateなcalculateをcalcでラップ）を削除
親クラスで実装しているprivateなメソッドについて、今後計算処理を厳密化する場合に子クラス側で拡張する可能性があるのでvisibilityをprotectedに変更

PR提出前のチェックリスト:

[ ] PRの関心はただ一つだけになっている & 文法的に正しく、明確かつ完全なタイトルと本文になっている
[ ] 良いコミットメッセージを書いている
[x] 関連issueがある場合、コミットメッセージにClosing Keywordsを使っている
[x] Featureブランチは最新版のmainブランチに追随している (そうでなければrebaseすること)
[x] 関連するコミットはsquashした
[ ] テストを追加した
[x] bin/lintとbin/rspecを実行した