Skip to content

Soft deprecate azure-resource-provider-folder and remove from feeds#3836

Open
timotheeguerin wants to merge 11 commits intoAzure:mainfrom
timotheeguerin:deprecate-arm-resource-provider-folder
Open

Soft deprecate azure-resource-provider-folder and remove from feeds#3836
timotheeguerin wants to merge 11 commits intoAzure:mainfrom
timotheeguerin:deprecate-arm-resource-provider-folder

Conversation

@timotheeguerin
Copy link
Member

Right now this just document as deprecated but doesn't flag a warning as we are still deciding if we should fix v1 spec or wait until they are migrated.

@microsoft-github-policy-service microsoft-github-policy-service bot added eng emitter:autorest Issues for @azure-tools/typespec-autorest emitter labels Jan 24, 2026
@pkg-pr-new
Copy link

pkg-pr-new bot commented Jan 24, 2026

Open in StackBlitz

npm i https://pkg.pr.new/Azure/typespec-azure/@azure-tools/typespec-autorest@3836
npm i https://pkg.pr.new/Azure/typespec-azure/@azure-tools/typespec-autorest-canonical@3836

commit: 725e000

@azure-sdk
Copy link
Collaborator

azure-sdk commented Jan 24, 2026

All changed packages have been documented.

  • @azure-tools/typespec-autorest-canonical
  • @azure-tools/typespec-autorest
Show changes

@azure-tools/typespec-autorest - deprecation ✏️

Deprecate azure-resource-provider-folder option.,> ,> diff lang=yaml,> -azure-resource-provider-folder: "resource-manager",> -output-file: "{azure-resource-provider-folder}/{service-name}/{version-status}/{version}/openapi.json",> +output-file: "resource-manager/{service-name}/{version-status}/{version}/openapi.json",>

@azure-tools/typespec-autorest-canonical - deprecation ✏️

Deprecate azure-resource-provider-folder option.,> ,> diff lang=yaml,> -azure-resource-provider-folder: "resource-manager",> -output-file: "{azure-resource-provider-folder}/{service-name}/{version-status}/{version}/openapi.json",> +output-file: "resource-manager/{service-name}/{version-status}/{version}/openapi.json",>

@azure-sdk
Copy link
Collaborator

azure-sdk commented Jan 24, 2026

You can try these changes here

🛝 Playground 🌐 Website

@timotheeguerin timotheeguerin added the int:azure-specs Run integration tests against azure-rest-api-specs label Jan 24, 2026
azure-resource-provider-folder: "data-plane"
emitter-output-dir: "{project-root}/.."
output-file: "{azure-resource-provider-folder}/{service-name}/{version-status}/{version}/openapi.json"
output-file: "data-plane/{service-name}/{version-status}/{version}/openapi.json"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this should also be {emitter-output-dir}/{version-status}/{version}/openapi.json

): Promise<Record<K, OpenAPI2Document>> {
return compileMultipleOpenAPI(
code,
Object.fromEntries(versions.map((x) => [x, resolvePath("stable", x, "openapi.json")])),
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Perhaps we should also try to resolve in preview, if this produces nothing?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

emitter:autorest Issues for @azure-tools/typespec-autorest emitter eng int:azure-specs Run integration tests against azure-rest-api-specs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants