-
Notifications
You must be signed in to change notification settings - Fork 4.1k
migrate workloads-monitors to autorest v4 #27346
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
migrate workloads-monitors to autorest v4 #27346
Conversation
| Thanks for your contribution! The pull request validation has started. Please revisit this comment for updated status. |
54335d0 to
cf741b7
Compare
|
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 workloads-monitors module from autorest v3 to autorest v4, introducing various new features and API changes for improved functionality and consistency.
- Updates code generation configuration to use newer autorest version with enhanced feature support
- Introduces new managed identity parameter patterns with simplified configuration options
- Adds new parameter sets and JSON input methods for various cmdlets
Reviewed Changes
Copilot reviewed 60 out of 60 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| tools/StaticAnalysis/Exceptions/Az.Workloads/BreakingChangeIssues.csv | Documents breaking changes from API property type modifications |
| src/Workloads/Workloads/help/*.md | Updates cmdlet help documentation with new parameter patterns and type changes |
| src/Workloads/Workloads/ChangeLog.md | Documents new features and parameter changes |
| src/Workloads/Workloads/Az.Workloads.psd1 | Updates module metadata and dependencies |
| src/Workloads/Monitors.Autorest/* | Regenerated autorest code with v4 patterns and configurations |
| - Additional information about change #1 | ||
| --> | ||
| ## Upcoming Release | ||
| * Introduced various new features by upgrading code generator. Please see details [here](https://github.com/Azure/azure-powershell/blob/main/documentation/Autorest-powershell-v4-new-features.md). |
Copilot
AI
Aug 29, 2025
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.
[nitpick] The changelog entry references external documentation but doesn't provide a concise summary of what new features were actually introduced in this module. Consider adding a brief overview of the key new features alongside the external link.
| * Introduced various new features by upgrading code generator. Please see details [here](https://github.com/Azure/azure-powershell/blob/main/documentation/Autorest-powershell-v4-new-features.md). | |
| * Upgraded code generator to Autorest PowerShell v4, introducing several new features: | |
| - Improved error handling and diagnostics for cmdlets. | |
| - Enhanced support for complex parameter types and validation. | |
| - Streamlined output formatting for better readability. | |
| - Increased performance and reduced module size. | |
| - Added support for new authentication scenarios. | |
| - For a full list of new features, please see details [here](https://github.com/Azure/azure-powershell/blob/main/documentation/Autorest-powershell-v4-new-features.md). |
|
/azp run |
|
Azure Pipelines will not run the associated pipelines, because the pull request was updated after the run command was issued. Review the pull request again and issue a new run command. |
126c7de to
863d218
Compare
Description
Please also review & merge record test #28740
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.