Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Dec 11, 2025

Description

Many RPs use Azure.ResourceManager.Legacy.Operations.list instead of the standard Azure.ResourceManager.Operations.list. Both are already provided by the Azure.ResourceManager base library and should not be generated in client SDKs.

Changes:

  • Added Azure.ResourceManager.Legacy.Operations.list to the _methodsToOmit HashSet in ManagementInputLibrary.cs, matching the existing omission pattern for the non-legacy version

This ensures the legacy operations list method is automatically filtered during code generation alongside the standard version.


This checklist is used to make sure that common guidelines for a pull request are followed.

General Guidelines

  • Title of the pull request is clear and informative.
  • There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

  • Pull request includes test coverage for the included changes.

SDK Generation Guidelines

  • If an SDK is being regenerated based on a new swagger spec, a link to the pull request containing these swagger spec changes has been included above.
  • The generate.cmd file for the SDK has been updated with the version of AutoRest, as well as the commitid of your swagger spec or link to the swagger spec, used to generate the code.
  • The *.csproj and AssemblyInfo.cs files have been updated with the new version of the SDK.
Original prompt

This section details on the original issue you should resolve

<issue_title>[Azure Mgmt Generator] Auto omit Azure.ResourceManager.Legacy.Operations.list as well</issue_title>
<issue_description>We have omitted the operation with crossLanguageDefinitionId Azure.ResourceManager.Operations.list, but in a lot of RPs, they are using the legacy version, we should automatically omit this as well.</issue_description>

Comments on the Issue (you are @copilot in this section)


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Copilot AI changed the title [WIP] Auto omit Azure.ResourceManager.Legacy.Operations.list [Azure Mgmt Generator] Auto omit Azure.ResourceManager.Legacy.Operations.list Dec 11, 2025
Copilot AI requested a review from ArcturusZhang December 11, 2025 09:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Azure Mgmt Generator] Auto omit Azure.ResourceManager.Legacy.Operations.list as well

2 participants