Skip to content

Conversation

jchavaherrera
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 Copilot AI review requested due to automatic review settings August 12, 2025 15:00
@github-actions github-actions bot added the Mgmt This issue is related to a management package. label Aug 12, 2025
Copy link
Contributor

@Copilot 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 creates a new Azure SDK for Virtual Enclaves management operations. The SDK follows Azure Resource Manager patterns and provides .NET bindings for managing Virtual Enclaves resources in Azure.

Key changes include:

  • Initial project structure and configuration files for the Virtual Enclaves SDK
  • Standard Azure SDK documentation and project setup
  • Sample and test project scaffolding

Reviewed Changes

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

Show a summary per file
File Description
src/Azure.ResourceManager.VirtualEnclaves.csproj Main SDK project file with package metadata and version information
samples/Azure.ResourceManager.VirtualEnclaves.Samples.csproj Sample project configuration with necessary dependencies
assets.json Asset repository configuration for SDK assets management
README.md Comprehensive documentation with usage examples and getting started guide
Directory.Build.props MSBuild properties configuration for the package
CHANGELOG.md Release history documentation for version 1.0.0-beta.1
Azure.ResourceManager.VirtualEnclaves.sln Visual Studio solution file containing all projects

Copy link
Member

@jsquire jsquire left a comment

Choose a reason for hiding this comment

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

@jchavaherrera:

Your account lacks the public memberships to the Microsoft and Azure GitHub organizations required of an internal contributor. Any memberships that you currently hold for these organizations are marked as private. Please review the Azure SDK onboarding documentation and use the associated Teams channel for support.

You can verify the state of your account by running the Validate-AzsdkCodeOwner script from the Azure SDK tools repository.

Please also be sure to add yourself to CODEOWNERS for this library, if you will be maintaining it going forward.

@jchavaherrera jchavaherrera requested review from m-redding and a team as code owners August 19, 2025 19:15
@jchavaherrera
Copy link
Member Author

@jchavaherrera:

Your account lacks the public memberships to the Microsoft and Azure GitHub organizations required of an internal contributor. Any memberships that you currently hold for these organizations are marked as private. Please review the Azure SDK onboarding documentation and use the associated Teams channel for support.

You can verify the state of your account by running the Validate-AzsdkCodeOwner script from the Azure SDK tools repository.

Please also be sure to add yourself to CODEOWNERS for this library, if you will be maintaining it going forward.

@jsquire:
Thanks for the observations, I changed my memberships to public, added myself to the codeowners file as well as a group with the engineers that will maintain the code.

Let me know if more changes are needed

@jsquire
Copy link
Member

jsquire commented Aug 19, 2025

@jchavaherrera: Account changes look good. You'll need to engage @ronniegeraghty to discuss a service label, as those are centrally managed.

@jchavaherrera
Copy link
Member Author

@jchavaherrera: Account changes look good. You'll need to engage @ronniegeraghty to discuss a service label, as those are centrally managed.

@jsquire Got it! in the meantime I removed the service label and I'll talk to him about it. Thanks!

@@ -918,6 +918,9 @@
# ServiceLabel: %VideoAnalyzer
# ServiceOwners: @giakas

# ServiceLabel: %VirtualEnclaves
Copy link
Member

Choose a reason for hiding this comment

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

After you have a valid label, this will need to move into the management section that starts on L955 and include %Mgmt. Entries should be ordered alphabetically by label name.

Copy link

github-actions bot commented Aug 19, 2025

API Change Check

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

Azure.ResourceManager.VirtualEnclaves

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