summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorAnastasia Klimchuk <aklm@flashrom.org>2023-06-23 21:04:55 +1000
committerAnastasia Klimchuk <aklm@chromium.org>2023-07-19 13:06:22 +0000
commitf17c5069e3227362b3107ad8f6a49f33e5ccda59 (patch)
treeffcfcdf1dc39a543e0b8783a74c8dd866415ee0c
parente0fe2c7ca1b9754480b1cb55a572f7230835c393 (diff)
downloadflashrom-f17c5069e3227362b3107ad8f6a49f33e5ccda59.tar.gz
flashrom-f17c5069e3227362b3107ad8f6a49f33e5ccda59.tar.bz2
flashrom-f17c5069e3227362b3107ad8f6a49f33e5ccda59.zip
doc: Add Team page which describes Gerrit groups
Change-Id: I3118b2b036eab93e901814447543b02c760c6a80 Signed-off-by: Anastasia Klimchuk <aklm@flashrom.org> Reviewed-on: https://review.coreboot.org/c/flashrom/+/76075 Reviewed-by: Nikolai Artemiev <nartemiev@google.com> Tested-by: build bot (Jenkins) <no-reply@coreboot.org> Reviewed-by: Peter Marheine <pmarheine@chromium.org> Reviewed-by: Alexander Goncharov <chat@joursoir.net> Reviewed-by: Stefan Reinauer <stefan.reinauer@coreboot.org>
-rw-r--r--doc/about_flashrom/index.rst4
-rw-r--r--doc/about_flashrom/team.rst61
-rw-r--r--doc/index.rst1
3 files changed, 66 insertions, 0 deletions
diff --git a/doc/about_flashrom/index.rst b/doc/about_flashrom/index.rst
new file mode 100644
index 000000000..11d845f63
--- /dev/null
+++ b/doc/about_flashrom/index.rst
@@ -0,0 +1,4 @@
+.. toctree::
+ :maxdepth: 1
+
+ team
diff --git a/doc/about_flashrom/team.rst b/doc/about_flashrom/team.rst
new file mode 100644
index 000000000..9d002bb66
--- /dev/null
+++ b/doc/about_flashrom/team.rst
@@ -0,0 +1,61 @@
+=========
+Team
+=========
+
+flashrom development process is happening in Gerrit.
+All contributors and users who have a Gerrit account can send patches,
+add comments to patches and vote +1..-1 on patches.
+
+All contributors and users are expected to follow Development guidelines,
+Code of Conduct and Friendliness guidelines.
+
+There are two special groups in Gerrit.
+
+"flashrom reviewers" group
+==========================
+
+Members of the group can do full approval of patches (i.e. vote +2).
+
+In general, members of the group have some area of responsibility in the MAINTAINERS file,
+and are automatically added as reviewers to patches when the patch touches this area.
+
+The responsibilities are the following.
+
+* React to patches when added as a reviewer.
+
+* Try to respond to technical questions on the mailing list if the topic is something you know about
+ and can provide a useful response.
+
+* Know development guidelines and check the patches you are reviewing align with the guidelines.
+
+"flashrom developers" group
+===========================
+
+Members of the group can merge patches.
+The responsibilities for the members of the group are described in more details below.
+
+There is no expectation on how much time you spend on your duties, some non-zero amount of time,
+whatever capacity you have. But in general, you stay around on flashrom.
+
+If you disappear for some time (life happens), especially for a longer time, like several months,
+especially without a warning: you implicitly agree that the others will handle the duties and make decisions if needed
+(potentially without waiting for you to come back, if the decision is needed quickly).
+
+* Merge all contributors's patches (when they are ready), not just your own.
+
+* Be at least vaguely aware what development efforts are ongoing, this helps to make decisions
+ in what order the patches should be merged, and where could be merge conflicts.
+
+* Know development guidelines, and educate other contributors if needed (e.g. give links).
+
+* React to patches when added as a reviewer.
+
+* Try to respond to technical questions on the mailing list if the topic is something you know about
+ and can provide a useful response.
+
+* From time to time show up in real-time channel(s) and/or dev meetings.
+
+* Follow the Code of Conduct and Friendliness guidelines, be a good example for others.
+
+* Bonus point: if you work in a [software] company, educate and help contributors from your company
+ with upstream culture and dev guidelines.
diff --git a/doc/index.rst b/doc/index.rst
index 6a0f8158b..d30815533 100644
--- a/doc/index.rst
+++ b/doc/index.rst
@@ -9,6 +9,7 @@
dev_guide/index
classic_cli_manpage
contact
+ about_flashrom/index
how_to_add_docs
documentation_license