-
Notifications
You must be signed in to change notification settings - Fork 1.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Switch contribution method from CLA to DCO #2292
Comments
We need to figure out and document it. Asking us to sign-off manually every time is annoying. |
Would be good if we can fail the commit message check by |
Make CI enforce correct license banner + correct DCO sign-off in commit message: |
Thanks! |
This issue has been marked stale because it has not seen activity within six months. If you believe this to be in error, please contact one of the code owners, listed in the |
FYI: We use commitlint with Husky and commitlint has https://github.com/conventional-changelog/commitlint/blob/master/docs/reference-rules.md#signed-off-by that we can use to check DCO. |
There are multiple ways how to achieve that, see https://jjasghar.github.io/blog/2016/10/04/signing-commits-in-git for a good overview. I am personally using the Further reading:
|
|
Adding this comment for SEO purposes. If you see this error while committing:
The solution (shown in the document dhmlau shared above) is to use |
Description / Steps to reproduce / Feature proposal
We're switching to use Developer Certificate of Origin (DCO) for
loopback-next
repo, after switched it for theloopback4-example-shopping
andloopback.io-search
repos.Acceptance Criteria
husky
to check the sign-off, so that it will fail right away without signing instead of waiting for CI to fail.loopback-next
loopback-next
repoReference
The text was updated successfully, but these errors were encountered: