Improve `tea time` #319

Merged
lunny merged 6 commits from noerw/tea:better-times-are-coming into master 2 years ago
noerw commented 2 years ago
Collaborator
  • improve doc string
  • adds --fields flag (and more fields to print: id, repo)
  • adds --mine flag to show times of login user across all repos
    • does someone have a better name?
  • move the filtering based on --from, --until to the API side, now that this feature is available since gitea 1.11`
- improve doc string - adds `--fields` flag (and more fields to print: `id`, `repo`) - adds `--mine` flag to show times of login user across all repos - does someone have a better name? - move the filtering based on `--from`, `--until` to the API side, now that this feature is available since gitea 1.11`
noerw added the
kind/enhancement
label 2 years ago
noerw added 5 commits 2 years ago
6543 reviewed 2 years ago
Dismissed
&cli.BoolFlag{
Name: "mine",
Aliases: []string{"m"},
Usage: "Show Print the total duration at the end",
6543 commented 2 years ago
Collaborator

cp issue ;)

cp issue ;)
noerw marked this conversation as resolved
} else if user == "" {
// get all tracked times on the repo
times, _, err = client.ListRepoTrackedTimes(ctx.Owner, ctx.Repo, gitea.ListTrackedTimesOptions{})
times, _, err = client.ListRepoTrackedTimes(ctx.Owner, ctx.Repo, opts)
6543 commented 2 years ago
Collaborator

add fields = []string{"created", "repo", "issue", "user", "duration"} ?

add `fields = []string{"created", "repo", "issue", "user", "duration"}` ?
noerw marked this conversation as resolved
noerw added 1 commit 2 years ago
50c6006767
code review
6543 approved these changes 2 years ago
Dismissed
6543 added the
status/needs-reviews
label 2 years ago
6543 added this to the v0.7.0 milestone 2 years ago
noerw requested review from lunny 2 years ago
lunny approved these changes 2 years ago
Dismissed
lunny merged commit b5c670ebf8 into master 2 years ago
lunny referenced this issue from a commit 2 years ago
6543 deleted branch better-times-are-coming 2 years ago
continuous-integration/drone/pr Build is passing
The pull request has been merged as b5c670ebf8.
Sign in to join this conversation.
Loading…
There is no content yet.