search

samg / timetrap

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

Sequel update #165

Closed jostber closed 6 years ago

jostber commented 6 years ago

Remove sequel dependency version specification to make install more flexible.

patrickdavey commented 6 years ago

I think the formatter should be a pull request into the custom formatters repo here

Also, while I'm not sure if samg is going to approve this anyway, I suggest at least putting a minimum version on the sequel library would be better than leaving it completely open. I can understand however pegging it to a major version (if perhaps not a patch)

jostber commented 6 years ago

On 19.09.17,21:04, Patrick Davey wrote:

I think the formatter should be a pull request into the custom formatters repo here

Also, while I'm not sure if samg is going to approve this anyway, I suggest at least putting a minimum version on the sequel library would be better than leaving it completely open. I can understand however pegging it to a major version (if perhaps not a patch)

Seems like some cruft got into this pull request. I will close it and resubmit.

samg commented 6 years ago

Happy to merge this the sequel version change once you resubmit.

Thanks!