Skip to content

build: establish DCO sign-off requirement for all contributions#1914

Merged
pjbgf merged 1 commit into
go-git:mainfrom
pjbgf:dco
Apr 25, 2026
Merged

build: establish DCO sign-off requirement for all contributions#1914
pjbgf merged 1 commit into
go-git:mainfrom
pjbgf:dco

Conversation

@pjbgf
Copy link
Copy Markdown
Member

@pjbgf pjbgf commented Mar 20, 2026

Introduce a Developer Certificate of Origin (DCO) check into the PR validation workflow and document the process in CONTRIBUTING.md.

Open source projects depend on clear provenance for every line of code they accept. Without it, the project and its users are exposed to potential IP disputes — a contributor could later claim their employer owned the code, or that it was derived from an incompatible source. The DCO (https://developercertificate.org/) is the lightweight, widely-adopted answer to this: by adding a Signed-off-by trailer to each commit, contributors certify in writing that they have the right to submit the work under the project's license (Apache 2.0).

This is particularly important as go-git grows in adoption. Downstream users — especially those in regulated industries or with strict supply chain policies — need confidence that the project has proper IP hygiene. Many ecosystems (Linux kernel, CNCF projects) already require this as a baseline.

Copilot AI review requested due to automatic review settings March 20, 2026 06:49
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds a Developer Certificate of Origin (DCO) sign-off requirement to strengthen contribution provenance and IP hygiene, enforced via PR validation and documented for contributors.

Changes:

  • Document the DCO sign-off requirement and how to apply it in CONTRIBUTING.md.
  • Add a GitHub Actions job to fail PR validation when any PR commit is missing a Signed-off-by: trailer.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
CONTRIBUTING.md Documents the DCO requirement and provides contributor instructions for signing off commits.
.github/workflows/pr-validation.yml Adds a PR CI job that checks each PR commit message for a Signed-off-by: line.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread .github/workflows/pr-validation.yml
Comment thread .github/workflows/pr-validation.yml Outdated
Comment thread CONTRIBUTING.md Outdated
@pjbgf pjbgf force-pushed the dco branch 2 times, most recently from 3fb68a7 to 646bbe7 Compare March 20, 2026 07:22
@pjbgf pjbgf requested a review from mcuadros March 20, 2026 07:23
Introduce a Developer Certificate of Origin (DCO) check into the PR
validation workflow and document the process in CONTRIBUTING.md.

Open source projects depend on clear provenance for every line of code
they accept. Without it, the project and its users are exposed to
potential IP disputes — a contributor could later claim their employer
owned the code, or that it was derived from an incompatible source.
The DCO (https://developercertificate.org/) is the lightweight,
widely-adopted answer to this: by adding a Signed-off-by trailer to
each commit, contributors certify in writing that they have the right
to submit the work under the project's license (Apache 2.0).

This is particularly important as go-git grows in adoption. Downstream
users — especially those in regulated industries or with strict supply
chain policies — need confidence that the project has proper IP hygiene.
Many ecosystems (Linux kernel, CNCF projects) already require this as a
baseline.

Signed-off-by: Paulo Gomes <[email protected]>
Assisted-By: Claude Sonnet 4.6 <[email protected]>
@pjbgf pjbgf merged commit 4128bfe into go-git:main Apr 25, 2026
16 checks passed
@pjbgf pjbgf deleted the dco branch April 25, 2026 20:46
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.

3 participants