Skip to content

Conversation

@qwordy
Copy link
Member

@qwordy qwordy commented Apr 16, 2020

Description

#11961
--version
The version of the extension. If you want to use the exact version number specified, please specify --no-auto-upgrade.

Auto upgrade is on by default. So remind users.

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.

@qwordy qwordy requested review from arrownj and yungezz April 16, 2020 10:03
c.argument('settings', type=validate_file_or_dict, help='Extension settings in JSON format. A JSON file path is also accepted.')
c.argument('protected_settings', type=validate_file_or_dict, help='Protected settings in JSON format for sensitive information like credentials. A JSON file path is also accepted.')
c.argument('version', help='The version of the extension')
c.argument('version', help='The version of the extension. If you want to use the exact version number specified, please specify --no-auto-upgrade.')
Copy link
Member

Choose a reason for hiding this comment

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

To pin extension version to fix version, pls specify --no-auto-upgrade.

Suggested change
c.argument('version', help='The version of the extension. If you want to use the exact version number specified, please specify --no-auto-upgrade.')
c.argument('version', help='The version of the extension. To pin extension version to specific version number, please specify --no-auto-upgrade.')

@yonzhan yonzhan added this to the S168 milestone Apr 16, 2020
@yonzhan
Copy link
Collaborator

yonzhan commented Apr 16, 2020

add to S168

@qwordy qwordy merged commit dc52ce2 into Azure:dev Apr 17, 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.

4 participants