-
Notifications
You must be signed in to change notification settings - Fork 4.1k
[Az.CloudService] The code base is going to be refactored, the following cmdlet adds a BreakingChange announcement #28470
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
Conversation
…ing cmdlet adds a BreakingChange announcement
| Thanks for your contribution! The pull request validation has started. Please revisit this comment for updated status. |
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 PR adds breaking change announcements for the Az.CloudService module in preparation for a future codebase refactoring that will change property types from arrays/objects to lists across multiple cmdlets.
- Added breaking change announcements for 15 CloudService cmdlets in the ChangeLog
- Updated module dependencies and configuration files for the AutoRest generation
- Added OutputBreakingChange attributes to affected custom cmdlet implementations
Reviewed Changes
Copilot reviewed 14 out of 14 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| src/CloudService/CloudService/ChangeLog.md | Added breaking change announcement for 15 cmdlets changing from array/object to list types |
| src/CloudService/CloudService/Az.CloudService.psd1 | Updated Az.Accounts dependency version and module configuration |
| src/CloudService/CloudService.sln | Updated solution file with new project GUIDs and platform configurations |
| src/CloudService/CloudService.Autorest/README.md | Added AutoRest breaking change directives for all affected cmdlets |
| src/CloudService/CloudService.Autorest/custom/*.ps1 | Added OutputBreakingChange attributes to custom cmdlet implementations |
| src/CloudService/CloudService.Autorest/docs/Az.CloudService.md | Updated module GUID |
| src/CloudService/CloudService.Autorest/generate-info.json | Updated generation ID |
| src/CloudService/CloudService.Autorest/Properties/AssemblyInfo.cs | Added new assembly information file |
| src/CloudService/CloudService.Autorest/resources/README.md | Added new resources directory documentation |
| - Additional information about change #1 | ||
| --> | ||
| ## Upcoming Release | ||
| * Added breaking change announcement for below commands from array or object to list. |
Copilot
AI
Sep 2, 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.
The phrase 'for below commands' is grammatically incorrect. It should be 'for the following commands' or 'for the commands below'.
| * Added breaking change announcement for below commands from array or object to list. | |
| * Added breaking change announcement for the following commands from array or object to list. |
#28572
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.