support for clocker set start '10 minutes/hours ago', issue #27

fnogatz / clocker

Command-line tool to track project hours

Other

420 stars 31 forks source link

support for clocker set start '10 minutes/hours ago', issue #27 #29

Closed nevilgeorge-zz closed 8 years ago

nevilgeorge-zz commented 8 years ago

clocker start

clocker status

elapsed time: 00:00:03

clocker set start '10 minutes ago'

clocker status

elapsed time: 00:10:04

clocker set start '1 hour ago'

clocker status

elapsed time: 1:00:04

fnogatz commented 8 years ago

Hi Nevil! Thank you for your contribution. It seems like a good solution for #27. Nevertheless I would prefer to use an existing parsing library like parse-messy-time - just to separate the concerns.

nevilgeorge-zz commented 8 years ago

Ah okay, I will use that and update the PR. Thanks for taking a look at it!

nevilgeorge-zz commented 8 years ago

Just updated. Sorry for the delay.

fnogatz commented 8 years ago

Thanks for the update. I merged and changed the following things:

Moved the parseTime magic into to the updateDate function - otherwise even clocker set message '10 minutes ago' would result in parsed time
As a result, the parsing is applied for clocker set STAMP start '10min ago', too