diff mbox series

[bug#66430,v2] doc: Mention the responsibilities that blocking comes with.

Message ID b68ed47446842ae753ef18a41352695cc2d32472.1697207911.git.maxim.cournoyer@gmail.com
State New
Headers show
Series [bug#66430,v2] doc: Mention the responsibilities that blocking comes with. | expand

Commit Message

Maxim Cournoyer Oct. 13, 2023, 2:38 p.m. UTC
* doc/contributing.texi (Commit Access): Mention that blocking comes with
extra responsibilities.

Reviewed-by: Simon Tournier <zimon.toutoune@gmail.com>
---

Changes in v2:
 - Reword per suggestions

 doc/contributing.texi | 12 +++++++++++-
 1 file changed, 11 insertions(+), 1 deletion(-)


base-commit: 4ec6fd7817ec4073547fd71309374a293d7c436c

Comments

Simon Tournier Oct. 13, 2023, 3:10 p.m. UTC | #1
Hi Maxim,

On Fri, 13 Oct 2023 at 10:38, Maxim Cournoyer <maxim.cournoyer@gmail.com> wrote:

> +                                                                       A
> +participant wishing to block a proposal bears a special responsibility

The term “participant” could appear unclear when reading the complete
section «Commit Access».  Instead, I propose:

    contributor, without or with commit access,

IMHO, it makes clearer what are the expectations on both sides:
reviewer and submitter.

Aside this comment, it is LGTM.  But it is better to wait more
comments. :-)

Cheers,
simon
Maxim Cournoyer Oct. 14, 2023, 1:05 p.m. UTC | #2
Hello,

Simon Tournier <zimon.toutoune@gmail.com> writes:

> Hi Maxim,
>
> On Fri, 13 Oct 2023 at 10:38, Maxim Cournoyer <maxim.cournoyer@gmail.com> wrote:
>
>> +                                                                       A
>> +participant wishing to block a proposal bears a special responsibility
>
> The term “participant” could appear unclear when reading the complete
> section «Commit Access».  Instead, I propose:
>
>     contributor, without or with commit access,
>
> IMHO, it makes clearer what are the expectations on both sides:
> reviewer and submitter.
>
> Aside this comment, it is LGTM.  But it is better to wait more
> comments. :-)

I've reworded it like so:

--8<---------------cut here---------------start------------->8---
modified   doc/contributing.texi
@@ -2036,11 +2036,12 @@ Commit Access
 are committed to finding solutions that everyone can live with.  It
 implies that no decision is made against significant concerns and these
 concerns are actively resolved with proposals that work for everyone.  A
-participant wishing to block a proposal bears a special responsibility
-for finding alternatives, proposing ideas/code or explain the rationale
-for the status quo to resolve the deadlock.  To learn what consensus
-decision making means and understand its finer details, you are
-encouraged to read <https://www.seedsforchange.org.uk/consensus>.
+contributor (which may or may not have commit access), wishing to block
+a proposal bears a special responsibility for finding alternatives,
+proposing ideas/code or explain the rationale for the status quo to
+resolve the deadlock.  To learn what consensus decision making means and
+understand its finer details, you are encouraged to read
+<https://www.seedsforchange.org.uk/consensus>.
 
 The following sections explain how to get commit access, how to be ready
 to push commits, and the policies and community expectations for commits
--8<---------------cut here---------------end--------------->8---

I'll push a v3 shortly and let it sit there to let enough time for
others to chime in if they wish
diff mbox series

Patch

diff --git a/doc/contributing.texi b/doc/contributing.texi
index 864190b119..a18ee00845 100644
--- a/doc/contributing.texi
+++ b/doc/contributing.texi
@@ -2030,7 +2030,17 @@  Commit Access
 consensus and make decisions based on consensus.  To learn what
 consensus decision making means and understand its finer details, you
 are encouraged to read
-@url{https://www.seedsforchange.org.uk/consensus}.
+@url{https://www.seedsforchange.org.uk/consensus}.  It is expected from
+all contributors, and even more so from committers, to help build
+consensus and make decisions based on consensus.  By using consensus, we
+are committed to finding solutions that everyone can live with.  It
+implies that no decision is made against significant concerns and these
+concerns are actively resolved with proposals that work for everyone.  A
+participant wishing to block a proposal bears a special responsibility
+for finding alternatives, proposing ideas/code or explain the rationale
+for the status quo to resolve the deadlock.  To learn what consensus
+decision making means and understand its finer details, you are
+encouraged to read <https://www.seedsforchange.org.uk/consensus>.
 
 The following sections explain how to get commit access, how to be ready
 to push commits, and the policies and community expectations for commits