whiteleaf7 / narou

Narou.rb - 小説家になろうのダウンローダ&縦書き整形&管理アプリ。Kindle(などの電子書籍端末)でなろうを読む場合に超便利です!
Other
505 stars 55 forks source link

last_commit_year の意図 #397

Closed takotakot closed 2 years ago

takotakot commented 2 years ago

https://github.com/whiteleaf7/narou/blob/6ea823a39fa38ad807fab6b39f50e84e2e8e7c7d/spec/narou_spec.rb#L16

expect(Narou.last_commit_year).to eq Time.now.year

となっています。こちら、どういう意図で入れているテストでしょうか。作業ミス防止であれば、半年程度のマージンを許しても良いかもしれません(あるいは、今年と去年を許すとか)。

396 の circieci のテストは、2022年のコミットがブランチにまだないため、通っていないようです。

whiteleaf7 commented 2 years ago

@takotakot 更新した時に年が変わった場合に Copyright の表記にも反映させているのですが、それの忘れ防止です

whiteleaf7 commented 2 years ago

マージン設けたらそのマージン分気づく時期がずれるだけなので意味ない気がします

takotakot commented 2 years ago

@whiteleaf7 ありがとうございます。納得しました。Close します。