From patchwork Wed Jan 1 16:29:45 2020 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: =?utf-8?q?Ludovic_Court=C3=A8s?= X-Patchwork-Id: 19501 Return-Path: X-Original-To: patchwork@mira.cbaines.net Delivered-To: patchwork@mira.cbaines.net Received: by mira.cbaines.net (Postfix, from userid 113) id 56DCD179D5; Wed, 1 Jan 2020 16:31:14 +0000 (GMT) X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on mira.cbaines.net X-Spam-Level: X-Spam-Status: No, score=-1.9 required=5.0 tests=BAYES_00 autolearn=unavailable autolearn_force=no version=3.4.0 Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) by mira.cbaines.net (Postfix) with ESMTP id 02CB7179D3 for ; Wed, 1 Jan 2020 16:31:14 +0000 (GMT) Received: from localhost ([::1]:59782 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1imgtt-0008I4-GN for patchwork@mira.cbaines.net; Wed, 01 Jan 2020 11:31:13 -0500 Received: from eggs.gnu.org ([2001:470:142:3::10]:55891) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1imgtk-0008Hy-Ka for guix-patches@gnu.org; Wed, 01 Jan 2020 11:31:06 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1imgtj-00033E-Iq for guix-patches@gnu.org; Wed, 01 Jan 2020 11:31:04 -0500 Received: from debbugs.gnu.org ([209.51.188.43]:59723) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16) (Exim 4.71) (envelope-from ) id 1imgtj-00032X-Fg; Wed, 01 Jan 2020 11:31:03 -0500 Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2) (envelope-from ) id 1imgti-00023z-Be; Wed, 01 Jan 2020 11:31:02 -0500 X-Loop: help-debbugs@gnu.org Subject: [bug#38846] [PATCH 0/4] Move 'HACKING' to the manual, and a proposal for commit access Resent-From: Ludovic =?utf-8?q?Court=C3=A8s?= Original-Sender: "Debbugs-submit" Resent-CC: guix-maintainers@gnu.org, guix-patches@gnu.org Resent-Date: Wed, 01 Jan 2020 16:31:02 +0000 Resent-Message-ID: Resent-Sender: help-debbugs@gnu.org X-GNU-PR-Message: report 38846 X-GNU-PR-Package: guix-patches X-GNU-PR-Keywords: patch To: 38846@debbugs.gnu.org Cc: Ludovic =?utf-8?q?Court=C3=A8s?= , guix-maintainers@gnu.org X-Debbugs-Original-To: guix-patches@gnu.org X-Debbugs-Original-Xcc: guix-maintainers@gnu.org Received: via spool by submit@debbugs.gnu.org id=B.15778962047763 (code B ref -1); Wed, 01 Jan 2020 16:31:02 +0000 Received: (at submit) by debbugs.gnu.org; 1 Jan 2020 16:30:04 +0000 Received: from localhost ([127.0.0.1]:37461 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1imgsl-000219-Jt for submit@debbugs.gnu.org; Wed, 01 Jan 2020 11:30:03 -0500 Received: from lists.gnu.org ([209.51.188.17]:58102) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1imgsj-00020Z-R9 for submit@debbugs.gnu.org; Wed, 01 Jan 2020 11:30:02 -0500 Received: from eggs.gnu.org ([2001:470:142:3::10]:54493) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1imgsi-0007qe-LB for guix-patches@gnu.org; Wed, 01 Jan 2020 11:30:01 -0500 Received: from fencepost.gnu.org ([2001:470:142:3::e]:38090) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1imgsi-0008Sb-Gz; Wed, 01 Jan 2020 11:30:00 -0500 Received: from [2a01:e0a:1d:7270:af76:b9b:ca24:c465] (port=47364 helo=gnu.org) by fencepost.gnu.org with esmtpsa (TLS1.2:DHE_RSA_AES_256_CBC_SHA1:256) (Exim 4.82) (envelope-from ) id 1imgsh-0007kO-Rp; Wed, 01 Jan 2020 11:30:00 -0500 From: Ludovic =?utf-8?q?Court=C3=A8s?= Date: Wed, 1 Jan 2020 17:29:45 +0100 Message-Id: <20200101162945.4946-1-ludo@gnu.org> X-Mailer: git-send-email 2.24.1 MIME-Version: 1.0 X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.2.x-3.x [generic] X-BeenThere: debbugs-submit@debbugs.gnu.org X-Mailman-Version: 2.1.18 Precedence: list X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.2.x-3.x [generic] X-Received-From: 209.51.188.43 X-BeenThere: guix-patches@gnu.org List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: guix-patches-bounces+patchwork=mira.cbaines.net@gnu.org Sender: "Guix-patches" X-getmail-retrieved-from-mailbox: Patches Hello Guix! Happy new year, merry 12 nivôse, or whatever celebration is appropriate for you! :-) These patches do three things: 1. Move text from ‘HACKING’ to ‘doc/contributing.texi’. 2. Encourage patch review for committers. 3. Add a tentative policy for granting commit access (the last patch of this series). I expect #1 and #2 to be uncontroversial, but I’d like feedback on #3! So far, we’ve been giving commit access in a very ad-hoc fashion. Often it was Ricardo or myself who ended up taking care of that, even though other people have admin rights on Savannah to add/remove members. We briefly discussed it among maintainers after the maintainer collective expanded, and it seems to me that perhaps now is a good time to formalize things a bit—to clarify what contributors may expect and to increase transparency. Hence this proposal of a simple co-optation policy. As you know, Chris Baines has been working towards automated testing of submitted patches. One of the goals is to allow part of the QA to be automated, such that, eventually, approved merges could be automated. In that spirit, we would have an incentive to not add more committers (probably also a good thing security-wise). That’s why I added a note on this topic. What do people think? Thanks, Ludo’. Ludovic Courtès (4): doc: Add "Tracking Bugs and Patches" section. doc: Move "Commit Access" section from 'HACKING' to the manual. doc: Encourage patch review. DRAFT doc: Add a cooption policy for commit access. HACKING | 58 +------------- doc/contributing.texi | 171 ++++++++++++++++++++++++++++++++++++++++-- doc/guix.texi | 2 +- 3 files changed, 168 insertions(+), 63 deletions(-)