Skip to content

ci(release): sign off automated release commits#31

Merged
hannasdev merged 1 commit into
mainfrom
ci/fix-release-dco-signoff
May 12, 2026
Merged

ci(release): sign off automated release commits#31
hannasdev merged 1 commit into
mainfrom
ci/fix-release-dco-signoff

Conversation

@hannasdev
Copy link
Copy Markdown
Owner

Summary

  • Configures release-it to add when creating automated release commits.

Problem

The DCO workflow validates commits pushed to , including automated commits created by the release workflow. Those release commits were missing trailers and failed DCO.

Solution

  • Added in so release automation writes a DCO trailer on generated release commits.

Validation

model-switchboard@1.6.5 lint
eslint --max-warnings 0 . passed locally and in pre-commit.

Impact

  • Keeps DCO strict (no exemptions needed).
  • Prevents future release commits from failing DCO on .

Signed-off-by: Hanna Rosengren <4538260+hannasoderstromdev@users.noreply.github.com>
Copilot AI review requested due to automatic review settings May 12, 2026 20:02
Copy link
Copy Markdown

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

This PR updates the release automation configuration so that commits generated by release-it include a DCO Signed-off-by: trailer, preventing the DCO workflow from failing on automated release commits pushed to main.

Changes:

  • Configure release-it git commits to be created with --signoff via commitArgs.

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

@hannasdev hannasdev merged commit addbfd2 into main May 12, 2026
12 checks passed
@hannasdev hannasdev deleted the ci/fix-release-dco-signoff branch May 12, 2026 20:05
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.

2 participants