[Storage] List SKU #28581
[Storage] List SKU #28581
Conversation
| Thanks for your contribution! The pull request validation has started. Please revisit this comment for updated status. |
There was a problem hiding this comment.
Pull Request Overview
This PR adds a new Get-AzStorageSku cmdlet to the Azure Storage PowerShell module that lists available SKUs supported by Microsoft.Storage for a given subscription.
- Introduces a new AutoRest-generated cmdlet for retrieving storage SKUs
- Updates module dependencies and documentation
- Refactors some parameter names in existing cmdlets for consistency
Reviewed Changes
Copilot reviewed 25 out of 25 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
| src/Storage/Storage.sln | Updates project GUID for AutoRest Storage project |
| src/Storage/Storage.Management/Az.Storage.psd1 | Adds new Get-AzStorageSku function and updates required module version |
| src/Storage/Storage.Management/help/*.md | Updates help documentation with new cmdlet and fixes line wrapping issues |
| src/Storage/Storage.Autorest/test/Get-AzStorageSku.Tests.ps1 | Adds basic test coverage for the new Get-AzStorageSku cmdlet |
| src/Storage/Storage.Autorest/README.md | Updates API version references and removes Sku subject from exclusions |
| src/Storage/Storage.Autorest/Properties/AssemblyInfo.cs | Increments assembly version from 9.0.0 to 9.1.0 |
| ``` | ||
|
|
||
| ### DirectoryObject | ||
| ### DirecotryObject |
There was a problem hiding this comment.
Parameter set name 'DirecotryObject' contains a spelling error. It should be 'DirectoryObject'.
| ### DirecotryObject | |
| ### DirectoryObject |
Copilot uses AI. Check for mistakes.
| ``` | ||
|
|
||
| ### DirectoryObject | ||
| ### DirecotryObject |
There was a problem hiding this comment.
Parameter set name 'DirecotryObject' contains a spelling error. It should be 'DirectoryObject'.
| ### DirecotryObject | |
| ### DirectoryObject |
Copilot uses AI. Check for mistakes.
| ## PARAMETERS | ||
| ### -DefaultProfile | ||
| The credentials, account, tenant, and subscription used for communication with Azure. |
There was a problem hiding this comment.
The PARAMETERS section appears to be duplicated entirely in this help file. Remove the duplicate content from lines 1274-1420 to avoid redundancy and maintain clean documentation.
Copilot uses AI. Check for mistakes.
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.