search

cncf / techdocs

CNCF TechDocs Team
https://www.cncf.io
Apache License 2.0
37 stars 26 forks source link

Switch to CLA for this repo #244

Closed chalin closed 2 days ago

chalin commented 1 month ago

@nate-double-u - is it easy to setup EasyCLA?

nate-double-u commented 2 weeks ago

Sorry I missed this. Why would you like to shift?

chalin commented 3 days ago

Given that most contributors to this repo do so repeatedly, it we'll be saving time and effort by using CLA. Such contributors, like the techdocs team, have all signed the CLA (once!) and will be able to forget about signing every single commit in every single PR submitted to this repo via the web i/f or otherwise.

Said another way: DCO isn't a good match for this repo IMHO. E.g., GitHub web i/f patch commits don't always work -- I end up having to pull it in locally, signing, and pushing back. That's time wasted.

Same thing for local development, especially for PRs with multiple commits. I often forget to sign, and only realize after having submitted the PR; and then I have to fix the issue retroactively.

nate-double-u commented 2 days ago

Thanks for laying out the argument. I brought it to the CNCF leadership team and they prefer to stay with DCO. I think that all CNCF repos use DCO (tho, I'd have to double check that).