Commit graph

11279 commits

Author SHA1 Message Date
AliabbasMerchant
5fc722fbb8 Query Params in Web mode of Issue/PR creation 2020-05-23 14:40:49 +05:30
Corey Johnson
9c075abe93 Fix tests 2020-05-22 17:45:04 -07:00
Corey Johnson
9a290d7f43 Change interactive merge language 2020-05-22 15:30:05 -07:00
Corey Johnson
9c48e2da14 Delete the branch 2020-05-22 15:25:25 -07:00
Corey Johnson
9c88cd67ee Whoops 2020-05-22 15:21:51 -07:00
Corey Johnson
9113092f5c Delete via api 2020-05-22 15:19:39 -07:00
Corey Johnson
ff97fc138d Still delete remote branch if repo flag is used 2020-05-22 15:12:48 -07:00
Corey Johnson
e643765027 Merge remote-tracking branch 'origin/merge-interactive-merge' into remote-delete 2020-05-22 15:09:29 -07:00
Corey Johnson
858cd705ab Update name 2020-05-22 15:07:39 -07:00
Corey Johnson
368aacbff6 Fully qualify that ref 2020-05-22 14:55:32 -07:00
Corey Johnson
d7c933bc40 branchCmd 2020-05-22 14:52:29 -07:00
Corey Johnson
28d8a9e781 Use big D 2020-05-22 14:40:38 -07:00
Corey Johnson
b2c1b12bee Don't delete branch if the repo flag is used 2020-05-22 14:39:53 -07:00
AliabbasMerchant
8bfc935860 Fix Incorrect Template Type 2020-05-23 01:05:27 +05:30
AliabbasMerchant
4fd051b21d Issue/PR template selection flow mirrors that of GitHub 2020-05-23 00:25:13 +05:30
AliabbasMerchant
864d74d0b1 Categorize Templates as Legacy and NonLegacy 2020-05-23 00:03:38 +05:30
Nate Smith
fd49cf69e2
Merge pull request #988 from cli/fix-linter-from-prs
Enable linting for pull requests
2020-05-22 13:18:00 -05:00
Mislav Marohnić
54e68ae5b2 Enable linting for pull requests 2020-05-22 19:40:00 +02:00
Corey Johnson
41e67aa3e7 Fix typo 2020-05-22 10:04:02 -07:00
Mislav Marohnić
108f7bd66c Dirty workaround to display helpful error message on mistyped subcommands
When executing `gh pr re` (note the incomplete command name), Cobra
would just display the help text for `gh pr` on standard output, exit
with status 0, and not print any message that you have mistyped the
"re" subcommand. Each part of this behavior is wrong.

This workaround makes sure that the helpful error message is printed on
stderr:

    $ gh pr re
    unknown command "re" for "gh pr"

    Did you mean this?
            reopen
            ready
            review

However, the exit status is still 0, whereas it should be non-zero.
Since `HelpFunc` does not return an error argument, we cannot trigger an
error status from this workaround.
2020-05-22 19:02:34 +02:00
Mislav Marohnić
a33c00d389 Merge remote-tracking branch 'origin/master' into cobra1.0 2020-05-22 18:21:35 +02:00
Mislav Marohnić
1f3725c94d Use @me to avoid having to look up current user in pr status 2020-05-22 15:58:57 +02:00
Mislav Marohnić
331deb852b
Merge pull request #975 from cli/pr-reviewers-crash
Avoid crash in `pr view` with reviewers
2020-05-22 15:46:37 +02:00
Corey Johnson
a8fa9a2dfc Merge remote-tracking branch 'origin/merge-interactive-merge' into remote-delete 2020-05-20 18:21:45 -07:00
Corey Johnson
51162aa80c Merge remote-tracking branch 'origin/master' into merge-interactive-merge 2020-05-20 18:20:12 -07:00
Corey Johnson
479707d349 Merge remote-tracking branch 'origin/master' into remote-delete 2020-05-20 18:19:55 -07:00
vilmibm
5f28751555 just use a map + stdin test 2020-05-20 17:44:09 -05:00
vilmibm
f6137fa6ae handle json error in test 2020-05-20 17:16:59 -05:00
vilmibm
509be34af6 Merge remote-tracking branch 'origin/master' into wingkwong/master 2020-05-20 17:08:31 -05:00
vilmibm
f644c11edb tweak test 2020-05-20 17:03:56 -05:00
vilmibm
7453449d9a improve docs 2020-05-20 17:01:28 -05:00
Corey Johnson
ae70a814bd Update tests 2020-05-20 14:53:52 -07:00
vilmibm
93dc62f1b3 handle piped STDIN 2020-05-20 16:51:50 -05:00
Corey Johnson
80bdd1e071 print error 2020-05-20 14:36:09 -07:00
vilmibm
466edf8d0e take files as args. handle - 2020-05-20 16:27:28 -05:00
Corey Johnson
904c541d2d Only delete local branch if it exists 2020-05-20 14:19:44 -07:00
Corey Johnson
a303dabc5d Add remote deletion 2020-05-20 13:47:29 -07:00
Nate Smith
4e73c83fd1
Merge pull request #978 from cli/fix-linter
Fix lint errors
2020-05-20 13:59:07 -05:00
Corey Johnson
e5b78d3342 Fix lint errors 2020-05-20 11:21:05 -07:00
Corey Johnson
46a1e3cd53 Remove convertRepoInterfaceToRepository 2020-05-20 11:19:31 -07:00
Corey Johnson
5d99c56456 combine conditionals 2020-05-20 09:21:59 -07:00
Corey Johnson
b4783dd192 don't always switch to the default branch 2020-05-20 09:00:39 -07:00
Mislav Marohnić
d9e39226ca Clean up unused struct field
https://github.com/cli/cli/pull/976/checks?check_run_id=693382299
2020-05-20 17:41:33 +02:00
Mislav Marohnić
db9014fd7f Respect auth token from GITHUB_TOKEN environment variable
If GITHUB_TOKEN is non-blank, it overrides authentication info found in
the config file. The config file is, in fact, never consulted.
2020-05-20 17:35:10 +02:00
Corey Johnson
76368f92ee Return error 2020-05-20 08:32:08 -07:00
Corey Johnson
c3c3058b96 Merge remote-tracking branch 'origin/master' into merge-interactive-merge 2020-05-20 08:30:17 -07:00
Corey Johnson
1eedfe18bb Update default text 2020-05-20 08:26:34 -07:00
Mislav Marohnić
767521c055 Stop providing AuthLogin from context
The login name of the authenticated user will be readily available only
if authentication info comes from the config file. With other upcoming
authentication modes (for example, the GITHUB_TOKEN environment
variable), the token is the only piece of information we got, so we
would need to additionally query for the login name.

Since `issue status` and `pr status` are the only commands that need the
name of the authenticated user right now, have those commands explicitly
query for the login name. This results in an additional API query, but
simplifies Context implementation and future authentication approaches.
2020-05-20 17:09:13 +02:00
Nate Smith
e9d6e13339
Merge pull request #915 from mtfurlan/docs/manpage
Generate manpages from cobra
2020-05-20 09:58:50 -05:00
Mislav Marohnić
ea3a55c3d6 Ensure that cobra command tests don't write to system stdout/stderr 2020-05-20 16:28:35 +02:00