Add .NET management emitter to Authorization tspconfig.yaml#42261
Add .NET management emitter to Authorization tspconfig.yaml#42261jruttle wants to merge 1 commit intoAzure:mainfrom
Conversation
Add @azure-typespec/http-client-csharp-mgmt emitter configuration to enable .NET SDK generation for Microsoft.Authorization. The other four language emitters (Python, Java, TypeScript, Go) are already present — .NET was the only one missing, which blocked .NET SDK auto-generation. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Next Steps to Merge✅ All automated merging requirements have been met! To get your PR merged, see aka.ms/azsdk/specreview/merge.Comment generated by summarize-checks workflow run. |
ARM Review Request — Config-only changeThis PR makes a minimal config-only change — it adds 3 lines to ContextThe Authorization All 4 other SDK PRs have been successfully created from the existing tspconfig:
AvocadoAdded Could this get expedited ARM sign-off given it's purely an SDK emitter config addition with no API surface changes? Thank you! |
|
/merge |
|
Hi @ArthurMa1978 @darshanhs90 — this is a 3-line config-only change adding the .NET management emitter to the Authorization tspconfig.yaml (the only missing language emitter). All checks are passing and the |
|
Hi @ArthurMa1978 @darshanhs90 — following up on this PR. It's a 3-line config-only change adding the .NET management emitter to the Authorization The Avocado check failure is a pre-existing @iliu816 — could one of you approve so the |
|
@jruttle I'm not a code owner for Authorization. It seems like this request should go to @darshanhs90 according to azure-rest-api-specs/.github/CODEOWNERS Line 45 in fecd9f0 |
|
Thanks @iliu816 for pointing that out — really appreciate the help! 🙏 @darshanhs90 — could you take a look at this PR when you get a chance? It's a 3-line config-only change adding the .NET management emitter to the Authorization |
Summary
Add the
@azure-typespec/http-client-csharp-mgmtemitter configuration to the Authorizationtspconfig.yaml. This is the only missing language emitter - Python, Java, TypeScript, and Go are already configured.Motivation
The .NET SDK generation pipeline (7412) ran successfully after the TypeSpec PR (#41617) merged, but produced no changes because the tspconfig.yaml was missing the .NET emitter. This blocked auto-generation of the
Azure.ResourceManager.AuthorizationNuGet package with deny assignment CRUD support.All four other SDKs have been successfully generated:
Changes
@azure-typespec/http-client-csharp-mgmtemitter with:namespace: "Azure.ResourceManager.Authorization"emitter-output-dir: "{output-dir}/{service-dir}/{namespace}"This matches the pattern used by other management plane services (e.g., Microsoft.Advisor, Microsoft.Batch, Microsoft.AppConfiguration).
Checklist