Improve `tea time` #319

Merged
lunny merged 6 commits from noerw/tea:better-times-are-coming into master 4 months ago
noerw commented 4 months ago
  • 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 4 months ago
noerw added 5 commits 4 months ago
6543 reviewed 4 months ago
cmd/times/list.go Outdated
&cli.BoolFlag{
Name: "mine",
Aliases: []string{"m"},
Usage: "Show Print the total duration at the end",
6543 commented 4 months ago
Poster
Collaborator

cp issue ;)

cp issue ;)
noerw marked this conversation as resolved
cmd/times/list.go
} 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 4 months ago
Poster
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 4 months ago
50c6006767
code review
6543 approved these changes 4 months ago
6543 added the
status/needs-reviews
label 4 months ago
6543 added this to the v0.7.0 milestone 4 months ago
noerw requested review from lunny 4 months ago
lunny approved these changes 4 months ago
lunny merged commit b5c670ebf8 into master 4 months ago
lunny referenced this issue from a commit 4 months ago
6543 deleted branch better-times-are-coming 4 months ago

Reviewers

6543 approved these changes 4 months ago
lunny approved these changes 4 months 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.