From 2d8ba2580cc905ede0a660108422830cb7bb0a6f Mon Sep 17 00:00:00 2001 From: Daniel Jaglowski Date: Mon, 12 Feb 2024 11:33:23 -0600 Subject: [PATCH] [chore] Define code ownership in slightly more open terms (#31117) See #31115 Our existing definition and process for code ownership are quite good but this PR suggests a few minor tweaks. - Add triaging as a code owner responsibility. Since this is arguably the easiest thing a code owner can do, we should highlight that it is helpful work. - Move "The existing Code Owners are actively looking for more help" to the top of the list of reasons why additional code owners may be needed, since this is most often the case. - Add a note about code ownership not needing to be a full time job. The goal is not to lower the bar in terms of quality, but to encourage lower bandwidth contributors not to self-select out. --- CONTRIBUTING.md | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md index 3b2e20fd330f..7a7efcb7d413 100644 --- a/CONTRIBUTING.md +++ b/CONTRIBUTING.md @@ -361,14 +361,16 @@ Example label comment: ## Becoming a Code Owner -A Code Owner is responsible for a component within Collector Contrib, as indicated by the [CODEOWNERS file](https://github.com/open-telemetry/opentelemetry-collector-contrib/blob/main/.github/CODEOWNERS). That responsibility includes maintaining the component, responding to issues, and reviewing pull requests. +A Code Owner is responsible for a component within Collector Contrib, as indicated by the [CODEOWNERS file](https://github.com/open-telemetry/opentelemetry-collector-contrib/blob/main/.github/CODEOWNERS). That responsibility includes maintaining the component, triaging and responding to issues, and reviewing pull requests. Sometimes a component may be in need of a new or additional Code Owner. A few reasons this situation may arise would be: -- The component was never assigned a Code Owner. + +- The existing Code Owners are actively looking for more help. - A previous Code Owner stepped down. - An existing Code Owner has become unresponsive. See [unmaintained stability status](https://github.com/open-telemetry/opentelemetry-collector#unmaintained). -- The existing Code Owners are actively looking for new Code Owners to help. +- The component was never assigned a Code Owner. +Code Ownership does not have to be a full-time job. If you can find a couple hours to help out on a recurring basis, please consider pursuing Code Ownership. ### Requirements