-
Notifications
You must be signed in to change notification settings - Fork 9
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
OpenJS Graduation: Adopt DCO or CLA #86
Comments
To satisfy this is it a matter of either adding a PR template, or signing off on each commit using I think Kepler.gl already has DCO bot implemented. Our publish scripts might need a minor update to include sign off in the commits they make if we went that route, other than that I'm wondering what else needs to change? I'm probably fine with either option, but it does seem like the PR template would break the least amount of things. Just to clarify, is using the |
I am poking around on the same questions. I'll get some clear answers and add them here. |
From CPC call - Jun 4: @bensternthal - taking an approach to understand what Node does then where we can go for recommendations when projects want to adopt or stay with DCO. Will work with legal on this.
|
|
Thanks for this context @ibgreen - it seems like DCO needs to be part of the solution from what I heard today, and @bensternthal raised that it is going to involve some discussions with legal (and likely some legal fees). @bensternthal if there is something I can do to help support this happy to join in - it's an area I would like to learn more about. |
Yep DCO will always be an option. I'm just looking to solidify OpenJS best practices on implementation. |
I am opening this issue in Vis.gl but it is applicable to all the repos underneath the visgl organization.
Part of the OpenJS graduation requirements is to adopt a DCO or CLA. DCO is probably a bit more lightweight, but I would be happy to help implement whichever you choose. If we went with DCO, there is a PROBOT app that we could turn on for the entire org.
Note this might also be as simple as including the DCO in a PR template, I think this is the approach Node.js has taken.
This is not a minor change so let's discuss how best to go about implementing and ensuring any questions or concerns you have are answered.
@chrisgervang for visibility
The text was updated successfully, but these errors were encountered: