From 7edb57e594d03b2ab60384a41fe071ffe3f56bdb Mon Sep 17 00:00:00 2001 From: Christoph Wurst Date: Mon, 17 Apr 2023 19:16:21 +0200 Subject: [PATCH] feat(devmanual): Document app maintainer role Signed-off-by: Christoph Wurst --- .../app_publishing_maintenance/index.rst | 3 ++ .../app_publishing_maintenance/maintainer.rst | 33 +++++++++++++++++++ 2 files changed, 36 insertions(+) create mode 100644 developer_manual/app_publishing_maintenance/maintainer.rst diff --git a/developer_manual/app_publishing_maintenance/index.rst b/developer_manual/app_publishing_maintenance/index.rst index 4bb814f0f..c1237b4d6 100644 --- a/developer_manual/app_publishing_maintenance/index.rst +++ b/developer_manual/app_publishing_maintenance/index.rst @@ -2,9 +2,12 @@ App publishing and maintenance ============================== +Transforming source files into a deliverable is documented in the :ref:`app-release-process`. + .. toctree:: :maxdepth: 2 + maintainer release_process Publishing App on the App Store publishing diff --git a/developer_manual/app_publishing_maintenance/maintainer.rst b/developer_manual/app_publishing_maintenance/maintainer.rst new file mode 100644 index 000000000..ce174e572 --- /dev/null +++ b/developer_manual/app_publishing_maintenance/maintainer.rst @@ -0,0 +1,33 @@ +.. _app-maintainer: + +=========== +Maintainers +=========== + +.. sectionauthor:: Christoph Wurst + +The role of a Nextcloud app maintainer is to oversee all processes around an app. The responsibilities are: + +* Triage bug reports + + + Make sure reported bugs are reproducible, ask for more info if necessary + + Check for duplicates + + Close resolved tickets + +* Manage pull requests + + + Ensure pull request receive a review in reasonable time + + Merge pull requests that pass the peer review and all continuous integration checks + + Trigger backports if applicable + +* Manage releases + + + Ensure pre-releases are ready for deployment in pre-production of Nextcloud server + + Ensure compatible releases are ready for deployments of Nextcloud server releases + +* Manage vulnerabilities + + + Monitor vulnerable dependencies, e.g. through Github security alerts, and release updates timely + + Coordinate disclosed vulnerabilities with the Nextcloud security team + +To avoid the so called `bus factor `_, it is highly recommended to have more than one maintainer for each app. Apps can have one primary maintainer and one or more backup maintainers. For transparency, it can make sense to declare the maintainers of an app in the app's repository README file, so that others know whom to contact. \ No newline at end of file