Skip to content

Conversation

@arrownj
Copy link
Contributor

@arrownj arrownj commented Sep 29, 2020

Description
Previously when param persist is already on/off, it will raise an CLIError, which does not make sense. Update it to a warning message.

Testing Guide

History Notes

[Component Name 1] BREAKING CHANGE: az command a: Make some customer-facing breaking change.
[Component Name 2] az command b: Add some customer-facing feature.


This checklist is used to make sure that common guidelines for a pull request are followed.

@arrownj arrownj self-assigned this Sep 29, 2020
@yonzhan
Copy link
Collaborator

yonzhan commented Sep 29, 2020

Config

@yonzhan yonzhan requested review from jiasli and jsntcy September 29, 2020 10:36
@arrownj
Copy link
Contributor Author

arrownj commented Sep 30, 2020

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 2 pipeline(s).

@arrownj arrownj merged commit 6b544ff into Azure:dev Sep 30, 2020
Juliehzl pushed a commit to Juliehzl/azure-cli that referenced this pull request Sep 30, 2020
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.

3 participants