Skip to content

Update CODEOWNERS: fabrizio is already in ingest-docs team; use apm-agent-net team - #313

Merged
trentm merged 1 commit into
mainfrom
trentm-codeowners-docs-group
Oct 30, 2025
Merged

trentm merged 1 commit into
mainfrom
trentm-codeowners-docs-group

Conversation

@trentm

@trentm trentm commented Oct 21, 2025

Copy link
Copy Markdown
Member
  1. @theletterf This drops your user specifically as an owner, in favour of the existing ingest-docs team, which is what the other elastic-otel-*/.github/CODEOWNERS files do.
  2. @stevejgordon Would you like to have @elastic/apm-agent-net team (https://github.com/orgs/elastic/teams/apm-agent-net) be an automatic reviewer for all PRs? Totally your call. The Java, Node and Python teams are doing this. EDOT PHP is not.

@trentm trentm self-assigned this Oct 21, 2025

@theletterf theletterf left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thank you!

Comment thread .github/CODEOWNERS

@stevejgordon stevejgordon left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM, thanks

@trentm
trentm enabled auto-merge (squash) October 30, 2025 20:33
@trentm
trentm merged commit fe6a654 into main Oct 30, 2025
9 checks passed
@trentm
trentm deleted the trentm-codeowners-docs-group branch November 3, 2025 21:18
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.

3 participants