add times ls #242

New Issue

Based on looking into the code there seemed to be one inconsistence (based on my point of view) because times.go named all cmd parts like CmdTrackedTimes... in controdiction to other parts issues.go which are named CmdIssues*

I would have expected to name the cmd in times.go like CmdTimes... instead?

Is there a reason for that? Or do I just not seeing the whole picture?

Based on looking into the code there seemed to be one inconsistence (based on my point of view) because `times.go` named all cmd parts like `CmdTrackedTimes...` in controdiction to other parts `issues.go` which are named `CmdIssues*` I would have expected to name the cmd in `times.go` like `CmdTimes...` instead? Is there a reason for that? Or do I just not seeing the whole picture?

noerw commented

2020-12-06 23:22:37 +00:00

"times" seemed too imprecise for function names, and "tracked times" too long for UX.
¯_(ツ)_/¯

"times" seemed too imprecise for function names, and "tracked times" too long for UX. ¯\_(ツ)_/¯

khmarbaise referenced a pull request that will close this issue

2020-12-06 23:38:11 +00:00

add `times ls` #272

noerw added the

status/has-pull

label 2020-12-07 00:00:41 +00:00

6543 referenced this issue from a commit

2020-12-07 12:29:49 +00:00