Re: [gentoo-dev] [PATCH] .github: Add pull request template

[Sam James <sam@gentoo.org>]

Michał Górny <mgorny@gentoo.org> writes:

> Signed-off-by: Michał Górny <mgorny@gentoo.org>
> ---
>  .github/pull_request_template.md | 12 ++++++++++++
>  1 file changed, 12 insertions(+)
>  create mode 100644 .github/pull_request_template.md
>
> The idea is to increase awareness of the AI policy, as well as other
> rules, and to inform users before they submit a PR.
>
> Screenshots @ https://github.com/gentoo/gentoo/pull/36503
>
>
> diff --git a/.github/pull_request_template.md b/.github/pull_request_template.md
> new file mode 100644
> index 000000000000..9e6fe061db11
> --- /dev/null
> +++ b/.github/pull_request_template.md
> @@ -0,0 +1,12 @@
> +<!-- Please put the pull request description above -->
> +
> +---
> +
> +Please check all the boxes that apply:
> +
> +- [ ] I can submit this contribution in agreement with the [Copyright Policy](https://www.gentoo.org/glep/glep-0076.html#certificate-of-origin).
> +- [ ] This contribution has not been created with the assistance of
> Natural Language Processing artificial intelligence tools, in
> accordance with [AI
> policy](https://wiki.gentoo.org/wiki/Project:Council/AI_policy).
> +- [ ] I have certified the above via adding a `Signed-off-by` line to *every* commit in the pull request.
> +- [ ] I have run `pkgcheck scan --commits --net` to check for issues with my commits.
> +
> +Please note that all boxes must be checked for the pull request to be merged.

I'm OK with the proposal as-is, but would be interested in hearing
suggestions to alleviate ulm's concern of developers feeling they must
tick every single box as well. But that might not be doable.

xgqt's comments wrt testing are interesting but maybe better with us
linking to a checklist instead, rather than something users have to
declare in the github PR. Not sure.

Anyway, thanks for this, I've wanted this for a while anyway as it's
more elegant than the Larry bot method. Glad you came around ;)

thanks,
sam