Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 13 Jan 2026 09:56:18 -0500
From:      John Baldwin <jhb@FreeBSD.org>
To:        Warner Losh <imp@FreeBSD.org>, src-committers@FreeBSD.org, dev-commits-src-all@FreeBSD.org, dev-commits-src-main@FreeBSD.org
Subject:   Re: git: 7f8b4db9e001 - main - github: Add more context to pull requests
Message-ID:  <6e64a14f-1fa0-45ec-b73b-e84f1dbb8153@FreeBSD.org>
In-Reply-To: <6964a4b0.3d7cb.74aa5260@gitrepo.freebsd.org>

index | next in thread | previous in thread | raw e-mail

On 1/12/26 02:37, Warner Losh wrote:
> The branch main has been updated by imp:
> 
> URL: https://cgit.FreeBSD.org/src/commit/?id=7f8b4db9e001d94974a081e5fd58c4f3994429cb
> 
> commit 7f8b4db9e001d94974a081e5fd58c4f3994429cb
> Author:     Warner Losh <imp@FreeBSD.org>
> AuthorDate: 2026-01-10 16:20:20 +0000
> Commit:     Warner Losh <imp@FreeBSD.org>
> CommitDate: 2026-01-12 07:18:11 +0000
> 
>      github: Add more context to pull requests
>      
>      Add code to fetch and decode CODEOWNERS and automatically @ people in
>      the review.
>      
>      Create a new file, .github/path-rules.txt that has a set of paths to
>      match and specific warnings about that part of the tree. We'll use this
>      to try to wave off pull requests in certain areas of the tree, as well
>      as remind people when contrib is involved, etc.
>      
>      Sponsored by:           Netflix
> ---
>   .github/path-rules.txt          |  10 ++++
>   .github/workflows/checklist.yml | 106 ++++++++++++++++++++++++++++++++++++----
>   2 files changed, 106 insertions(+), 10 deletions(-)
> 
> diff --git a/.github/path-rules.txt b/.github/path-rules.txt
> new file mode 100644
> index 000000000000..297cfe8def35
> --- /dev/null
> +++ b/.github/path-rules.txt
> @@ -0,0 +1,10 @@
> +#
> +# Format the similar to CODEOWNERS: Each line has a path, whitespace and a
> +# message for contributors.
> +#
> +sys/contrib/device-tree	:caution: No changes should be made here by pull request
> +# Catch all
> +contrib		:warning: Contributed software usually managed by vendor branch
> +crypto		:warning: Contributed crypto software usually managed by vendor branch
> +sys/contrib	:warning: Contributed software usually managed by vendor branch
> +sys/crypto	:warning: Contributed crypto software usually managed by vendor branch

sys/crypto isn't really the same.  We store the generated OpenSSL assembly files there
for use by ossl(4), but most of sys/crypto is not vendored (e.g. ccp, the non-assembly ossl
bits, aesni are all just bespoke).

-- 
John Baldwin



home | help

Want to link to this message? Use this
URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?6e64a14f-1fa0-45ec-b73b-e84f1dbb8153>