Skip to content

Conversation

@haiyuazhang
Copy link
Member

Contributing to the Azure SDK

Please see our CONTRIBUTING.md if you are not familiar with contributing to this repository or have questions.

For specific information about pull request etiquette and best practices, see this section.

Copilot AI review requested due to automatic review settings November 3, 2025 03:08
@github-actions github-actions bot added the Mgmt This issue is related to a management package. label Nov 3, 2025
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR updates the Azure.ResourceManager.DependencyMap SDK by regenerating code from updated TypeSpec specifications. The changes primarily involve:

  • Updating the commit hash reference in tsp-location.yaml
  • Adding an emitter package JSON path configuration
  • Removing all sample test files
  • Extensive refactoring of generated REST operations and model classes to use newer code generation patterns

Reviewed Changes

Copilot reviewed 3 out of 96 changed files in this pull request and generated no comments.

Show a summary per file
File Description
tsp-location.yaml Updated spec commit hash and added emitter configuration
Sample_*.cs (tests) Removed all generated sample test files
RestOperations files Refactored from high-level API methods to low-level HTTP message creation
Model classes Updated serialization patterns, renamed internal fields, improved documentation
ProviderConstants.cs Changed from static to partial class, added XML documentation
CodeGenTypeAttribute.cs Added new internal attribute for code generation metadata

@github-actions
Copy link

github-actions bot commented Nov 3, 2025

API Change Check

APIView identified API level changes in this PR and created the following API reviews

Azure.ResourceManager.DependencyMap

@haiyuazhang haiyuazhang marked this pull request as draft November 3, 2025 03:20
@haiyuazhang haiyuazhang marked this pull request as ready for review November 3, 2025 05:36
@haiyuazhang haiyuazhang merged commit 65c57f1 into Azure:main Nov 3, 2025
17 checks passed
nick863 pushed a commit that referenced this pull request Nov 5, 2025
* update tsp-location.yaml.

* working in progress.

* update commit id in tsp-location.yaml

* regen.

* regen.

* update commit id in tsp-location.yaml

* update commit id and Library_Iventory.md

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

Labels

Mgmt This issue is related to a management package.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants