chore: document DCO sign-off requirement in CONTRIBUTING.md#1312
Open
KrisF-Midnight wants to merge 6 commits intomainfrom
Open
chore: document DCO sign-off requirement in CONTRIBUTING.md#1312KrisF-Midnight wants to merge 6 commits intomainfrom
KrisF-Midnight wants to merge 6 commits intomainfrom
Conversation
Adds a Developer Certificate of Origin (DCO) section to CONTRIBUTING.md explaining the sign-off requirement, how to sign off with git commit -s, and how to retroactively sign off prior commits. Also adds a DCO sign-off checkbox to the PR template. The DCO GitHub App is now installed on the repo and will block PRs whose commits are missing Signed-off-by trailers. Signed-off-by: Krisztian Feher <krisztian.feher@iohk.io>
gilescope
approved these changes
Apr 13, 2026
Contributors can install a local .git/hooks/prepare-commit-msg hook so that every commit in this repo is signed off automatically, without needing to remember the -s flag each time. Signed-off-by: Krisztian Feher <krisztian.feher@iohk.io>
ozgb
reviewed
Apr 14, 2026
|
|
||
| A DCO GitHub App runs on every pull request and will block merges until all commits are signed off. | ||
|
|
||
| ### Automating sign-off |
Contributor
There was a problem hiding this comment.
We should add some exception here - if you automate sign-off, code-assistant tools (e.g. Claude, Codex) should be disallowed from creating commits
A human needs to sign the commit: https://github.com/torvalds/linux/blob/master/Documentation/process/coding-assistants.rst#signed-off-by-and-developer-certificate-of-origin
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Overview
The organization-wide DCO migration is underway (replacing CLA Assistant). The DCO GitHub App is now installed on this repo and every PR is checked against it, but
CONTRIBUTING.mddoes not yet tell contributors how to comply. This PR documents the sign-off requirement.Changes:
CONTRIBUTING.mdexplaininggit commit -s, amending prior commits withgit rebase --signoff HEAD~N, and links to https://developercertificate.org/.git commit -s..github/pull_request_template.md.Wording mirrors midnightntwrk/compact-tree-sitter#35 for consistency across repos.
🗹 TODO before merging
📌 Submission Checklist
git commit -s) for the DCO🧪 Testing Evidence
Docs-only change. Rendered Markdown inspected locally.
🔱 Fork Strategy
Links