From d8e542e29f7758816744f5bef14806649de51ca9 Mon Sep 17 00:00:00 2001 From: Sven Tiffe <63734962+sventiffe@users.noreply.github.com> Date: Wed, 26 Jan 2022 10:17:56 +0100 Subject: [PATCH 1/2] Clarification of priority label semantics --- maintaining/maintainers-guide.md | 19 +++++++++++-------- 1 file changed, 11 insertions(+), 8 deletions(-) diff --git a/maintaining/maintainers-guide.md b/maintaining/maintainers-guide.md index 90cc1af0d9..01efac6a3b 100644 --- a/maintaining/maintainers-guide.md +++ b/maintaining/maintainers-guide.md @@ -142,28 +142,31 @@ issues. compliant to the [Breaking Change](https://docs.google.com/document/d/1q5GGRxKrF_mnwtaPKI487P8OdDRh2nN7jX6U-FXnHL0/edit?pli=1#heading=h.ceof6vpkb3ik) policy. No practical workaround exists. + * Issues marked as P0 are under active development. * [**P1**](https://github.com/bazelbuild/bazel/labels/P1) - Critical defect or feature which should be addressed in the next release, or a serious issue that impacts many users (including the development of the Bazel project), but a practical workaround exists. Typically does not require immediate action. In high demand and planned in the current quarter's roadmap. + * Issues marked as P1 are under active development. * [**P2**](https://github.com/bazelbuild/bazel/labels/P2) - Defect or feature that should be addressed but we don't currently work on. Moderate live issue in a released Bazel version that is inconvenient for a user that needs to be addressed in an future release and/or a easy workaround exists. + * Issues marked as P2 are part of the actively maintained backlog. While the + Bazel team is not actively working on these issues, they are considered to + be re-reviewed at later point in time. * [**P3**](https://github.com/bazelbuild/bazel/labels/P3) - Desirable minor bug fix or enhancement with small impact. Not prioritized into Bazel roadmaps or - any imminent release. May never be fixed. + any imminent release. May never be fixed by the Bazel team. + * Issues marked as P3 are not actively managed by the Bazel team. + Community contributions are encouraged, the Bazel team will accept PRs. + Please reach out to the team prior submitting PRs. * [**P4**](https://github.com/bazelbuild/bazel/labels/P4) - Low priority defect or feature request that is unlikely to get closed. Can also be kept open for a potential re-prioritization if more users are impacted. -* [**ice-box**](https://github.com/bazelbuild/bazel/issues?q=label%3Aice-box+is%3Aclosed) - - Issues that we currently don't have time to deal with nor the - time to accept contributions. We will close these issues to indicate that - nobody is working on them, but will continue to monitor their validity over - time and revive them if enough people are impacted and if we happen to have - resources to deal with them. As always, feel free to comment or add reactions - to these issues even when closed. + * Issues marked as P4 are not actively managed until further notice. + Unsolicited PRs will be rejected. ## Team labels From a51eae2cce4174688874f5ff69ec6a29fc001cb2 Mon Sep 17 00:00:00 2001 From: Sven Tiffe <63734962+sventiffe@users.noreply.github.com> Date: Tue, 1 Mar 2022 20:19:50 +0100 Subject: [PATCH 2/2] Update maintainers-guide.md --- maintaining/maintainers-guide.md | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/maintaining/maintainers-guide.md b/maintaining/maintainers-guide.md index 01efac6a3b..9b902a192e 100644 --- a/maintaining/maintainers-guide.md +++ b/maintaining/maintainers-guide.md @@ -160,8 +160,9 @@ issues. fix or enhancement with small impact. Not prioritized into Bazel roadmaps or any imminent release. May never be fixed by the Bazel team. * Issues marked as P3 are not actively managed by the Bazel team. - Community contributions are encouraged, the Bazel team will accept PRs. - Please reach out to the team prior submitting PRs. + Community contributions are welcome. Before sending a PR, please reach out + in the issue first to discuss your solution and work with the Bazel team to + find the best reviewer. * [**P4**](https://github.com/bazelbuild/bazel/labels/P4) - Low priority defect or feature request that is unlikely to get closed. Can also be kept open for a potential re-prioritization if more users are impacted.