meta: require DCO signoff in commit message guidelines#62510
meta: require DCO signoff in commit message guidelines#62510jasnell wants to merge 2 commits intonodejs:mainfrom
Conversation
|
Review requested:
|
aduh95
left a comment
There was a problem hiding this comment.
I don't know about the multiple author rules, sometimes PRs get abandoned and picked up by someone else, this imply we have no choice but to not attribute the work to the original author – but if "should" means "it is recommended", I guess it's fine
|
That actually gets a bit tricky. If I start work, then abandon it, and you pick it up later... My part of the contribution is still mine and I should sign off on it. I should have signed off from the start. |
|
@nodejs/tsc |
|
Assuming we move forward with this requirement, we should pin an issue in the tracker for a bit calling it out. |
|
@nodejs/tsc ... i'm planning to leave this open for a couple more days in case there are objections / concerns. So far none have been raised. |
Refs: nodejs/core-validate-commit#141