Skip to content

Conversation

@azure-sdk
Copy link
Collaborator

Generated by branded - http-client-csharp-mgmt build 20251205.1
Triggered from branch: main

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 TypeSpec HTTP Client C# Management generator from version 1.0.0-alpha.20251202.1 to 1.0.0-alpha.20251205.1, resulting in standardized XML documentation comments across generated SDK files.

Key Changes:

  • Updated generator dependency version in package.json and package-lock.json files
  • Regenerated SDK collection and resource files with standardized documentation comments
  • All changes are documentation-only, with no functional code modifications

Reviewed changes

Copilot reviewed 1 out of 116 changed files in this pull request and generated 1 comment.

File Description
eng/azure-typespec-http-client-csharp-mgmt-emitter-package.json Updated generator version dependency
eng/azure-typespec-http-client-csharp-mgmt-emitter-package-lock.json Updated lock file with new dependency version and integrity hash
sdk/**/Generated/*Collection.cs (100+ files) Standardized Exists() and GetIfExists() method documentation
sdk/**/Generated/*Resource.cs (50+ files) Standardized Update() method documentation

Note: There is a consistent spelling issue across all generated files where "azure" should be capitalized as "Azure" in the documentation comments. This appears to be a generator-level issue that should be addressed in the generator itself rather than in individual files.

Files not reviewed (1)
  • eng/azure-typespec-http-client-csharp-mgmt-emitter-package-lock.json: Language not supported


/// <summary>
/// Get a InstanceResource
/// Checks to see if the resource exists in azure.
Copy link

Copilot AI Dec 6, 2025

Choose a reason for hiding this comment

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

The word "azure" should be capitalized to "Azure" to maintain consistency with Microsoft branding and standard capitalization practices.

Copilot uses AI. Check for mistakes.
@github-actions
Copy link

github-actions bot commented Dec 6, 2025

API Change Check

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

Azure.ResourceManager.EdgeActions
Azure.ResourceManager.ImpactReporting

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant