Skip to content

Conversation

@ivarprudnikov
Copy link
Member

Added users from the team who contributed to the source in the past year

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.

@ivarprudnikov ivarprudnikov marked this pull request as ready for review December 8, 2025 18:15
Copilot AI review requested due to automatic review settings December 8, 2025 18:15
@ivarprudnikov ivarprudnikov requested review from a team, jsquire and m-redding as code owners December 8, 2025 18:15
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 adds 6 new service owners to the Confidential Ledger SDK team, representing contributors who have worked on the codebase in the past year.

  • Added 6 new GitHub users as code owners and service owners for the confidentialledger SDK
  • Updated both PRLabel and ServiceOwners sections consistently

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.

There's a few accounts that need adjustments for this to merge:

  • prathod09 is missing a public membership to the Microsoft org
  • ryazhang-microsoft is missing a public membership to the Microsoft org
  • msftsettiy is missing a public membership to the Microsoft org
  • weirenlai is missing a public membership to the Microsoft org

Please review the Azure SDK onboarding documentation and use the associated Teams channel for support.

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

@ivarprudnikov
Copy link
Member Author

@jsquire Accounts updated, it seems that codeowners linter was only checking for Azure members but not Microsoft org membership IMO.

@ivarprudnikov ivarprudnikov requested a review from jsquire December 9, 2025 18:44
@jsquire jsquire merged commit 59b3f81 into Azure:main Dec 10, 2025
13 checks passed
@ivarprudnikov ivarprudnikov deleted the chore/add-confidentialledger-as-contributors branch December 10, 2025 10:30
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.

4 participants