From 15cd58a517d6e53fc23178d4d515a9dc47a179fa Mon Sep 17 00:00:00 2001 From: jack-berg <34418638+jack-berg@users.noreply.github.com> Date: Fri, 16 Aug 2024 14:01:14 -0500 Subject: [PATCH] Update CODEOWNERS to reflect spec sponsor role (#4170) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Followup to https://github.com/open-telemetry/community/pull/2070 Should be merged in lockstep with https://github.com/open-telemetry/opentelemetry-proto/pull/572 Upon merging, I will: - Rename [spec-approvers](https://github.com/orgs/open-telemetry/teams/specs-approvers) to `spec-sponsors` - Add all [current sponsors](https://github.com/open-telemetry/community/blob/main/community-members.md#specifications-and-proto) to `spec-sponsors` - Delete [spec-metrics-approvers](https://github.com/orgs/open-telemetry/teams/specs-metrics-approvers), [spec-trace-approvers](https://github.com/orgs/open-telemetry/teams/specs-trace-approvers), [spec-logs-approvers](https://github.com/orgs/open-telemetry/teams/specs-logs-approvers) Note, there are still references to spec approvers in [CONTRIBUTING.md](https://github.com/open-telemetry/opentelemetry-specification/blob/main/CONTRIBUTING.md). As discussed in #3821 there are a variety of issues with the spec issue management process, including consistency. I'm not trying to solve all of them in this PR - rather, I want to make incremental progress and reflect a decision that was already made about the spec sponsor role. --------- Co-authored-by: Trask Stalnaker Co-authored-by: Robert PajÄ…k Co-authored-by: Cijo Thomas --- .github/CODEOWNERS | 20 +------------------- 1 file changed, 1 insertion(+), 19 deletions(-) diff --git a/.github/CODEOWNERS b/.github/CODEOWNERS index fdf64f6758a..707f7a48dfb 100644 --- a/.github/CODEOWNERS +++ b/.github/CODEOWNERS @@ -13,22 +13,4 @@ # # Global owners, will be the owners for everything in the repo. Membership is tracked via https://github.com/open-telemetry/community/blob/main/community-members.md -* @open-telemetry/technical-committee @open-telemetry/specs-approvers - -# Trace owners (global + trace) -development/trace/ @open-telemetry/technical-committee @open-telemetry/specs-approvers @open-telemetry/specs-trace-approvers -specification/trace/ @open-telemetry/technical-committee @open-telemetry/specs-approvers @open-telemetry/specs-trace-approvers - -# Metrics owners (global + metrics) -development/metrics/ @open-telemetry/technical-committee @open-telemetry/specs-approvers @open-telemetry/specs-metrics-approvers -specification/metrics/ @open-telemetry/technical-committee @open-telemetry/specs-approvers @open-telemetry/specs-metrics-approvers - -# Logs owners (global + logs) -development/logs/ @open-telemetry/technical-committee @open-telemetry/specs-approvers @open-telemetry/specs-logs-approvers -specification/logs/ @open-telemetry/technical-committee @open-telemetry/specs-approvers @open-telemetry/specs-logs-approvers - -# Semantic Conventions owners (global) -specification/semantic-conventions.md @open-telemetry/technical-committee @open-telemetry/specs-approvers @open-telemetry/specs-semconv-approvers -specification/telemetry-stability.md @open-telemetry/technical-committee @open-telemetry/specs-approvers @open-telemetry/specs-semconv-approvers -# Note: Common is about naming conventions, relevant to semconv -specification/common/ @open-telemetry/technical-committee @open-telemetry/specs-approvers @open-telemetry/specs-semconv-approvers +* @open-telemetry/technical-committee @open-telemetry/spec-sponsors