Norwin
802bdf7dc5
Don't require a body for comment PR reviews ( #399 )
...
fixes #372
Co-authored-by: Norwin <git@nroo.de >
Co-authored-by: 6543 <6543@obermui.de >
Reviewed-on: https://gitea.com/gitea/tea/pulls/399
Reviewed-by: 6543 <6543@obermui.de >
Reviewed-by: Andrew Thornton <art27@cantab.net >
Co-authored-by: Norwin <noerw@noreply.gitea.io >
Co-committed-by: Norwin <noerw@noreply.gitea.io >
2021-09-23 02:18:25 +08:00
Norwin
1731e00ebd
Don't skip reading the local repo when --repo specifies a repo slug ( #398 )
...
I added this check in #327 , but it wasn't needed at all
as the error case it intended to catch where already handled by checking if the path exists.
fixes #378
Co-authored-by: Norwin <git@nroo.de >
Reviewed-on: https://gitea.com/gitea/tea/pulls/398
Reviewed-by: 6543 <6543@obermui.de >
Reviewed-by: Andrew Thornton <art27@cantab.net >
Co-authored-by: Norwin <noerw@noreply.gitea.io >
Co-committed-by: Norwin <noerw@noreply.gitea.io >
2021-09-23 00:12:56 +08:00
Norwin
7b7c7f57be
tea pr create: make local repo optional ( #393 )
...
this is a partial fix to #378 , making the command available outside of a local repo.
new behaviour:
- when run interactively without local repo context, the head repo prompt is not pre-populated
- when run with flags without local repo context, it will complain unless `--head` is specified
refactor:
- pass TeaContext down to task.CreatePull
Co-authored-by: Norwin <git@nroo.de >
Co-authored-by: 6543 <6543@obermui.de >
Reviewed-on: https://gitea.com/gitea/tea/pulls/393
Reviewed-by: Alexey 〒erentyev <axifive@noreply.gitea.io >
Reviewed-by: 6543 <6543@obermui.de >
Co-authored-by: Norwin <noerw@noreply.gitea.io >
Co-committed-by: Norwin <noerw@noreply.gitea.io >
2021-09-22 23:48:21 +08:00
Norwin
6e728cf812
Accept more main branch names for login detection ( #396 )
...
Also consider `main` and `trunk` as options
to determine a login through its configured remote
fixes #381
Co-authored-by: Norwin <git@nroo.de >
Reviewed-on: https://gitea.com/gitea/tea/pulls/396
Reviewed-by: Lunny Xiao <xiaolunwen@gmail.com >
Reviewed-by: Alexey 〒erentyev <axifive@noreply.gitea.io >
Co-authored-by: Norwin <noerw@noreply.gitea.io >
Co-committed-by: Norwin <noerw@noreply.gitea.io >
2021-09-14 14:59:11 +08:00
Norwin
808e8b1c5a
Correctly match login by ssh host with port ( #391 )
...
fixes #380
note: It seems like it was expected that `SSHHost` only contains the host portion. So this may be breaking (although I don't believe many people used the feature like that with a custom ssh port). I can't think of a good reason to *not* specify the port in that field, including the port seems more intuitive
Co-authored-by: Norwin <git@nroo.de >
Reviewed-on: https://gitea.com/gitea/tea/pulls/391
Reviewed-by: Andrew Thornton <art27@cantab.net >
Reviewed-by: Lunny Xiao <xiaolunwen@gmail.com >
Co-authored-by: Norwin <noerw@noreply.gitea.io >
Co-committed-by: Norwin <noerw@noreply.gitea.io >
2021-09-06 18:52:34 +08:00
Norwin
9201250f74
fix printing issue deadline ( #388 )
...
fixes #387
Reviewed-on: https://gitea.com/gitea/tea/pulls/388
Reviewed-by: Lunny Xiao <xiaolunwen@gmail.com >
Reviewed-by: 6543 <6543@obermui.de >
Co-authored-by: Norwin <noerw@noreply.gitea.io >
Co-committed-by: Norwin <noerw@noreply.gitea.io >
2021-09-06 18:45:24 +08:00
Norwin
5b28a05eb7
Implement notification subcommands ( #389 )
...
- [x] enhance notification listing
- add `--states` and `--type` filters
- toggle per-user or per-repo listing via `--mine` flag
- print more fields
- [x] add subcommands to mark notifications as read, unread, pinned, unpinned. operates on
- all notifications matching the `--state` and `--mine` filter flags, or
- a notification specified by ID.
- [ ] ~~add a `--fields` flag for notifications listing.~~ *not in this PR*
- [ ] ~~interactive mode~~ *not in this PR*. this would go well together with #324
fixes #243 , fixes #155
based on initial work in #283 and #386 , but opening a new PR for @6543 to review as I changed quite a lot
---
### ⚠️ breaking ⚠️
- `tea notifications --all` has moved to `tea notifications --mine`
- `tea notifications` now only works with the context of a remote repo.
To run this outside of a local git dir, run either `tea n --mine` or `tea n --repo <my/repo>`
---
Co-authored-by: Karl Heinz Marbaise <kama@soebes.de >
Co-authored-by: 6543 <6543@obermui.de >
Co-authored-by: Norwin Roosen <git@nroo.de >
Reviewed-on: https://gitea.com/gitea/tea/pulls/389
Reviewed-by: 6543 <6543@obermui.de >
Reviewed-by: Alexey 〒erentyev <axifive@noreply.gitea.io >
Co-authored-by: Norwin <noerw@noreply.gitea.io >
Co-committed-by: Norwin <noerw@noreply.gitea.io >
2021-09-06 01:11:17 +08:00
Norwin
3fca309f2c
Fix adding login without token on private instances ( #392 )
...
fixes #365
Co-authored-by: Norwin <git@nroo.de >
Reviewed-on: https://gitea.com/gitea/tea/pulls/392
Reviewed-by: 6543 <6543@obermui.de >
Reviewed-by: Andrew Thornton <art27@cantab.net >
Co-authored-by: Norwin <noerw@noreply.gitea.io >
Co-committed-by: Norwin <noerw@noreply.gitea.io >
2021-08-30 23:19:45 +08:00
Norwin
d6df0a53b5
Update Dependencies ( #390 )
...
Co-authored-by: Norwin Roosen <git@nroo.de >
Co-authored-by: Norwin <git@nroo.de >
Reviewed-on: https://gitea.com/gitea/tea/pulls/390
Reviewed-by: 6543 <6543@obermui.de >
Reviewed-by: Andrew Thornton <art27@cantab.net >
Co-authored-by: Norwin <noerw@noreply.gitea.io >
Co-committed-by: Norwin <noerw@noreply.gitea.io >
2021-08-30 23:18:50 +08:00
6543
4b9907fb54
Notifications Add State Field ( #384 )
...
Reviewed-on: https://gitea.com/gitea/tea/pulls/384
Reviewed-by: Lunny Xiao <xiaolunwen@gmail.com >
Reviewed-by: Andrew Thornton <art27@cantab.net >
Reviewed-by: Norwin <noerw@noreply.gitea.io >
Co-authored-by: 6543 <6543@obermui.de >
Co-committed-by: 6543 <6543@obermui.de >
2021-08-16 21:23:16 +08:00
6543
ab4e11ae4d
Update gitea go-sdk to v0.15.0 ( #385 )
...
Update "code.gitea.io/sdk/gitea" to latest release
Reviewed-on: https://gitea.com/gitea/tea/pulls/385
Reviewed-by: Lunny Xiao <xiaolunwen@gmail.com >
Reviewed-by: Andrew Thornton <art27@cantab.net >
Co-authored-by: 6543 <6543@obermui.de >
Co-committed-by: 6543 <6543@obermui.de >
2021-08-16 20:46:15 +08:00
nukuta
546fcc16de
Handle XDG directories with spaces in autocomplete commands ( #383 )
...
On some OSs ([i.e. macOS](https://github.com/adrg/xdg#xdg-base-directory )), XDG directories contain spaces, so we need to wrap the resulting path used in autocomplete sourcing commands in quotation marks.
Co-authored-by: Matthew Daley <mattd@bugfuzz.com >
Reviewed-on: https://gitea.com/gitea/tea/pulls/383
Reviewed-by: Andrew Thornton <art27@cantab.net >
Reviewed-by: Norwin <noerw@noreply.gitea.io >
Co-authored-by: nukuta <nukuta@noreply.gitea.io >
Co-committed-by: nukuta <nukuta@noreply.gitea.io >
2021-07-24 00:14:23 +08:00
Norwin
0f4f669cf0
Fix parsing of --description for issue/pr create ( #371 )
...
The `--description` flag didn't work; regression of #331
Co-authored-by: Norwin Roosen <git@nroo.de >
Co-authored-by: 6543 <6543@obermui.de >
Reviewed-on: https://gitea.com/gitea/tea/pulls/371
Reviewed-by: Lunny Xiao <xiaolunwen@gmail.com >
Reviewed-by: KN4CK3R <kn4ck3r@noreply.gitea.io >
Co-authored-by: Norwin <noerw@noreply.gitea.io >
Co-committed-by: Norwin <noerw@noreply.gitea.io >
2021-07-03 23:23:38 +08:00
Norwin
2bdd72dfff
Improve error messages ( #370 )
...
fixes #367
Co-authored-by: Norwin Roosen <git@nroo.de >
Co-authored-by: Lunny Xiao <xiaolunwen@gmail.com >
Reviewed-on: https://gitea.com/gitea/tea/pulls/370
Reviewed-by: 6543 <6543@obermui.de >
Reviewed-by: KN4CK3R <kn4ck3r@noreply.gitea.io >
Co-authored-by: Norwin <noerw@noreply.gitea.io >
Co-committed-by: Norwin <noerw@noreply.gitea.io >
2021-07-03 22:39:05 +08:00
techknowlogick
64770a771f
rename s3 bucket name ( #375 )
...
Reviewed-on: https://gitea.com/gitea/tea/pulls/375
Co-authored-by: techknowlogick <techknowlogick@gitea.io >
Co-committed-by: techknowlogick <techknowlogick@gitea.io >
2021-06-30 04:36:05 +08:00
6543
ebb2c38a0a
Return useful error on wrong sshkey path ( #374 )
...
close #366
Reviewed-on: https://gitea.com/gitea/tea/pulls/374
Reviewed-by: Lunny Xiao <xiaolunwen@gmail.com >
Reviewed-by: Norwin <noerw@noreply.gitea.io >
Co-authored-by: 6543 <6543@obermui.de >
Co-committed-by: 6543 <6543@obermui.de >
2021-06-29 15:54:43 +08:00
Norwin
616127cedc
Add missing flags ( #369 )
...
fixes #368
Co-authored-by: Norwin Roosen <git@nroo.de >
Co-authored-by: Lunny Xiao <xiaolunwen@gmail.com >
Reviewed-on: https://gitea.com/gitea/tea/pulls/369
Reviewed-by: Andrew Thornton <art27@cantab.net >
Reviewed-by: Lunny Xiao <xiaolunwen@gmail.com >
Co-authored-by: Norwin <noerw@noreply.gitea.io >
Co-committed-by: Norwin <noerw@noreply.gitea.io >
2021-06-22 22:23:09 +08:00
plgruener
3129e60a73
text editor selection: follow unix defacto standards ( #356 )
...
Currently, `tea` only supports the $EDITOR env var to open the user's preferred editor (used for reviewing pull requests).
Standard \*nix practice is, however, to check for $VISUAL first and only then use $EDITOR as fallback.
This is also done by Git itself, see man git-var(1).
(Actually, the order there is $GIT_EDITOR > core.editor > $VISUAL > $EDITOR > vi)
Co-authored-by: plgruener <pl.gruener@gmail.com >
Reviewed-on: https://gitea.com/gitea/tea/pulls/356
Reviewed-by: Norwin <noerw@noreply.gitea.io >
Reviewed-by: 6543 <6543@obermui.de >
Co-authored-by: plgruener <plgruener@noreply.gitea.io >
Co-committed-by: plgruener <plgruener@noreply.gitea.io >
2021-06-21 20:08:27 +08:00
Norwin
df724b4006
Add tab completion for fish shell ( #364 )
...
as title, fixes #361
Handling of fish shell is different in urfave/cli; urfave/cli provides a generator for the shell script needed (probably because the fish `completion` syntax isn't flexible enough to let the application handle the completion at runtime? idk)
This means that the fish completion can become out of sync with the tea binary.
If we want to account for that, on each application run we need to
- check if `~/.config/fish/conf.d/tea_completion.fish` exists; if so
- check if the tea version that wrote it is the currently running version
- if not, rewrite the file.
Not sure this is worth the complexity & cost
It generates a completion that also suggests file names, which looks kinda messy: Didn't find a way around this, but [there may be a way](5bb54ace57/fish.go (L160-L180) )

Co-authored-by: Norwin Roosen <git@nroo.de >
Co-authored-by: 6543 <6543@obermui.de >
Reviewed-on: https://gitea.com/gitea/tea/pulls/364
Reviewed-by: 6543 <6543@obermui.de >
Reviewed-by: Lunny Xiao <xiaolunwen@gmail.com >
Co-authored-by: Norwin <noerw@noreply.gitea.io >
Co-committed-by: Norwin <noerw@noreply.gitea.io >
2021-05-24 04:42:21 +08:00
Brahim HAMDOUNI
ffdbdb3d02
Check negative limit command parameter ( #358 ) ( #359 )
...
fix #358
Co-authored-by: Brahim Hamdouni <brahim@hamdouni.com >
Reviewed-on: https://gitea.com/gitea/tea/pulls/359
Reviewed-by: techknowlogick <techknowlogick@gitea.io >
Reviewed-by: Norwin <noerw@noreply.gitea.io >
Co-authored-by: Brahim HAMDOUNI <hamdouni@noreply.gitea.io >
Co-committed-by: Brahim HAMDOUNI <hamdouni@noreply.gitea.io >
2021-05-15 22:16:24 +08:00
Norwin
568fde1ce5
Add missing flags to org & labels subcommands ( #357 )
...
The `tea orgs` command is an alias to `tea orgs list`, and as such should have the same flags.
fixes #354
Co-authored-by: Norwin Roosen <git@nroo.de >
Co-authored-by: 6543 <6543@obermui.de >
Reviewed-on: https://gitea.com/gitea/tea/pulls/357
Reviewed-by: Lunny Xiao <xiaolunwen@gmail.com >
Reviewed-by: 6543 <6543@obermui.de >
Co-authored-by: Norwin <noerw@noreply.gitea.io >
Co-committed-by: Norwin <noerw@noreply.gitea.io >
2021-05-13 02:32:20 +08:00
Norwin
46945415c9
Enable release builds for darwin/arm64 ( #360 )
...
Co-authored-by: Norwin Roosen <git@nroo.de >
Reviewed-on: https://gitea.com/gitea/tea/pulls/360
Reviewed-by: techknowlogick <techknowlogick@gitea.io >
Reviewed-by: 6543 <6543@obermui.de >
Co-authored-by: Norwin <noerw@noreply.gitea.io >
Co-committed-by: Norwin <noerw@noreply.gitea.io >
2021-05-11 08:03:39 +08:00
Tamás Gérczei
195bd2199c
contributed container build definition ( #350 )
...
as discussed with @noerw on Discord
Co-authored-by: Tamás Gérczei <tamas@gerczei.eu >
Reviewed-on: https://gitea.com/gitea/tea/pulls/350
Reviewed-by: 6543 <6543@obermui.de >
Reviewed-by: Norwin <noerw@noreply.gitea.io >
Reviewed-by: techknowlogick <techknowlogick@gitea.io >
Co-authored-by: Tamás Gérczei <tgerczei@noreply.gitea.io >
Co-committed-by: Tamás Gérczei <tgerczei@noreply.gitea.io >
2021-03-30 19:13:23 +08:00
Norwin
0bf844018c
Add tea pr merge ( #348 )
...
fixes #343
Co-authored-by: Norwin Roosen <git@nroo.de >
Co-authored-by: 6543 <6543@obermui.de >
Reviewed-on: https://gitea.com/gitea/tea/pulls/348
Reviewed-by: 6543 <6543@obermui.de >
Reviewed-by: Andrew Thornton <art27@cantab.net >
Co-authored-by: Norwin <noerw@noreply.gitea.io >
Co-committed-by: Norwin <noerw@noreply.gitea.io >
2021-03-18 03:56:05 +08:00
Norwin
2319724bb2
Update Changelog ( #346 )
...
smal nit's missing
Co-authored-by: Norwin Roosen <git@nroo.de >
Co-authored-by: 6543 <6543@obermui.de >
Reviewed-on: https://gitea.com/gitea/tea/pulls/346
Reviewed-by: Lunny Xiao <xiaolunwen@gmail.com >
Reviewed-by: techknowlogick <techknowlogick@gitea.io >
Reviewed-by: 6543 <6543@obermui.de >
Co-authored-by: Norwin <noerw@noreply.gitea.io >
Co-committed-by: Norwin <noerw@noreply.gitea.io >
v0.7.0
2021-03-12 20:44:41 +08:00
Norwin
222d0501df
Detect markdown line width, resolve relative URLs ( #332 )
...
~~this is semi-blocked by https://github.com/charmbracelet/glamour/pull/96 , but behaviour isn't really worse than the previous behaviour (most links work, some are still broken)~~
#### testcase for link resolver
```
tea pr 332
tea checkout 332 && make install && tea pr 332
```
- [rel](./332)
- [abs](/gitea/tea/pulls/332)
- [full](https://gitea.com/gitea/tea/pulls/332 )
Co-authored-by: Norwin Roosen <git@nroo.de >
Co-authored-by: 6543 <6543@obermui.de >
Reviewed-on: https://gitea.com/gitea/tea/pulls/332
Reviewed-by: 6543 <6543@obermui.de >
Reviewed-by: Andrew Thornton <art27@cantab.net >
Co-authored-by: Norwin <noerw@noreply.gitea.io >
Co-committed-by: Norwin <noerw@noreply.gitea.io >
2021-03-12 20:28:46 +08:00
6543
cb404b53b5
Changelog v0.7.0 ( #345 )
...
Reviewed-on: https://gitea.com/gitea/tea/pulls/345
Reviewed-by: techknowlogick <techknowlogick@gitea.io >
Reviewed-by: Lunny Xiao <xiaolunwen@gmail.com >
Co-authored-by: 6543 <6543@obermui.de >
Co-committed-by: 6543 <6543@obermui.de >
2021-03-12 08:41:54 +08:00
Norwin
3abc5a5b42
Allow checking out PRs with deleted head branch ( #341 )
...
..by explicitly fetching `refs/pulls/:idx/head` from the base repo.
Sorry, I mixed this with a split-up of `PullCheckout()`. I can try to separate that, if preferred
Co-authored-by: Norwin Roosen <git@nroo.de >
Co-authored-by: 6543 <6543@obermui.de >
Reviewed-on: https://gitea.com/gitea/tea/pulls/341
Reviewed-by: 6543 <6543@obermui.de >
Reviewed-by: Andrew Thornton <art27@cantab.net >
Co-authored-by: Norwin <noerw@noreply.gitea.io >
Co-committed-by: Norwin <noerw@noreply.gitea.io >
2021-03-12 02:16:02 +08:00
Norwin
6f738df4a5
Add more issue / pr creation params ( #331 )
...
adds assignees, labels, deadline, milestone params
- [x] add flags to `tea issue create` (this is BREAKING, `-b` moved to `-d` for consistency with pr create)
- [x] add interactive mode to `tea issue create`
- [x] add flags to `tea pr create`
- [x] add interactive mode to `tea pr create`
fixes #171 , fixes #303
Co-authored-by: Norwin Roosen <git@nroo.de >
Co-authored-by: 6543 <6543@obermui.de >
Reviewed-on: https://gitea.com/gitea/tea/pulls/331
Reviewed-by: 6543 <6543@obermui.de >
Reviewed-by: Lunny Xiao <xiaolunwen@gmail.com >
Co-authored-by: Norwin <noerw@noreply.gitea.io >
Co-committed-by: Norwin <noerw@noreply.gitea.io >
2021-03-08 19:48:03 +08:00
6543
d22b314701
Introduce workaround for missing pull head sha ( #340 )
...
fix #318
test with `tea pr 58`
Co-authored-by: Norwin Roosen <git@nroo.de >
Reviewed-on: https://gitea.com/gitea/tea/pulls/340
Reviewed-by: Norwin <noerw@noreply.gitea.io >
Reviewed-by: Andrew Thornton <art27@cantab.net >
Co-authored-by: 6543 <6543@obermui.de >
Co-committed-by: 6543 <6543@obermui.de >
2021-03-08 03:45:50 +08:00
6543
786c713ff5
[CI] use golang v1.16 ( #339 )
...
Reviewed-on: https://gitea.com/gitea/tea/pulls/339
Reviewed-by: Andrew Thornton <art27@cantab.net >
Reviewed-by: Lunny Xiao <xiaolunwen@gmail.com >
Co-authored-by: 6543 <6543@obermui.de >
Co-committed-by: 6543 <6543@obermui.de >
2021-03-05 20:37:50 +08:00
Norwin
d474883e90
don't push before creating a pull ( #334 )
...
Not sure if this is the best way, but it's the simplest way to fix #333 .
Everything else is overly complex due to a chicken-egg problem:
Knowing which remote / branch to push involves requires prompting the user,
which requires to have a upstream branch pushed to detect default values.
Co-authored-by: Norwin Roosen <git@nroo.de >
Reviewed-on: https://gitea.com/gitea/tea/pulls/334
Reviewed-by: 6543 <6543@obermui.de >
Reviewed-by: Andrew Thornton <art27@cantab.net >
Reviewed-by: khmarbaise <khmarbaise@noreply.gitea.io >
Co-authored-by: Norwin <noerw@noreply.gitea.io >
Co-committed-by: Norwin <noerw@noreply.gitea.io >
2021-03-05 18:27:09 +08:00
6543
0d98cbd657
Update Vendors ( #337 )
...
* update & migrate gitea sdk (Fix Delete Tag Issue)
* upgraded github.com/AlecAivazis/survey v2.2.7 => v2.2.8
* upgraded github.com/adrg/xdg v0.2.3 => v0.3.1
* upgraded github.com/araddon/dateparse
* upgraded github.com/olekukonko/tablewriter v0.0.4 => v0.0.5
* upgraded gopkg.in/yaml.v2 v2.3.0 => v2.4.0
Reviewed-on: https://gitea.com/gitea/tea/pulls/337
Reviewed-by: Norwin <noerw@noreply.gitea.io >
Reviewed-by: khmarbaise <khmarbaise@noreply.gitea.io >
Co-authored-by: 6543 <6543@obermui.de >
Co-committed-by: 6543 <6543@obermui.de >
2021-03-05 18:06:25 +08:00
Norwin
15c4edba1a
Don't exit if we can't find a local repo with a remote matching to a login ( #336 )
...
This enables to run commands that need minimal context (i.e. `tea n --all`) to run anywhere.
fixes #329
Co-authored-by: Norwin Roosen <git@nroo.de >
Reviewed-on: https://gitea.com/gitea/tea/pulls/336
Reviewed-by: Lunny Xiao <xiaolunwen@gmail.com >
Reviewed-by: 6543 <6543@obermui.de >
Co-authored-by: Norwin <noerw@noreply.gitea.io >
Co-committed-by: Norwin <noerw@noreply.gitea.io >
2021-03-05 16:56:15 +08:00
Norwin
e96cfdbbe7
tea pr checkout: dont create local branches (#314 )
...
This avoids creation of local branches, to avoid cluttering the local repo:
- if the commit already exists on the tip of a local branch, check that one out
- otherwise check out the remote tracking branch (`refs/remotes/<remote>/<head>`), and suggest what to do if you want to make changes.
I'm not certain this behaviour is actually better, I suggest leaving this open for a while for people to try out the new behaviour:
```
tea pr checkout 314
make install
```
fixes #293
Co-authored-by: Norwin Roosen <git@nroo.de >
Co-authored-by: 6543 <6543@obermui.de >
Reviewed-on: https://gitea.com/gitea/tea/pulls/314
Reviewed-by: 6543 <6543@obermui.de >
Reviewed-by: Lunny Xiao <xiaolunwen@gmail.com >
Co-authored-by: Norwin <noerw@noreply.gitea.io >
Co-committed-by: Norwin <noerw@noreply.gitea.io >
2021-03-02 21:50:11 +08:00
Norwin
3c1efd33e2
InitCommand() robustness ( #327 )
...
fixes #320
Co-authored-by: Norwin Roosen <git@nroo.de >
Reviewed-on: https://gitea.com/gitea/tea/pulls/327
Reviewed-by: Andrew Thornton <art27@cantab.net >
Reviewed-by: 6543 <6543@obermui.de >
Co-authored-by: Norwin <noerw@noreply.gitea.io >
Co-committed-by: Norwin <noerw@noreply.gitea.io >
2021-03-01 06:29:26 +08:00
Norwin
9c8321f2e0
tea comment: handle piped stdin ( #322 )
...
fixes #321
Co-authored-by: Norwin Roosen <git@nroo.de >
Reviewed-on: https://gitea.com/gitea/tea/pulls/322
Reviewed-by: 6543 <6543@obermui.de >
Reviewed-by: Andrew Thornton <art27@cantab.net >
Co-authored-by: Norwin <noerw@noreply.gitea.io >
Co-committed-by: Norwin <noerw@noreply.gitea.io >
2021-03-01 01:47:36 +08:00
Norwin
b5c670ebf8
Improve tea time ( #319 )
...
better docs
add --mine flag
hm, is there a better name? 🤔
do time filtering serverside
make printed fields dynamic
add --fields to tea times ls
code review
Co-authored-by: Norwin Roosen <git@nroo.de >
Reviewed-on: https://gitea.com/gitea/tea/pulls/319
Reviewed-by: 6543 <6543@obermui.de >
Reviewed-by: Lunny Xiao <xiaolunwen@gmail.com >
Co-Authored-By: Norwin <noerw@noreply.gitea.io >
Co-Committed-By: Norwin <noerw@noreply.gitea.io >
2020-12-23 12:58:36 +08:00
Norwin
95ef061711
Update dependencies ( #316 )
...
update xdg
update survey
update go-sdk
Co-authored-by: Norwin Roosen <git@nroo.de >
Reviewed-on: https://gitea.com/gitea/tea/pulls/316
Reviewed-by: Lunny Xiao <xiaolunwen@gmail.com >
Reviewed-by: 6543 <6543@obermui.de >
Co-Authored-By: Norwin <noerw@noreply.gitea.io >
Co-Committed-By: Norwin <noerw@noreply.gitea.io >
2020-12-22 01:11:08 +08:00
Norwin
32b7b771cc
Add tea comment and show comments of issues/pulls ( #313 )
...
show comments of PR
TODO: there needs to be a way to force running non-interactively
add `tea comment` to post a comment
add --comments flag, prompt only if necessary
don't prompt if --comments is provided, or output is piped
show comments for issues, add --comments flag
tea comment: print resulting comment
Merge branch 'master' into issue-172-comments
remove debug print statement
unrelated, but better than opening another PR for this ;)
Merge remote-tracking branch 'upstream/master' into issue-172-comments
ret err
fix lint
Co-authored-by: Norwin Roosen <git@nroo.de >
Co-authored-by: 6543 <6543@obermui.de >
Reviewed-on: https://gitea.com/gitea/tea/pulls/313
Reviewed-by: Lunny Xiao <xiaolunwen@gmail.com >
Reviewed-by: 6543 <6543@obermui.de >
Co-Authored-By: Norwin <noerw@noreply.gitea.io >
Co-Committed-By: Norwin <noerw@noreply.gitea.io >
2020-12-22 00:07:35 +08:00
Norwin
9efee7bf99
Add tea issues --fields, allow printing labels ( #312 )
...
generalize list printing with dynamic fields
refactor print.IssuesList to use tableFromItems()
preparatory refactor
print.IssuesList: allow printing labels
move formatters to formatters.go
expose more printable fields on issue
add generic flags.FieldsFlag
add fields flag to tea issues, tea ms issues
validate provided fields
add strict username, or formatted user fields
change default fields
tea issues -> replace updated with labels
tea ms issues -> replace author with labels, reorder
Validate provided fields
Co-authored-by: Norwin Roosen <git@nroo.de >
Co-authored-by: 6543 <6543@obermui.de >
Reviewed-on: https://gitea.com/gitea/tea/pulls/312
Reviewed-by: Lunny Xiao <xiaolunwen@gmail.com >
Reviewed-by: 6543 <6543@obermui.de >
Co-Authored-By: Norwin <noerw@noreply.gitea.io >
Co-Committed-By: Norwin <noerw@noreply.gitea.io >
2020-12-21 23:41:07 +08:00
Norwin
8bb5c15745
Add commands for reviews ( #315 )
...
add interactive `tea pr review`
it's amazingly simple
vendor gitea.com/noerw/unidiff-comments
add `tea pr lgtm|reject` shorthands
vendor slimmed down diff parser
review diff: default to true
if users want a shortcut, they can use lgtm or reject subcmds
`tea pr approve`: accept optional comment
Co-authored-by: Norwin Roosen <git@nroo.de >
Reviewed-on: https://gitea.com/gitea/tea/pulls/315
Reviewed-by: Lunny Xiao <xiaolunwen@gmail.com >
Reviewed-by: 6543 <6543@obermui.de >
Co-Authored-By: Norwin <noerw@noreply.gitea.io >
Co-Committed-By: Norwin <noerw@noreply.gitea.io >
2020-12-21 23:22:22 +08:00
Norwin
43a58bdba1
Proper help text & new README structure ( #311 )
...
add cli.AppHelpTemplate for customization
customize tea help view
tea --version : improve parseability
Rework README to include tea help output
It's an antipattern to have different help texts aimed at the same
users. So now that we have a good cli help text, lets use it here.
This eases maintenance, and at the same time gives an honest impression
on what we have to offer, while also encouraging to improve the internal
help text in the future.
I feel a bit sad for the GIF, but it was becoming outdated anyway..
group commands by category
add new demo gif
shows the (probably) most useful workflow
readme improvement
Merge branch 'master' into improve-app-help
code review
Merge branch 'master' into improve-app-help
restructure installation section
Co-authored-by: Norwin Roosen <git@nroo.de >
Reviewed-on: https://gitea.com/gitea/tea/pulls/311
Reviewed-by: Lunny Xiao <xiaolunwen@gmail.com >
Reviewed-by: khmarbaise <khmarbaise@noreply.gitea.io >
Co-Authored-By: Norwin <noerw@noreply.gitea.io >
Co-Committed-By: Norwin <noerw@noreply.gitea.io >
2020-12-21 21:37:20 +08:00
Martin Reboredo
43e9943757
Add interactive mode for tea milestone create ( #310 )
...
Implement interactive milestone creation
Return fmt.Errorf when title is empty
Incorporate deadline functionality
Use dateparse and cleanup CreateMilestone task
Signed-off-by: Martin Reboredo <yakoyoku@gmail.com >
Co-authored-by: Martin Reboredo <yakoyoku@gmail.com >
Reviewed-on: https://gitea.com/gitea/tea/pulls/310
Reviewed-by: Norwin <noerw@noreply.gitea.io >
Reviewed-by: 6543 <6543@obermui.de >
Co-Authored-By: Martin Reboredo <yakoyakoyokuyoku@noreply.gitea.io >
Co-Committed-By: Martin Reboredo <yakoyakoyokuyoku@noreply.gitea.io >
2020-12-18 02:50:07 +08:00
Norwin
8b588f5313
make PR workflow helpers more robust ( #300 )
...
improve handling of remote deleted branches
split git.TeaDeleteBranch
only delete remote branch if we have permission
add missing err check
Co-authored-by: Norwin Roosen <git@nroo.de >
Reviewed-on: https://gitea.com/gitea/tea/pulls/300
Reviewed-by: 6543 <6543@obermui.de >
Reviewed-by: Lunny Xiao <xiaolunwen@gmail.com >
Co-Authored-By: Norwin <noerw@noreply.gitea.io >
Co-Committed-By: Norwin <noerw@noreply.gitea.io >
2020-12-17 22:00:16 +08:00
Martin Reboredo
a2e8b47c57
Implement PR closing and reopening ( #304 )
...
Implement pull request closing/reopening
Signed-off-by: Martin Reboredo <yakoyoku@gmail.com >
Correct year and `pull` description
Apply changes from #291
Return fmt.Errorf instead of log.Fatal if no pull index was supplied
Co-authored-by: Martin Reboredo <yakoyoku@gmail.com >
Reviewed-on: https://gitea.com/gitea/tea/pulls/304
Reviewed-by: Norwin <noerw@noreply.gitea.io >
Reviewed-by: appleboy <appleboy.tw@gmail.com >
Co-Authored-By: Martin Reboredo <yakoyakoyokuyoku@noreply.gitea.io >
Co-Committed-By: Martin Reboredo <yakoyakoyokuyoku@noreply.gitea.io >
2020-12-17 06:47:12 +08:00
Norwin
83b73ce78e
Show PR CI status ( #306 )
...
fix layout of pr reviews
show PR CI status
put conflict info in status list
remove line
show merged state
deduplicate reviews by user
Co-authored-by: Norwin Roosen <git@nroo.de >
Reviewed-on: https://gitea.com/gitea/tea/pulls/306
Reviewed-by: Lunny Xiao <xiaolunwen@gmail.com >
Reviewed-by: 6543 <6543@obermui.de >
Co-Authored-By: Norwin <noerw@noreply.gitea.io >
Co-Committed-By: Norwin <noerw@noreply.gitea.io >
2020-12-17 01:16:50 +08:00
Norwin
782a6318f3
Add more command shorthands ( #307 )
...
add more command aliases
breaking: s/notif/n
Co-authored-by: Norwin Roosen <git@nroo.de >
Reviewed-on: https://gitea.com/gitea/tea/pulls/307
Reviewed-by: 6543 <6543@obermui.de >
Reviewed-by: Lunny Xiao <xiaolunwen@gmail.com >
Co-Authored-By: Norwin <noerw@noreply.gitea.io >
Co-Committed-By: Norwin <noerw@noreply.gitea.io >
2020-12-17 00:47:40 +08:00
Norwin
a948fd7e10
Refactor error handling ( #308 )
...
use fmt instead of log
log.Fatal -> return err
set non-zero exit code on error
print to default err log
cleanup
fix vet
Co-authored-by: Norwin Roosen <git@nroo.de >
Co-authored-by: 6543 <6543@obermui.de >
Reviewed-on: https://gitea.com/gitea/tea/pulls/308
Reviewed-by: Lunny Xiao <xiaolunwen@gmail.com >
Reviewed-by: 6543 <6543@obermui.de >
Co-Authored-By: Norwin <noerw@noreply.gitea.io >
Co-Committed-By: Norwin <noerw@noreply.gitea.io >
2020-12-17 00:18:10 +08:00
Norwin
287df8a715
Add command to install shell completion ( #309 )
...
add autocompletion files to contrib/
curl -o contrib/autocomplete.zsh https://raw.githubusercontent.com/urfave/cli/master/autocomplete/zsh_autocomplete
curl -o contrib/autocomplete.sh https://raw.githubusercontent.com/urfave/cli/master/autocomplete/bash_autocomplete
add powershell
add `tea meta autocomplete`
closes #86
update docs
Co-authored-by: Norwin Roosen <git@nroo.de >
Reviewed-on: https://gitea.com/gitea/tea/pulls/309
Reviewed-by: Lunny Xiao <xiaolunwen@gmail.com >
Reviewed-by: 6543 <6543@obermui.de >
Co-Authored-By: Norwin <noerw@noreply.gitea.io >
Co-Committed-By: Norwin <noerw@noreply.gitea.io >
2020-12-17 00:01:59 +08:00