upgrade websites module to autorest v4#28268
Open
Pan-Qi wants to merge 39 commits intoautorest-v4-breakingchangefrom
Open
upgrade websites module to autorest v4#28268Pan-Qi wants to merge 39 commits intoautorest-v4-breakingchangefrom
Pan-Qi wants to merge 39 commits intoautorest-v4-breakingchangefrom
Conversation
| Thanks for your contribution! The pull request validation has started. Please revisit this comment for updated status. |
bf40356 to
345bc0f
Compare
Contributor
There was a problem hiding this comment.
Pull Request Overview
This PR upgrades the Azure PowerShell Websites module to use AutoRest v4, introducing various new features and breaking changes.
- Upgrades the code generator from AutoRest v3 to v4, bringing new parameter sets and parameter types
- Updates cmdlet signatures, particularly for Static Web App cmdlets with new identity management parameters
- Changes many API response models from versioned namespaces to unversioned interfaces
Reviewed Changes
Copilot reviewed 112 out of 112 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| tools/StaticAnalysis/Exceptions/Az.LoadTesting/BreakingChangeIssues.csv | Documents breaking changes including type changes, removed parameters, and removed parameter sets |
| src/Websites/Websites/help/*.md | Updates documentation for cmdlets with new parameter sets, type changes, and corrected descriptions |
| src/Websites/Websites/ChangeLog.md | Documents the upgrade and breaking changes for New-AzStaticWebApp identity parameters |
| src/Websites/Websites/Az.Websites.psd1 | Updates module metadata including required Az.Accounts version and file paths |
| ### Example 2: Create or updates the app settings of a static site build by pipeline | ||
| ```powershell | ||
| Get-AzStaticWebAppBuildAppSetting -ResourceGroupName resourceGroup -Name staticweb00 -EnvironmentName 'default' | New-AzStaticWebAppBuildAppSetting -AppSetting @{'buildsetting1' = 'someval'; 'buildsetting2' = 'someval2' } | ||
| Get-AzStaticWebAppBuildAppSetting -ResourceGroupName resourceGroup -Name taticweb00 -EnvironmentName 'default' | New-AzStaticWebAppBuildAppSetting -AppSetting @{'buildsetting1' = 'someval'; 'buildsetting2' = 'someval2' } |
There was a problem hiding this comment.
There's a typo in the example: 'taticweb00' should be 'staticweb00' to match the expected static web app naming convention.
Suggested change
| Get-AzStaticWebAppBuildAppSetting -ResourceGroupName resourceGroup -Name taticweb00 -EnvironmentName 'default' | New-AzStaticWebAppBuildAppSetting -AppSetting @{'buildsetting1' = 'someval'; 'buildsetting2' = 'someval2' } | |
| Get-AzStaticWebAppBuildAppSetting -ResourceGroupName resourceGroup -Name staticweb00 -EnvironmentName 'default' | New-AzStaticWebAppBuildAppSetting -AppSetting @{'buildsetting1' = 'someval'; 'buildsetting2' = 'someval2' } |
|
To the author of the pull request, |
6 tasks
Member
|
/azp run |
Contributor
|
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. |
Member
|
/azp run |
Contributor
|
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. |
naveedaz
approved these changes
Feb 10, 2026
Contributor
|
/azp run |
Contributor
|
Commenter does not have sufficient privileges for PR 28268 in repo Azure/azure-powershell |
Contributor
|
@mishapos and @skylathadani-ms can you guys review too? |
…bug fix (#29414) Co-authored-by: Sam Lee <samlee3@microsoft.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Co-authored-by: Yabo Hu <yabhu@microsoft.com>
#29439) Co-authored-by: JingnanXu <jingnanxu@microsoft.com>
Co-authored-by: Devansh Goswami <degoswami@microsoft.com> Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
…dation (#29445) Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: audreyttt <225061541+audreyttt@users.noreply.github.com> Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> Co-authored-by: Audrey Toney <audreytoney@microsoft.com>
…kup (#29408) Co-authored-by: Selasie Mortey <smortey@microsoft.com> Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
…ke idempotent (#29473) Co-authored-by: Jonathan Ruttle <jruttle@microsoft.com> Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Co-authored-by: Yabo Hu <yabhu@microsoft.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
Preannouncement PR:
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.