Skip to content

fix(ci): apply DCO enforcement baseline - #232

Merged
Vishnu2707 merged 2 commits into
devfrom
fix/231-dco-cutoff
Aug 5, 2026
Merged

fix(ci): apply DCO enforcement baseline#232
Vishnu2707 merged 2 commits into
devfrom
fix/231-dco-cutoff

Conversation

@TFT444

@TFT444 TFT444 commented Aug 5, 2026

Copy link
Copy Markdown
Collaborator

Summary

Fixes false DCO failures caused by historical commits predating enforcement, especially on squash-based dev to main release pull requests.

  • subtracts the ancestry of the explicit PR governance: enforce DCO and document continuity checks #208 DCO baseline from the PR commit range
  • continues checking new work from branches created before the baseline
  • preserves merge-commit exclusion
  • treats an all-exempt range as successful
  • documents the cutoff and adds real Git-history regression coverage

Validation

Closes #231

Signed-off-by: Tanvir Farhad <tamimtarafder12@gmail.com>
@github-actions

github-actions Bot commented Aug 5, 2026

Copy link
Copy Markdown

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

Scanned Files

None

Signed-off-by: Tanvir Farhad <tamimtarafder12@gmail.com>
@TFT444

TFT444 commented Aug 5, 2026

Copy link
Copy Markdown
Collaborator Author

@Vishnu2707 I found and fixed an additional edge case: legacy commits on an unmerged side branch are now grandfathered by timestamp, while every commit created after DCO was introduced is still enforced. This was validated against the old #227 failure range and PR #198's history; all checks are green. Please review and merge #232 first so I can finish updating #198 without false DCO failures.

@Vishnu2707 Vishnu2707 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Approved.

@Vishnu2707
Vishnu2707 merged commit 2fb51fb into dev Aug 5, 2026
20 checks passed
@Vishnu2707
Vishnu2707 deleted the fix/231-dco-cutoff branch August 5, 2026 13:09
@Vishnu2707

Copy link
Copy Markdown
Member

@TFT444 , you can work on updating #198

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.

fix(ci): enforce DCO only for commits after policy introduction

2 participants