search

rdev-hackaton / GitHubTimeTracker

MIT License
5 stars 1 forks source link

[enchancement] allow whitespaces in time string #33

Closed Valian closed 8 years ago

Valian commented 8 years ago

Currently this is wrong because of space between hours and minutes:

:clock1: 1h 12m

Fix regex to allow such notation

ffigiel commented 8 years ago

Shouldn't we keep the time string format as concise as possible? That was the motivation behind such restricted format. Guys?

ffigiel commented 8 years ago

By the way, we should probably drop the support for days (the 1d part), right? This doesn't make much sense

ffigiel commented 8 years ago

Once we decide on that, let's mention the time entry syntax in the readme