Skip to content
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

Add only_show_errors and no_color #1924

Merged
merged 1 commit into from
Apr 27, 2020

Conversation

jiasli
Copy link
Member

@jiasli jiasli commented Apr 10, 2020

Add only_show_errors and no_color to doc new features introduced by Azure/azure-cli#12604.

We may need to update the help message of --only-show-errors accordingly.

> az version -h

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

@opbld32
Copy link

opbld32 commented Apr 10, 2020

Docs Build status updates of commit ce8fd11:

🕙 Full build: the average full build time is 7 min(s) 22 sec(s), this is based on the last 544 build(s) for this repository.

@@ -88,6 +87,8 @@ When you provide a default value, that argument is no longer required by any com
| __core__ | output | string | The default output format. Can be one of `json`, `jsonc`, `tsv`, or `table`. |
| | disable\_confirm\_prompt | boolean | Turn confirmation prompts on/off. |
| | collect\_telemetry | boolean | Allow Microsoft to collect anonymous data on the usage of the CLI. For privacy information, see the [Azure CLI Terms of Use](https://aka.ms/AzureCliLegal). |
| | only\_show\_errors | boolean | Only show errors during command invocation. In other words, only errors will be written to `stderr`. It suppresses warnings from preview, deprecated and experimental commands. It is also available for individual commands with the `--only-show-errors` parameter. |
| | no\_color | boolean | Disable color. Originally colored messages will be prefixed with `DEBUG`, `INFO`, `WARNING` and `ERROR`. This bypasses the issue of a third-party library where the terminal's color cannot revert back after a `stdout` redirection. |
Copy link
Member Author

Choose a reason for hiding this comment

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

I am not sure whether we need to put the details of the context here, including the original issue.

Suggested change
| | no\_color | boolean | Disable color. Originally colored messages will be prefixed with `DEBUG`, `INFO`, `WARNING` and `ERROR`. This bypasses the issue of a third-party library where the terminal's color cannot revert back after a `stdout` redirection. |
| | no\_color | boolean | Disable color. Originally colored messages will be prefixed with `DEBUG`, `INFO`, `WARNING` and `ERROR`. This bypasses the issue of the third-party library colorama ([#200](https://github.com/tartley/colorama/issues/200)) where the terminal's color cannot revert back after a `stdout` redirection. |

@jiasli jiasli requested review from achandmsft and yonzhan April 10, 2020 02:43
@opbld30
Copy link

opbld30 commented Apr 10, 2020

Docs Build status updates of commit ce8fd11:

✅ Validation status: passed

File Status Preview URL Details
docs-ref-conceptual/azure-cli-configuration.md ✅Succeeded View (azure-cli-latest)
View (azure-cli-2017-03-09-profile)
View (azure-cli-2018-03-01-hybrid)

For more details, please refer to the build report.

Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

@dbradish-microsoft dbradish-microsoft marked this pull request as ready for review April 27, 2020 15:41
@dbradish-microsoft dbradish-microsoft merged commit 870bab2 into MicrosoftDocs:master Apr 27, 2020
@jiasli jiasli deleted the configuration branch April 28, 2020 05:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants