Skip to content

Support --only-show-errors #179

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.

Sign up for GitHub

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

Jump to bottom
Merged
jiasli merged 3 commits into from
Mar 16, 2020
Merged

Support --only-show-errors #179

jiasli merged 3 commits into from
Mar 16, 2020

Conversation

@jiasli
Copy link
Member

@jiasli jiasli commented Mar 10, 2020 •
edited
Loading

Support --only-show-errors to disable warnings, including preview, deprecation warnings during command invocation and logger.warning().

There are 3 ways to achieve this effect:

  • Use --only-show-errors as command parameter
  • Use config 
    [core]
only_show_errors=True
  • Use env var CLI_CORE_ONLY_SHOW_ERRORS=True

It appears in every command's --help message:

Global Arguments
    ...
    --only-show-errors : Only show errors, suppressing warnings.
    ...

@jiasli jiasli self-assigned this Mar 10, 2020
@yungezz yungezz requested a review from arrownj March 11, 2020 01:01
@jiasli jiasli mentioned this pull request Mar 11, 2020
Merged
@jiasli jiasli changed the base branch from master to dev March 16, 2020 11:26
jiasli added 2 commits March 16, 2020 19:53
@jiasli
Merge branch 'dev' into only-show-errors
59d6fd8 
# Conflicts:
#	knack/cli.py
#	knack/invocation.py
#	tests/test_preview.py
@jiasli
Refactor
f1fb5a1
@jiasli jiasli merged commit 6d6bbea into microsoft:dev Mar 16, 2020
@jiasli jiasli mentioned this pull request Mar 16, 2020
Merged
jiasli added a commit that referenced this pull request Mar 20, 2020
@jiasli
Merge dev to master (#181)
5f0bcc2 
* Allow disabling color (#171)

* Support --only-show-errors (#179)

* Add experimental tag (#180)
@jiasli jiasli deleted the only-show-errors branch March 24, 2020 13:08
@jiasli jiasli mentioned this pull request Oct 16, 2021
Open
@jiasli jiasli mentioned this pull request Nov 12, 2021
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Juliehzl Juliehzl Awaiting requested review from Juliehzl

@mmyyrroonn mmyyrroonn Awaiting requested review from mmyyrroonn

@haroldrandom haroldrandom Awaiting requested review from haroldrandom

@yonzhan yonzhan Awaiting requested review from yonzhan

@yungezz yungezz Awaiting requested review from yungezz

@arrownj arrownj Awaiting requested review from arrownj

Assignees

@jiasli jiasli

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@jiasli