-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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
Find an easy way to accept contributions #443
Comments
After reading up on the CLA, it seems to me that it is worse than the DCO. With the DCO we have only technical issues with sign-offs, the CLA brings in legal barriers, e.g. from here:
Node.js solves the sign-off issue with distinguishing between contributors and collaborators. Basically, a contributor is anybody who can open a PR, a collaborator is one who reviews, sign-offs and merges PRs: https://nodejs.org/en/get-involved/development/#landing-pull-requests. There are no special requirements to a contributor's commits like it should be signed off, the burden is put on the collaborator, e.g. nodejs/node@b811464. If a contributor wants to become a collaborator he should learn how to sign off commits for sure. Actually, after landing a nontrivial PR, a contributor automatically becomes a collaborator and gets the write access to the Node.js repo. With this PR one of us could review it, rebase with sign off and merge. It sounds like the very easy way to get new contributions in and grows contributors base. |
I did a bit more research and found out that GitHub has the default contribution agreement for all users:
This default does not work for us though because we override it with the DCO:
@svenefftinge @marcdumais Maybe we just should drop the DCO and stick to the GitHub default agreement? GitHub Terms of Service: https://help.github.com/articles/github-terms-of-service/#6-contributions-under-repository-license. |
Great finding, thank you! Sounds like that's by far the best option. |
Let me check on our side if this looks acceptable and get back to you. |
We just require PRs to have signed-off commits now so maybe this issue can be closed? |
This might be moot in any case - when we move to the Eclipse Foundation, they have their own standards about how to accept contributions. |
|
It should present the Certificate of Origin and have a Sign here, and keep a db of email -> signed that we can control and export.
The text was updated successfully, but these errors were encountered: