Skip to content

meta: add DCO sign-off check to commit workflow#62473

Open
jasnell wants to merge 1 commit intonodejs:mainfrom
jasnell:jasnell/require-signoff-workflow
Open

meta: add DCO sign-off check to commit workflow#62473
jasnell wants to merge 1 commit intonodejs:mainfrom
jasnell:jasnell/require-signoff-workflow

Conversation

@jasnell
Copy link
Copy Markdown
Member

@jasnell jasnell commented Mar 28, 2026

All commits are technically supposed to be signed off with a DCO sign-off (using Signed-off-by) but we haven't enforced that. We really ought to be.

Signed-off-by: James M Snell jasnell@gmail.com

All commits are technically supposed to be signed off
with a DCO sign-off (using `Signed-off-by`) but we
haven't enforced that. We really ought to be.

Signed-off-by: James M Snell <jasnell@gmail.com>
@jasnell jasnell requested review from mcollina and ruyadorno March 28, 2026 05:25
@nodejs-github-bot
Copy link
Copy Markdown
Collaborator

Review requested:

  • @nodejs/actions

@nodejs-github-bot nodejs-github-bot added the meta Issues and PRs related to the general management of the project. label Mar 28, 2026
@mcollina
Copy link
Copy Markdown
Member

This is best done via the DCO bot.

@jasnell
Copy link
Copy Markdown
Member Author

jasnell commented Mar 28, 2026

Are you referring to https://github.com/apps/dco? If so, works for me.. I opened this solely as a nudge ;-)

@mcollina
Copy link
Copy Markdown
Member

That one!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

meta Issues and PRs related to the general management of the project.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants