samg / timetrap

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

Add auto sheet support #84

Closed UnderpantsGnome closed 10 years ago

UnderpantsGnome commented 10 years ago

@samg let me know what you think of this approach.

I think it's going to need a way to accept a specific sheet for in/out

Pulling all of the Helpers into Timer feels a bit heavy handed.

I also didn't want to update the readme until I knew if this direction worked.

samg commented 10 years ago

This is looking pretty good to me. I'm planning to find time to take a closer look over the next few days and get this merged in. Thanks for the contribution!

UnderpantsGnome commented 10 years ago

Let me know if you need/want any more help on this.

When I get some time I'm going to play with a similar flow but populating the note. My initial cut will be to auto-note based on git branch. That and the auto-sheet patch will make my workflow super smooth.

samg commented 10 years ago

Merged in 313f16f, 37351e5, a991a93

Thanks! I'll try to get a new gem out soon that includes this. Let me know if you're able to update the README. Otherwise I'll do it before releasing.