Skip to content

ci: enforce contributor sign-off (DCO/CLA) + checklist gates - #53

Merged
rvnminers-A-and-N merged 1 commit into
mainfrom
ci/enforce-signoff
Jun 29, 2026
Merged

rvnminers-A-and-N merged 1 commit into
mainfrom
ci/enforce-signoff

Conversation

@rvnminers-A-and-N

@rvnminers-A-and-N rvnminers-A-and-N commented Jun 27, 2026

Copy link
Copy Markdown
Collaborator

Makes the CLA/sign-off an actual check, not just an honor-system line β€” relevant now that more contributors are coming.

Changes

  • dco CI job: on every PR, verifies each non-merge commit base..head carries a Signed-off-by: line; fails with a fix hint (git commit -s --amend / git rebase --signoff) if any is missing. Sign-off certifies the DCO + the project CLA.
  • PR template: the 'signed' item now requires signed AND signed-off commits; new licensing gate β€” all data must be commercial-clean (no restricted/NonCommercial).

To fully enforce

Add "DCO β€” commit sign-off" to branch protection's required status checks (Settings β†’ Branches β†’ main), alongside the existing Python/.NET checks.

- Add a `dco` CI job: on every PR, verify each non-merge commit in the PR carries
  a `Signed-off-by` line; fail with a fix hint if any is missing. Make it a required
  check in branch protection to block unsigned-off merges.
- PR template: require signed AND signed-off commits (sign-off = DCO + CLA), and add
  a licensing gate β€” all data must be commercial-clean (no restricted/NonCommercial).

Signed-off-by: Austin L. <86896075+rvnminers-A-and-N@users.noreply.github.com>
@rvnminers-A-and-N
rvnminers-A-and-N merged commit fad8831 into main Jun 29, 2026
3 checks passed
@rvnminers-A-and-N
rvnminers-A-and-N deleted the ci/enforce-signoff branch June 29, 2026 14:35
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.

1 participant