samg / timetrap

Simple command line timetracker
http://rubygems.org/gems/timetrap
Other
1.48k stars 115 forks source link

Add option '-s|--sheet' for the `in` and `resume` commands #139

Closed ashumkin closed 7 years ago

ashumkin commented 8 years ago

It would more convenient to use these commands with such an option. in command can start the task in the specified timesheet without a need to switch to the latter. resume command can resume the last task in the CURRENT sheet but to the last stopped task (from another sheet in common case).

samg commented 7 years ago

Looks like this is an approach to the same problem described in https://github.com/samg/timetrap/pull/116#issuecomment-239144937 .

Do you agree it'd be better to just make resume operate reopen the last entry of the current sheet, which was the previous behavior It would simplify the interface and avoid the need to add a --sheet flag to resume.

ashumkin commented 7 years ago

Do you agree it'd be better to just make resume operate reopen the last entry of the current sheet,

Yes, I agree. This was the only reason for me to implement -s option (just not to break the exising behaviour) for resume command.

samg commented 7 years ago

https://rubygems.org/gems/timetrap/versions/1.12.0 has a fix for this.

ashumkin commented 7 years ago

thank you!