Skip to content

docs: add pull request template - #41

Merged
Maxerns merged 1 commit into
getsotto:mainfrom
Rayan-and-beyond:add-pr-template
Sep 17, 2026
Merged

Maxerns merged 1 commit into
getsotto:mainfrom
Rayan-and-beyond:add-pr-template

Conversation

@Rayan-and-beyond

Copy link
Copy Markdown
Contributor

Summary

Adds .github/pull_request_template.md so GitHub's PR form reminds contributors to sign off commits.

Closes #14

Test plan

docs only

  • I signed off every commit (git commit -s / Developer Certificate of Origin)

Notes

Follows the exact spec from the issue: one-line summary, Closes #, a short test plan section, and a DCO sign-off checkbox, in that order. No CI check, DCO app, or CODEOWNERS added — explicitly out of scope per the issue.

@Maxerns Maxerns left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks @Rayan-and-beyond this matches #14 exactly.

The PR has the summary, Closes #, test plan, and the DCO checkbox copied verbatim, with nothing extra added. CI is fully green too.

Only blocker is the missing sign-off on the commit. Please run:

git commit --amend -s
git push --force-with-lease

Once that’s done, I’ll merge.

Signed-off-by: Rayan-and-beyond <263488867+Rayan-and-beyond@users.noreply.github.com>
@Rayan-and-beyond

Copy link
Copy Markdown
Contributor Author

Updated the commit sign-off and force-pushed the branch. Thanks for the review.

@Maxerns
Maxerns merged commit 7d2462f into getsotto:main Sep 17, 2026
31 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add a pull-request template that reminds contributors to sign off (DCO)

2 participants