floraison / fugit

time tools (cron, parsing, durations, ...) for Ruby, rufus-scheduler, and flor
MIT License
355 stars 29 forks source link

Eliminate "mismatched indentations" warning by aligning case - when - else properly #46

Closed amatsuda closed 3 years ago

amatsuda commented 3 years ago

Requiring fugit gem causes a Ruby warning "mismatched indentations" in nat.rb

$ ruby -rfugit -we 'p Fugit::VERSION'
.../gems/fugit-1.4.0/lib/fugit/nat.rb:410: warning: mismatched indentations at 'else' with 'case' at 406
"1.4.0"

From 6ce09d0651a2ec8be80cfde7798297b51e966cfe I can guess that this line was intentionally written this way for debugging purpose, but IMO the released gem might better be warning-free.

jmettraux commented 3 years ago

Thanks a lot!

jmettraux commented 3 years ago

@amatsuda fugit 1.4.1 released. Thanks again.

amatsuda commented 3 years ago

@jmettraux Oh, I didn't expect that, but 👍 !