-
Notifications
You must be signed in to change notification settings - Fork 3.1k
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
Advisor Module Updates #5334
Advisor Module Updates #5334
Conversation
View a preview at https://prompt.ws/r/Azure/azure-cli/5334 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please set up a meeting with us. A quick review of the advisor module gives me concern that the module is not ready for GA (which is separate from the service GA).
@@ -3,6 +3,10 @@ | |||
Release History | |||
=============== | |||
|
|||
1.0.0 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If this is a GA release, it should be 2.0.0
- Removed 'recommendation generate' command and added a '--refresh' option to 'recommendation list' instead. - Renamed 'configuration get' to 'configuration list'. - Renamed 'configuration set' to 'configuration update'. - Test recording updates.
@@ -3,6 +3,10 @@ | |||
Release History | |||
=============== | |||
|
|||
1.0.0 | |||
++++++ | |||
* GA release. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Here you will want to call out whatever breaking changes were made. For example
* BC: `advisor configuration set` renamed to `advisor configuration update`
and so on.
View a preview at https://prompt.ws/r/Azure/azure-cli/5334 |
The needs to be rebased and missed the code-complete for Sprint 31. Moving to Sprint 32. |
Get latest changes from upstream
Merge with upstream
- Removed 'recommendation generate' command and added a '--refresh' option to 'recommendation list' instead. - Renamed 'configuration get' to 'configuration list'. - Renamed 'configuration set' to 'configuration update'. - Test recording updates.
This checklist is used to make sure that common guidelines for a pull request are followed.
General Guidelines
Command Guidelines
(see Authoring Command Modules)