Skip to content

adding myself as approver; removing trask#53074

Merged
mattsains-msft merged 1 commit intoAzure:mainfrom
mattsains-msft:codeowner-mattsains
Oct 10, 2025
Merged

adding myself as approver; removing trask#53074
mattsains-msft merged 1 commit intoAzure:mainfrom
mattsains-msft:codeowner-mattsains

Conversation

@mattsains-msft
Copy link
Copy Markdown
Member

No description provided.

Copilot AI review requested due to automatic review settings October 8, 2025 18:50
@mattsains-msft mattsains-msft requested review from a team, jsquire and m-redding as code owners October 8, 2025 18:50
Copy link
Copy Markdown
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 CODEOWNERS file to replace one team member with another as an approver for Azure Monitor OpenTelemetry components. The change removes @trask and adds @mattsains-msft as a code owner across multiple Azure Monitor packages.

Key changes:

  • Updated code ownership assignments for Azure Monitor OpenTelemetry components
  • Maintained consistent reviewer coverage by replacing one reviewer with another

@github-actions github-actions Bot added Community Contribution Community members are working on the issue customer-reported Issues that are reported by GitHub users external to the Azure organization. labels Oct 8, 2025
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Oct 8, 2025

Thank you for your contribution @mattsains-msft! We will review the pull request and get back to you soon.

@mattsains-msft mattsains-msft enabled auto-merge (squash) October 8, 2025 19:19
@weshaggard
Copy link
Copy Markdown
Member

.github/CODEOWNERS(637,1): error : Error(s) on line 637 Source Line: /sdk/monitor/Azure.Monitor.OpenTelemetry.AspNetCore/ @rajkumar-rangaraj @harsimar @mattsains-msft @xiang17 -mattsains-msft is an invalid user. Ensure the user exists, is public member of Azure and has write permissions.

@mattsains-msft please follow the instructions at https://aka.ms/azsdk/access to get the correct permissions first.

Copy link
Copy Markdown
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.

@mattsains-msft :

Your account lacks the public GitHub organization memberships and permissions required of an internal contributor. 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.

Copy link
Copy Markdown
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.

Account looks good!

@mattsains-msft mattsains-msft merged commit fb79304 into Azure:main Oct 10, 2025
17 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Community Contribution Community members are working on the issue customer-reported Issues that are reported by GitHub users external to the Azure organization.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants