Skip to content
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

Closed
1 of 6 tasks
dhmlau opened this issue Jan 25, 2019 · 9 comments
Closed
1 of 6 tasks

Switch contribution method from CLA to DCO #2292

dhmlau opened this issue Jan 25, 2019 · 9 comments
Labels
Internal Tooling Issues related to our tooling and monorepo infrastructore

Comments

@dhmlau
Copy link
Member

dhmlau commented Jan 25, 2019

Description / Steps to reproduce / Feature proposal

We're switching to use Developer Certificate of Origin (DCO) for loopback-next repo, after switched it for the loopback4-example-shopping and loopback.io-search repos.

Acceptance Criteria

Reference

@dhmlau dhmlau added the Internal Tooling Issues related to our tooling and monorepo infrastructore label Jan 25, 2019
@raymondfeng
Copy link
Contributor

@bajtos also mentioned there's a way to configure git to automatically sign. Might want to include that instruction too.

We need to figure out and document it. Asking us to sign-off manually every time is annoying.

@jannyHou
Copy link
Contributor

Would be good if we can fail the commit message check byhusky when the sign-off is missing.

@jannyHou
Copy link
Contributor

Make CI enforce correct license banner + correct DCO sign-off in commit message:
jerryscript-project/jerryscript-debugger-ts#9

@dhmlau
Copy link
Member Author

dhmlau commented Jan 31, 2019

Thanks!
Let me get the final confirmation before proceeding. Thanks!

@stale
Copy link

stale bot commented Jan 27, 2020

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 CODEOWNERS file at the top-level of this repository. This issue will be closed within 30 days of being stale.

@stale stale bot added the stale label Jan 27, 2020
@achrinza achrinza removed the stale label Feb 12, 2020
@raymondfeng
Copy link
Contributor

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.

@dhmlau dhmlau added this to the Mar 2020 milestone Feb 20, 2020
@bajtos
Copy link
Member

bajtos commented Feb 21, 2020

there's a way to configure git to automatically sign.

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 ~/.gitmessage approach.

Further reading:

@dhmlau dhmlau removed this from the Mar 2020 milestone Feb 28, 2020
@dhmlau
Copy link
Member Author

dhmlau commented Aug 19, 2020

  • Instructions for signing and setting alias to sign are captured under: https://loopback.io/doc/en/contrib/code-contrib.html.
  • loopback-next repo (together with loopback4-example-shopping, loopback.io, loopback-datasource-juggler) is using DCO now. Will gradually switch the other connector repos to DCO soon.

@dhmlau dhmlau closed this as completed Aug 19, 2020
@joeytwiddle
Copy link
Contributor

Adding this comment for SEO purposes.

If you see this error while committing:

✖   message must be signed off [signed-off-by]

The solution (shown in the document dhmlau shared above) is to use git commit -s ... to sign the commit message.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Internal Tooling Issues related to our tooling and monorepo infrastructore
Projects
None yet
Development

No branches or pull requests

6 participants