Do not consider owners of Public-Plugins as plugin maintainers
Core maintainers are owners of the Public-Plugins project to grant them
permission to submit changes on plugin projects where either no
maintainers are available or don't have sufficient time to review
all contributions. Though core maintainers should not be marked as
maintainers of such plugins. Instead core maintainers should be added
as owners directly on the project where they feel responsible for
maintenance like e.g. core plugins.
Issue: Issue 12805
Change-Id: Ifd8b50c6a0b9bdfc8a26cadac89f2b7669c58af2
diff --git a/tools/plugins.md.template b/tools/plugins.md.template
index 4215de1..dec2018 100644
--- a/tools/plugins.md.template
+++ b/tools/plugins.md.template
@@ -35,6 +35,9 @@
Both the initial (new maintainer) proposal and verdict are shared transparently thus
professionally that way.
+When there are no direct plugins maintainers, the Gerrit maintainers have the permissions
+to review and merge changes. However, regular maintenance of such plugins is not guaranteed.
+
## Plugin builds
There are build [ci-scripts] for most plugins. Each script builds that plugin against all
diff --git a/tools/plugins.py b/tools/plugins.py
index f75d978..c06affb 100644
--- a/tools/plugins.py
+++ b/tools/plugins.py
@@ -299,8 +299,6 @@
# can't read group
pass
names = sorted(list(names))
- if parent == "Public-Plugins":
- names.insert(0, "Core maintainers")
csv = ", ".join(names)
return names, csv
@@ -318,7 +316,7 @@
@staticmethod
def _name_sorter(word):
pattern = re.compile(r"[\W]+")
- return word != "Core maintainers", pattern.sub("", word)
+ return pattern.sub("", word)
def _render_maintainers(self, output):
header = "|Maintainer|Plugins|"
