8000 Error for mutually exclusive json and watch flags by andyfeller · Pull Request #10016 · cli/cli · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Error for mutually exclusive json and watch flags #10016

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged

Conversation

andyfeller
Copy link
Member

Closes #10013

This commit enhances gh pr checks to exit with an error when mutually exclusive --json and --watch flags are used. Users are encouraged to either 1) retrieve information about the checks via --json or 2) watch and wait on PR checks that are being updated.

cc: @IvanUkhov

Closes #10013

This commit enhances `gh pr checks` to exit with an error when mutually exclusive `--json` and `--watch` flags are used.  Users are encouraged to either 1) retrieve information about the checks via `--json` or 2) watch and wait on PR checks that are being updated.
@andyfeller andyfeller requested a review from a team as a code owner December 5, 2024 15:26
@andyfeller andyfeller requested a review from jtmcg December 5, 2024 15:26
@cliAutomation cliAutomation added the external pull request originating outside of the CLI core team label Dec 5, 2024
@andyfeller andyfeller enabled auto-merge December 5, 2024 15:28
Copy link
Contributor
@jtmcg jtmcg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Simple enough. :shipit:

@andyfeller andyfeller merged commit d60e5ff into trunk Dec 5, 2024
19 checks passed
@andyfeller andyfeller deleted the andyfeller/cli-cli-10013-pr-checks-json-watch-mutex branch December 5, 2024 17:22
tmeijn pushed a commit to tmeijn/dotfiles that referenced this pull request Dec 6, 2024
This MR contains the following updates:

| Package | Update | Change |
|---|---|---|
| [cli/cli](https://github.com/cli/cli) | patch | `v2.63.1` -> `v2.63.2` |

MR created with the help of [el-capitano/tools/renovate-bot](https://gitlab.com/el-capitano/tools/renovate-bot).

**Proposed changes to behavior should be submitted there as MRs.**

---

### Release Notes

<details>
<summary>cli/cli (cli/cli)</summary>

### [`v2.63.2`](https://github.com/cli/cli/releases/tag/v2.63.2): GitHub CLI 2.63.2

[Compare Source](cli/cli@v2.63.1...v2.63.2)

#### What's Changed

-   Use consistent slice ordering in run download tests by [@&#8203;williammartin](https://github.com/williammartin) in cli/cli#10006
-   Fix bug when fetching bundles from OCI registry by [@&#8203;malancas](https://github.com/malancas) in cli/cli#10019
-   Use safepaths for run download by [@&#8203;williammartin](https://github.com/williammartin) in cli/cli#10009
-   Error for mutually exclusive json and watch flags by [@&#8203;andyfeller](https://github.com/andyfeller) in cli/cli#10016

**Full Changelog**: cli/cli@v2.63.1...v2.63.2

</details>

---

### Configuration

📅 **Schedule**: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).

🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied.

♻ **Rebasing**: Whenever MR becomes conflicted, or you tick the rebase/retry checkbox.

🔕 **Ignore**: Close this MR and you won't be reminded about this update again.

---

 - [ ] <!-- rebase-check -->If you want to rebase/retry this MR, check this box

---

This MR has been generated by [Renovate Bot](https://github.com/renovatebot/renovate).
<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNy40NDAuNyIsInVwZGF0ZWRJblZlciI6IjM3LjQ0MC43IiwidGFyZ2V0QnJhbmNoIjoibWFpbiIsImxhYmVscyI6WyJSZW5vdmF0ZSBCb3QiXX0=-->
@soelin185

This comment was marked as spam.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
external pull request originating outside of the CLI core team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Respect --watch when given --json in gh pr checks
4 participants
0