-
Notifications
You must be signed in to change notification settings - Fork 95
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
CLA and our policy? #208
Comments
Thanks for raising the question. As the library maintainer, you decide whether to merge a PR without the CLA signed. |
Ahahaha 😂 Thanks for the answer! I think we shouldn't adopt any "blanket policy" then, as Anyways, thank you for the explanation, @asvetlov ! |
Sorry for misleading you, I wrote the message late at night. Anyway, I agree that the CLI harms the project. Thanks to @warsaw comment #207 (comment) I'm convinced that dropping the CLA is a good idea for aio-libs. I apologize for the unconvinience. |
Just weighing (and wading) in on this, after reading all the latest discussion(s) and linked posts and some other related post: I think DCO is plenty sufficient for our needs here. This is not a novel or super fancy project. We're literally implementing the SMTP spec -- that's it. It's not glamorous or full of fancy mathematics or algebra or anything. Heck, there are at least dozens of other software projects that do exactly the same things that aiostmpd does (delivers email). We're not breaking new ground here, we're just making a sweet sweet project that lets people accept and deliver email using Python+asyncio, and no other deps. Which is /valuable/ and (evidently) some of us really like it. I just can't imagine anyone trying to go after aiosmtpd, especially. Once we've successfully implemented the entire SMTP(S) specs, I fully expect this project to just enter a level of maturity where it changes very little, and folks are able to depend on it to build their own projects and platforms. So: 👍 for DCO, 👎 for CLA, and 💯 for aiosmtpd 😍 |
Since it seems the CLA matter has been settled (I see no CLAassistant piping up, and no more license/cla checking), I vote for closing this issue. |
I've switched off the CLA after receiving several negative reactions. |
A bit surprising to see we suddenly have "CLA signing" as one of the "tests".
I have no problem with CLAs, although the abrupt implementation was ... jarring. Was totally caught unaware. (Personally I had no problem with the contents of the CLA and I have signed.)
However, I noticed there had been some resistances/disagreement with the "CLA" concept as a whole. As can be seen here and here.
Furthermore, some previously ready-to-merge PRs now gets the yellow bullet instead of the green checkmark.
Seems to me we're entering a minefield...
So, what will be the policy here?
The text was updated successfully, but these errors were encountered: