Skip to content

Add pull request template for AI usage disclosure - #1949

Merged
eduardo-rodrigues merged 2 commits into
HSF:mainfrom
preetk25627-dotcom:ai-pr-template-only
Oct 16, 2026
Merged

eduardo-rodrigues merged 2 commits into
HSF:mainfrom
preetk25627-dotcom:ai-pr-template-only

Conversation

@preetk25627-dotcom

@preetk25627-dotcom preetk25627-dotcom commented Jul 30, 2026 •

Copy link
Copy Markdown
Contributor

Summary

This PR adds a pull request template with an optional AI usage disclosure section.

It implements the lightweight approach discussed in #1919 by allowing contributors to indicate whether AI tools were used while preparing the pull request and, if applicable, briefly describe how they were used and what was manually reviewed.

PR intentionally focuses on the pull request template and does not introduce broader AI contribution policy documentation.

Related to #1919 and #1953.

@netlify

netlify Bot commented Jul 30, 2026 •

Copy link
Copy Markdown

✅ Deploy Preview for earnest-hotteok-b1e1bf ready!

Name Link
🔨 Latest commit 9bbd6fa
🔍 Latest deploy log https://app.netlify.com/projects/earnest-hotteok-b1e1bf/deploys/6a6af5f29462290008d41c92
😎 Deploy Preview https://deploy-preview-1949--earnest-hotteok-b1e1bf.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@cantel

cantel commented Jul 31, 2026

Copy link
Copy Markdown
Contributor

Hi @preetk25627-dotcom , thanks a lot for picking up on the issue, studying the conversations and proactively putting forth one of the concrete suggestions. I'm just busy writing a post on thoughts on the issues page, and then we can get back to this.

@cantel

cantel commented Jul 31, 2026 •

Copy link
Copy Markdown
Contributor

Let's see what the general opinion is. Perhaps what people might want to comment on here is whether we just want a tick box or also ask for the additional fields.
In addition why not another tick box: "I certify I have verified and understood the changes this PR implements."
Or, if we have a contributing file stating some basic rules (incl. disclose your AI usage), a tick box to say something like "I acknowledge the basic rules for contributors".

@eduardo-rodrigues

Copy link
Copy Markdown
Member

Hello. Indeed let's wait for others to be back online ...

In any case, the extra tick box @cantel suggests makes sense, and for a request template I would move it to https://github.com/HSF/.github, as suggested in #1919, so that it gets used everywhere.

Comment thread .github/pull_request_template.md Outdated
@eduardo-rodrigues

Copy link
Copy Markdown
Member

Thank you both and thank you @cantel for pushing forward with this.

I was going to make the suggestion that we move this and general PRs for CONTRIBUTING files or templates under the general organisation-wide repo https://github.com/HSF/.github. Apologies if this comes annoyingly late! We could still converge on the final version here, since this is "where it all started", and then close linking to the actual and identical PR, which we merge.

(See similar comment at the related PR #1953 (comment).)

@netlify

netlify Bot commented Oct 16, 2026 •

Copy link
Copy Markdown

✅ Deploy Preview for earnest-hotteok-b1e1bf ready!

Name Link
🔨 Latest commit 859a771
🔍 Latest deploy log https://app.netlify.com/projects/earnest-hotteok-b1e1bf/deploys/6ad21c08349fec75c5bd3f66
😎 Deploy Preview https://deploy-preview-1949--earnest-hotteok-b1e1bf.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@cantel

cantel commented Oct 16, 2026

Copy link
Copy Markdown
Contributor

Thank you both and thank you @cantel for pushing forward with this.

I was going to make the suggestion that we move this and general PRs for CONTRIBUTING files or templates under the general organisation-wide repo https://github.com/HSF/.github. Apologies if this comes annoyingly late! We could still converge on the final version here, since this is "where it all started", and then close linking to the actual and identical PR, which we merge.

(See similar comment at the related PR #1953 (comment).)

Hi @eduardo-rodrigues , I'll let you decide whether to apply it more widely : ) I am ready to submit the whole thing to .github instead, or just here. I consider this and the other PR ready now.

@eduardo-rodrigues

Copy link
Copy Markdown
Member

You made a point that it is best not to immediately impose this template on every repo under the HSF. We can move eventually, if preferred by many. Let's proceed then in here. Thanks.

@eduardo-rodrigues eduardo-rodrigues left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ready to merge also from me. Approving.

@eduardo-rodrigues

Copy link
Copy Markdown
Member

(Just forgot to restate what I suggested above - basically, add the link to the "contributing guidelines" in this template once we have that link. Can do at a later stage.)

@preetk25627-dotcom

Copy link
Copy Markdown
Contributor Author

@cantel @eduardo-rodrigues Thanks for the reviews and for refining the template. Happy to proceed here as agreed.

@eduardo-rodrigues

Copy link
Copy Markdown
Member

Hi. Merging to nicely close the week ("via this PR", a geek would say :D.)

@eduardo-rodrigues
eduardo-rodrigues merged commit c71bb4b into HSF:main Oct 16, 2026
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants