From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mp11.migadu.com ([2001:41d0:8:6d80::]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits)) by ms5.migadu.com with LMTPS id 2KbpGeGUvmNNDgEAbAwnHQ (envelope-from ) for ; Wed, 11 Jan 2023 11:52:17 +0100 Received: from aspmx1.migadu.com ([2001:41d0:8:6d80::]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits)) by mp11.migadu.com with LMTPS id gEP1GeGUvmMmDgEA9RJhRA (envelope-from ) for ; Wed, 11 Jan 2023 11:52:17 +0100 Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by aspmx1.migadu.com (Postfix) with ESMTPS id 1E16828386 for ; Wed, 11 Jan 2023 11:52:17 +0100 (CET) Received: from localhost ([::1] helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1pFYiC-0002mY-LL; Wed, 11 Jan 2023 05:52:04 -0500 Received: from eggs.gnu.org ([2001:470:142:3::10]) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1pFYiA-0002lX-Vj for guix-patches@gnu.org; Wed, 11 Jan 2023 05:52:02 -0500 Received: from debbugs.gnu.org ([209.51.188.43]) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_128_GCM_SHA256:128) (Exim 4.90_1) (envelope-from ) id 1pFYiA-0001TU-8C for guix-patches@gnu.org; Wed, 11 Jan 2023 05:52:02 -0500 Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2) (envelope-from ) id 1pFYi9-000055-QX for guix-patches@gnu.org; Wed, 11 Jan 2023 05:52:01 -0500 Subject: bug#59513: [PATCH v2] doc: contributing: Tweak the Commit Policy. Resent-From: Christopher Baines Original-Sender: "Debbugs-submit" Resent-To: guix-patches@gnu.org Resent-Date: Wed, 11 Jan 2023 10:52:01 +0000 Resent-Message-ID: Resent-Sender: help-debbugs@gnu.org X-GNU-PR-Message: cc-closed 59513 X-GNU-PR-Package: guix-patches X-GNU-PR-Keywords: patch moreinfo To: 59513-done@debbugs.gnu.org Mail-Followup-To: 59513@debbugs.gnu.org, mail@cbaines.net, mail@cbaines.net Received: via spool by 59513-done@debbugs.gnu.org id=D59513.167343429032712 (code D ref 59513); Wed, 11 Jan 2023 10:52:01 +0000 Received: (at 59513-done) by debbugs.gnu.org; 11 Jan 2023 10:51:30 +0000 Received: from localhost ([127.0.0.1]:41770 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1pFYhe-0008VX-0x for submit@debbugs.gnu.org; Wed, 11 Jan 2023 05:51:30 -0500 Received: from mira.cbaines.net ([212.71.252.8]:42080) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1pFYhb-0008VN-Ou for 59513-done@debbugs.gnu.org; Wed, 11 Jan 2023 05:51:28 -0500 Received: from localhost (unknown [IPv6:2a02:8010:68c1:0:54d1:d5d4:280e:f699]) by mira.cbaines.net (Postfix) with ESMTPSA id 4E3F827BBE9 for <59513-done@debbugs.gnu.org>; Wed, 11 Jan 2023 10:51:27 +0000 (GMT) Received: from felis (localhost [127.0.0.1]) by localhost (OpenSMTPD) with ESMTP id 700fc7ef for <59513-done@debbugs.gnu.org>; Wed, 11 Jan 2023 10:51:27 +0000 (UTC) References: <20221123104946.29480-1-mail@cbaines.net> <20221208112051.5019-1-mail@cbaines.net> User-agent: mu4e 1.8.11; emacs 28.2 From: Christopher Baines Date: Wed, 11 Jan 2023 10:50:22 +0000 In-reply-to: <20221208112051.5019-1-mail@cbaines.net> Message-ID: <871qo1fioh.fsf@cbaines.net> MIME-Version: 1.0 Content-Type: multipart/signed; boundary="=-=-="; micalg=pgp-sha512; protocol="application/pgp-signature" X-BeenThere: debbugs-submit@debbugs.gnu.org X-Mailman-Version: 2.1.18 Precedence: list X-BeenThere: guix-patches@gnu.org List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: guix-patches-bounces+larch=yhetil.org@gnu.org Sender: guix-patches-bounces+larch=yhetil.org@gnu.org X-Migadu-Country: US X-Migadu-Flow: FLOW_IN ARC-Seal: i=1; s=key1; d=yhetil.org; t=1673434337; a=rsa-sha256; cv=none; b=bPh1KeTP1kK7Zu+lBcm6V8bGSMd7gdgv+wTlFVcH+u3qLhKaiAfOsNl87oVokTWz6m2gu4 ZsKdwbt7ZkceojxggmIKvNAmc1UTcFf82zVPccs9lF/alfIAWToV2Giq3QlfLBfsv1SgEj TEDALNadT1px59tDJYl//FsNTf5z6S5XDGYz4cEqZeldH6mSR32VzJhyP8D19pd8I4Rsv/ WvSlm1gB7/hgIe+505yNnycLXnewsVd550GtCsMPJ7KIuKrb/9Si9byQmv60bNHne/PTj6 HXY8ftYdOWvbue8a5CZO8eKExSGy8yA2ZeGe6LI88nb8yyfCWLFGiJ6jMr7BOg== ARC-Authentication-Results: i=1; aspmx1.migadu.com; dkim=none; spf=pass (aspmx1.migadu.com: domain of "guix-patches-bounces+larch=yhetil.org@gnu.org" designates 209.51.188.17 as permitted sender) smtp.mailfrom="guix-patches-bounces+larch=yhetil.org@gnu.org"; dmarc=none ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=yhetil.org; s=key1; t=1673434337; h=from:from:sender:sender:reply-to:subject:subject:date:date: message-id:message-id:to:to:cc:mime-version:mime-version: content-type:content-type:resent-to:resent-from:resent-sender: resent-message-id:in-reply-to:in-reply-to:references:references: list-id:list-help:list-unsubscribe:list-subscribe:list-post; bh=gEaCDQnKhELen4Azqfb0dbhhy0DuYoGxNkUg3Pelouw=; b=Ic9cF6keugth8LbwXRmpkvuHYEL7/dQ4Y1+eItQrq87sykjDAhSOtB7ddnwop11j/86rxJ ZfULlRFgt18Q5E08jeti1yI16euw5HNw6M3WB4o3g9RNjdRdUdVn/eYoLNO6bFLSjMAYvS 1vpklvLwY5yxI3QKNatO+jxKQjNRqQAse7nlJXhQV5C3AVTJ7/OIzEdu2gAeDvSDxMdV+h PCT5PRxRAll+PAaejCZ26s9npo++x9ppmtJqrf6VL1T34UQCBFRB6FoV1xEl5jqUpx/8oL YHTTyD/DcYT6mLQyL6qLlEVAwZD0Q2PSRlIhnSZJxbss6gJ/IXZ9rkmAKYvfjA== X-Migadu-Spam-Score: -3.69 X-Spam-Score: -3.69 X-Migadu-Queue-Id: 1E16828386 X-Migadu-Scanner: scn1.migadu.com Authentication-Results: aspmx1.migadu.com; dkim=none; spf=pass (aspmx1.migadu.com: domain of "guix-patches-bounces+larch=yhetil.org@gnu.org" designates 209.51.188.17 as permitted sender) smtp.mailfrom="guix-patches-bounces+larch=yhetil.org@gnu.org"; dmarc=none X-TUID: fFlvxpi7iTnT --=-=-= Content-Type: text/plain Christopher Baines writes: > Add more examples of when it can be appropriate to push changes without > review, as I think this can be appropriate in the case of trivial changes (as > mentioned before), but also non-trivial fixes. > > No longer suggest pushing simple new packages or package upgrades (that don't > cause lots of rebuilds) without sending to guix-patches. Now there's some > automation for testing changes sent to guix-patches, sending changes there > before pushing can mean that more rigerious testing takes place and help speed > up substitutes becoming available. This is true, even if no human review takes > place. > > Only suggest waiting one week for review for simpler changes, wait two weeks > for more significant changes. > > Also, reorder some of the information in this section so it's grouped together > better. > > * doc/contributing.texi (Commit Policy): Tweak. > --- > doc/contributing.texi | 41 ++++++++++++++++++----------------------- > 1 file changed, 18 insertions(+), 23 deletions(-) I've now pushed this to master as 9aa2b7419854306b7ae78d4c4f7684316b834b1d, with some final tweaks. Thanks everyone for taking a look. Chris --=-=-= Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iQKlBAEBCgCPFiEEPonu50WOcg2XVOCyXiijOwuE9XcFAmO+lK5fFIAAAAAALgAo aXNzdWVyLWZwckBub3RhdGlvbnMub3BlbnBncC5maWZ0aGhvcnNlbWFuLm5ldDNF ODlFRUU3NDU4RTcyMEQ5NzU0RTBCMjVFMjhBMzNCMEI4NEY1NzcRHG1haWxAY2Jh aW5lcy5uZXQACgkQXiijOwuE9Xfh+xAApT0J3y8ugoDeqQnidRNLYmCw81TqhaYq rvlpJvjmqTOghmqZORlHUKtbGkdIiM6Gz9WSylrwvp8KgUfbHawu03DFveIlQB25 2/kzuYv+RtmtELBV6+PGC9grCoKdmI73bNwdFV51xBEPZK2GeUTkV05zzYrelt9u 8WD/byUT1Qo1N9PXFm2oxvbUfitQGoreTtAyCRa9Sc8N5ekbTk5tX0keNGrk8zCY umAh4PAkynYkwQUwukAqL2mE1rU11+OULtIIDSkFxfy/Pnkji3gY88jQh+5NTuFd XNBv/R4MM+Tgsa0oZ/ZhzGFkbeI/XdBHOLUMwfhl98Ij7Pj2/5xZOp+M5RK/ezBo nIvjEZyo2O5sLzir72yVXFGqFeciJmO50MwJcI5VdAcRtXCXdv6aBdkY7yizdoKg JTtLKlXr78caBxiLzSUDjnBwM6tdcxQQYf6achkAdX35RYNNOunUU/RnUuFXqPMu XT0nLLT2y8Dte2w3WP58AoDO6UOL6/D8zJlOkgVjyVkl0L5jPRhoKtNIofAE+HQn L3FaPew1WYx3ZxPz7qimPBkG1rP0HV3/BTr2IgXeJLMwGj/5GNBFnXVMfg8wnygz /aF/ALmFvooWXW8sMX40M+ftR6o8p75HjmUwg3ILi/si+gH7I4GhGTuPtqC/jtL7 QDl7ruKoF5M= =k/e7 -----END PGP SIGNATURE----- --=-=-=--