git: ceee1def00 - main - articles/committers-guide: Document change in MFH policy
Mateusz Piotrowski
0mp at FreeBSD.org
Tue Apr 20 14:46:05 UTC 2021
On 20/04/2021 15:12, Fernando Apesteguía wrote:
> On Tue, Apr 20, 2021 at 2:39 PM Mateusz Piotrowski <0mp at 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 at FreeBSD.org>
>>>> AuthorDate: 2021-04-20 10:14:58 +0000
>>>> Commit: Mateusz Piotrowski <0mp at 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.
Good point!
Here are some reviews for the hooks if you are interested:
https://reviews.freebsd.org/D29861
https://reviews.freebsd.org/D29860
https://reviews.freebsd.org/D29858
More information about the dev-commits-doc-all
mailing list