search

dominikbraun / timetrace

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

Add basic format option #114

Closed sfrique closed 3 years ago

sfrique commented 3 years ago

Initial very basic implementation for #109

I didn't write any test and actually used fmt.Printf should I create an out.Printf or something like this?

Please any suggestions are welcome, I am not really a golang dev =P

To have better options to replace and format, we would need to add more information to the status command.

dominikbraun commented 3 years ago

FYI: I briefly renamed two of the formatting variables, see #123.

sfrique commented 3 years ago

FYI: I briefly renamed two of the formatting variables, see #123.

Cool, I reviewed the PR. One thing to note is that I went for the variable at hand and the Formatter func names. You might want to update the FormatBreakTime and FormatTodayTime as well.