Skip to content

Docs: add PR template and AI contribution guidelines #16582

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
clonker wants to merge 2 commits into
base: develop
Choose a base branch
from
+36 −0
Open

Docs: add PR template and AI contribution guidelines #16582

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
4c3af07
Docs: Add AI-assisted contribution guidelines
clonker Apr 8, 2026
e1e400a
Add PR template
clonker Apr 8, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
22 changes: 22 additions & 0 deletions .github/PULL_REQUEST_TEMPLATE.md
View file
Open in desktop
Copy link
Copy Markdown
Collaborator

@cameel cameel Apr 9, 2026 •
edited
Loading

Choose a reason for hiding this comment

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

The new text partially overlaps with our current greeter. We should merge it into this template. Especially the self-review checklist would probably work better as a checkbox here.

Original file line number Diff line number Diff line change
@@ -0,0 +1,22 @@
## Description

<!-- Describe the purpose of this PR. -->

## Checklist
- [ ] I have read the [contributing guidelines](https://docs.soliditylang.org/en/latest/contributing.html)
- [ ] I have disclosed any use of AI tools in creating this PR (see AI Disclosure section below)
- [ ] I have personally reviewed, understood, and tested every change in this PR
Comment on lines +5 to +8
Copy link
Copy Markdown
Collaborator

@cameel cameel Apr 9, 2026

Choose a reason for hiding this comment

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

Perhaps we should add a checkbox acknowledging that the contribution is licensed the same as the repo? That would be much lighter than the DCO someone suggested earlier.


## AI Disclosure
<!--
See our contributing guidelines for the full AI usage policy:
https://docs.soliditylang.org/en/latest/contributing.html#ai-assisted-contributions

If you used AI tools in any part of this PR you MUST disclose it below.
Describe which tool(s) you used and for which parts.
-->

- [ ] No AI tools were used
- [ ] AI tools were used (details below)

<!-- If AI tools were used, describe which tools and for which parts here. -->
14 changes: 14 additions & 0 deletions docs/contributing.rst
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -88,6 +88,20 @@ We thoroughly review every PR and will help you get it right, but there are many

Thank you for your help!

AI-Assisted Contributions
=========================

We do not ban the use of AI tools, but we hold all contributions to the same high standard
regardless of how they were produced, and we require full transparency about their use.

Submitting AI-generated code means you have reviewed it, understand it, can explain it, and have
tested it as thoroughly as if you had written it by hand.

If you used AI tools to generate code, tests, or documentation in any part of your contribution,
disclosure in the pull request is mandatory. If we determine that a PR contains undisclosed
AI-generated content, we may close it.
Comment on lines +101 to +102
Copy link
Copy Markdown
Collaborator

@cameel cameel Apr 9, 2026

Choose a reason for hiding this comment

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

Suggested change
disclosure in the pull request is mandatory. If we determine that a PR contains undisclosed
AI-generated content, we may close it.
disclosure in the pull request is mandatory.
If we determine that a PR contains undisclosed AI-generated content, we may close it.



Running the Compiler Tests
==========================

Expand Down