Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 20 Apr 2021 15:12:18 +0200
From:      =?UTF-8?Q?Fernando_Apestegu=C3=ADa?= <fernando.apesteguia@gmail.com>
To:        Mateusz Piotrowski <0mp@freebsd.org>
Cc:        Mathieu Arnold <mat@freebsd.org>, doc-committers@freebsd.org,  dev-commits-doc-all@freebsd.org
Subject:   Re: git: ceee1def00 - main - articles/committers-guide: Document change in MFH policy
Message-ID:  <CAGwOe2YaUwnv%2BH79pGKendFBk6D=B10eOe8mpRTym8Zfxxb9AQ@mail.gmail.com>
In-Reply-To: <3252cd06-4140-0bc2-e958-983b4a2453c2@FreeBSD.org>
References:  <202104201032.13KAWNFn016258@gitrepo.freebsd.org> <20210420122849.rh5v2kohzvkxs4uu@aching.in.mat.cc> <3252cd06-4140-0bc2-e958-983b4a2453c2@FreeBSD.org>

next in thread | previous in thread | raw e-mail | index | archive | help
On Tue, Apr 20, 2021 at 2:39 PM Mateusz Piotrowski <0mp@freebsd.org> wrote:
>
> On 20/04/2021 14:28, Mathieu Arnold wrote:
> > On Tue, Apr 20, 2021 at 10:32:23AM +0000, Mateusz Piotrowski wrote:
> >> The branch main has been updated by 0mp:
> >>
> >> URL: https://cgit.FreeBSD.org/doc/commit/?id=ceee1def00ac69f06b9d86b2fb1ff04bb6c84d48
> >>
> >> commit ceee1def00ac69f06b9d86b2fb1ff04bb6c84d48
> >> Author:     Mateusz Piotrowski <0mp@FreeBSD.org>
> >> AuthorDate: 2021-04-20 10:14:58 +0000
> >> Commit:     Mateusz Piotrowski <0mp@FreeBSD.org>
> >> CommitDate: 2021-04-20 10:14:58 +0000
> >>
> >>      articles/committers-guide: Document change in MFH policy
> >>
> >>      Merging commits into the quarterly branch no longer requires an explicit
> >>      approval [1]. Let's remove outdated documentation and leave a note
> >>      explaining that there is no need for an explicit approval to MFH.
> > Note that the MFH line should still be used to note intent.
> Hmm, true. It does make sense to leave MFH in the table of metadata fields for this use case. I'll
> bring back the MFH entry with an updated definition.

The template still says:

# MFH:          <Ports tree branch name.  Request approval for merge.>

Maybe something like "State your intention to merge this change" would
be more appropriate.



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CAGwOe2YaUwnv%2BH79pGKendFBk6D=B10eOe8mpRTym8Zfxxb9AQ>