search

dominikbraun / timetrace

A simple CLI for tracking your working time.
Apache License 2.0
685 stars 76 forks source link

#38 revert flag #93

Closed joshuaherrera closed 3 years ago

joshuaherrera commented 3 years ago

This pull request resolves #38 .

Currently, I rely on the user input record / project key to fetch the correct record, since relying on the struct's Key field can cause the record / project to not be found as reported in issues #81 and #89 , though I don't think this will need to change when those issues are resolved.

dominikbraun commented 3 years ago

@joshuaherrera Feel free to ping me once this is ready to review!

joshuaherrera commented 3 years ago

@dominikbraun This is ready, I won't make anymore changes, unless requested.

dominikbraun commented 3 years ago

This PR is contained in release 0.8.0.

joshuaherrera commented 3 years ago

LGTM, thanks for implementing this powerful feature! This will be the main feature of the next minor release.

It was a pleasure! I really enjoy the project layout you set forth in timetrace, it makes coding quite enjoyable!