Quota Module TSP Migration - #29946
Draft
JiaSeng-v wants to merge 1 commit into
Draft
Conversation
Contributor
|
Azure Pipelines: Successfully started running 1 pipeline(s). There may be pipelines that require an authorized user to comment /azp run to run. |
Member
|
/azp run |
Contributor
|
Azure Pipelines: Successfully started running 3 pipeline(s). |
Contributor
There was a problem hiding this comment.
Pull request overview
This PR updates the Az.Quota module’s AutoRest-based project layout/configuration as part of a TypeSpec (TSP) migration, refreshing solution structure, generation inputs, module manifest metadata, and generated help/doc artifacts.
Changes:
- Added TypeSpec configuration and spec pinning for Quota generation (
tspconfig.yaml,tsp-location.yaml). - Updated module packaging/metadata (manifest + changelog) and refreshed generated docs/help content.
- Updated the Quota solution to reference the generated Quota.Autorest project path and added additional build configurations.
Reviewed changes
Copilot reviewed 18 out of 18 changed files in this pull request and generated 8 comments.
Show a summary per file
| File | Description |
|---|---|
| src/Quota/Quota/help/Remove-AzQuotaGroupQuotaSubscription.md | Updates cmdlet reference help content/synopsis for Remove-AzQuotaGroupQuotaSubscription. |
| src/Quota/Quota/help/Get-AzQuotaGroupQuotaSubscription.md | Updates cmdlet reference help content/synopsis for Get-AzQuotaGroupQuotaSubscription. |
| src/Quota/Quota/ChangeLog.md | Adds an Upcoming Release entry describing the change. |
| src/Quota/Quota/Az.Quota.psd1 | Refreshes module manifest metadata (generation date, Az.Accounts requirement, formatting). |
| src/Quota/Quota.sln | Re-points the Az.Quota project to generated output and expands solution configurations/folders. |
| src/Quota/Quota.Autorest/tspconfig.yaml | Introduces TypeSpec PowerShell emitter configuration and directives for Quota. |
| src/Quota/Quota.Autorest/tsp-location.yaml | Pins the REST API spec repo/commit and directory used for generation. |
| src/Quota/Quota.Autorest/test/README.md | Documentation tweak for test folder guidance. |
| src/Quota/Quota.Autorest/resources/README.md | Documentation tweak for resources folder guidance. |
| src/Quota/Quota.Autorest/README.md | Removes the legacy AutoRest YAML configuration block from the readme. |
| src/Quota/Quota.Autorest/Properties/AssemblyInfo.cs | Updates assembly metadata and header comments. |
| src/Quota/Quota.Autorest/generate-info.json | Updates generation identifier metadata. |
| src/Quota/Quota.Autorest/docs/Remove-AzQuotaGroupQuotaSubscription.md | Refreshes generated cmdlet documentation content for Remove-AzQuotaGroupQuotaSubscription. |
| src/Quota/Quota.Autorest/docs/README.md | Documentation tweak for docs folder guidance. |
| src/Quota/Quota.Autorest/docs/Get-AzQuotaGroupQuotaSubscription.md | Refreshes generated cmdlet documentation content for Get-AzQuotaGroupQuotaSubscription. |
| src/Quota/Quota.Autorest/docs/Az.Quota.md | Updates module doc metadata (module GUID). |
| src/Quota/Quota.Autorest/custom/README.md | Documentation updates for custom cmdlet guidance. |
| src/Quota/Quota.Autorest/custom/Az.Quota.custom.psm1 | Updates relative paths used when importing private/internal modules. |
Comment on lines
1
to
+4
| // Copyright (c) Microsoft Corporation. All rights reserved. | ||
| // Licensed under the MIT License. See License.txt in the project root for license information. | ||
| // Changes may cause incorrect behavior and will be lost if the code is regenerated. | ||
| // Copyright (c) Microsoft Corporation. All rights reserved. |
Comment on lines
+25
to
+27
| [assembly: System.Reflection.AssemblyTitleAttribute("Microsoft Azure PowerShell - AzureQuotaExtensionApi")] | ||
| [assembly: System.Reflection.AssemblyFileVersionAttribute("0.2.0")] | ||
| [assembly: System.Reflection.AssemblyVersionAttribute("0.2.0")] |
| - Used in C# and script cmdlets to suppress creating an exported cmdlet at build-time. These cmdlets will *not be exposed* by `Az.Quota`. | ||
| - `Microsoft.Azure.PowerShell.Cmdlets.Quota.InternalExportAttribute` | ||
| - Used in C# cmdlets to route exported cmdlets to the `../internal`, which are *not exposed* by `Az.Quota`. For more information, see [README.md](../internal/README.md) in the `../internal` folder. | ||
| - Used in C# cmdlets to route exported cmdlets to the `..\internal`, which are *not exposed* by `Az.Quota`. For more information, see [README.md](..\internal/README.md) in the `..\internal` folder. |
Comment on lines
20
to
+21
| ## Upcoming Release | ||
| * Internal code generation updated. No user-facing changes in this release. |
Comment on lines
23
to
28
| ### DeleteViaIdentityManagementGroup | ||
| ``` | ||
| Remove-AzQuotaGroupQuotaSubscription -GroupQuotaName <String> [-SubscriptionId <String>] | ||
| -ManagementGroupInputObject <IQuotaIdentity> [-DefaultProfile <PSObject>] [-AsJob] [-NoWait] [-PassThru] | ||
| [-WhatIf] [-Confirm] [<CommonParameters>] | ||
| ``` | ||
|
|
||
| ### DeleteViaIdentityGroupQuota | ||
| ``` | ||
| Remove-AzQuotaGroupQuotaSubscription [-SubscriptionId <String>] -GroupQuotaInputObject <IQuotaIdentity> | ||
| Remove-AzQuotaGroupQuotaSubscription -GroupQuotaName <String> -ManagementGroupInputObject <IQuotaIdentity> | ||
| [-DefaultProfile <PSObject>] [-AsJob] [-NoWait] [-PassThru] [-WhatIf] | ||
| [-Confirm] [<CommonParameters>] | ||
| ``` |
Comment on lines
29
to
33
| ### GetViaIdentityManagementGroup | ||
| ``` | ||
| Get-AzQuotaGroupQuotaSubscription -GroupQuotaName <String> [-SubscriptionId <String[]>] | ||
| -ManagementGroupInputObject <IQuotaIdentity> [-DefaultProfile <PSObject>] | ||
| [<CommonParameters>] | ||
| ``` | ||
|
|
||
| ### GetViaIdentityGroupQuota | ||
| ``` | ||
| Get-AzQuotaGroupQuotaSubscription [-SubscriptionId <String[]>] -GroupQuotaInputObject <IQuotaIdentity> | ||
| Get-AzQuotaGroupQuotaSubscription -GroupQuotaName <String> -ManagementGroupInputObject <IQuotaIdentity> | ||
| [-DefaultProfile <PSObject>] [<CommonParameters>] | ||
| ``` |
Comment on lines
29
to
33
| ### DeleteViaIdentityManagementGroup | ||
| ``` | ||
| Remove-AzQuotaGroupQuotaSubscription -GroupQuotaName <String> -ManagementGroupInputObject <IQuotaIdentity> | ||
| [-SubscriptionId <String>] [-DefaultProfile <PSObject>] [-AsJob] [-NoWait] [-PassThru] [-Confirm] [-WhatIf] | ||
| [<CommonParameters>] | ||
| [-DefaultProfile <PSObject>] [-AsJob] [-NoWait] [-PassThru] [-Confirm] [-WhatIf] [<CommonParameters>] | ||
| ``` |
Comment on lines
28
to
32
| ### GetViaIdentityManagementGroup | ||
| ``` | ||
| Get-AzQuotaGroupQuotaSubscription -GroupQuotaName <String> -ManagementGroupInputObject <IQuotaIdentity> | ||
| [-SubscriptionId <String[]>] [-DefaultProfile <PSObject>] [<CommonParameters>] | ||
| [-DefaultProfile <PSObject>] [<CommonParameters>] | ||
| ``` |
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
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.