- 
                Notifications
    You must be signed in to change notification settings 
- Fork 4.1k
[PS] Migrate AutoScale in Monitor module to autorest v4 #28081
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
base: main
Are you sure you want to change the base?
Conversation
| Thanks for your contribution! The pull request validation has started. Please revisit this comment for updated status. | 
| To the author of the pull request, | 
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.
Pull Request Overview
This pull request migrates the AutoScale module in Monitor to autorest v4, bringing breaking changes to maintain consistency with the new autorest version. The migration involves updating API generation patterns, parameter types, and documentation formats.
- Migration from autorest v3 to v4 with breaking changes to property types and parameter signatures
- Updated parameter types from specific enum types to string types for better flexibility
- Removed deprecated administrator notification parameters due to Azure classic administrator retirement
Reviewed Changes
Copilot reviewed 36 out of 36 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description | 
|---|---|
| tools/StaticAnalysis/Exceptions/Az.Monitor/BreakingChangeIssues.csv | Adds breaking change exceptions for type changes from enums to strings and arrays to lists | 
| src/Monitor/Monitor/help/*.md | Updates help documentation to reflect new parameter types and remove API version references | 
| src/Monitor/Monitor/ChangeLog.md | Documents retirement of classic administrator notification parameters | 
| src/Monitor/Monitor.sln | Reorders project references in solution file | 
| src/Monitor/Autoscale.Autorest/test/* | Updates test files with new environment configuration and removes deprecated parameters | 
| src/Monitor/Autoscale.Autorest/docs/*.md | Updates documentation to reflect new parameter types and URLs | 
| src/Monitor/Autoscale.Autorest/custom/*.ps1 | Updates custom cmdlet implementations with new type signatures | 
| src/Monitor/Autoscale.Autorest/README.md | Simplifies autorest configuration and removes breaking change directives | 
Description
Mandatory Checklist
Please choose the target release of Azure PowerShell. (⚠️ Target release is a different concept from API readiness. Please click below links for details.)
Check this box to confirm: I have read the Submitting Changes section of
CONTRIBUTING.mdand reviewed the following information:ChangeLog.mdfile(s) appropriatelysrc/{{SERVICE}}/{{SERVICE}}/ChangeLog.md.## Upcoming Releaseheader in the past tense.ChangeLog.mdif no new release is required, such as fixing test case only.