On Fri, 31 Aug 2018 at 09:36 Mariatta Wijaya <mariatta.wijaya@gmail.com> wrote:

It is possible to set the minimum number of lines changed required to 
trigger a CLA check.  There's also a minimum number of files, although 
that seems less useful.


I find it is tricky with CPython. one line change in any .rst file, perhaps trivial.
One line change in *.py or *.c file, might not be trivial and require CLA, issue number, news entry and so on.
I guess if we do it this way, we need better guidelines of what requires CLA and what's not.

To see this in different perspective, if someone wants to propose a trivial change and then refused to sign the CLA, I think it won't be hard for us to find another contributor who has signed the CLA to make that change instead.
So I'm inclined to make this a requirement.

If you chose to use that, I would prefer to use our own instance to
manage the database ourself, especially make backup.
It would be bad for a legal point of view if suddently
https://cla-assistant.io/ instance goes away and loose all its data.
 
I've pinged Ernest about this. He said it is reasonable for The PSF to host an instance of cla-assistant. And he will look into it.

I also wonder though, instead of hosting it ourselves, can't we just keep daily backups of the signed CLA? It's basically a list of GitHub usernames?
 
Perhaps that would be an easier task than hosting and maintaining it.

I think the key question is what sort of resiliency would we have in making this potential change? If we can backup the data regularly so that if we have to quickly turn around and either stand up our own instance of cla-assistant or tweak our CLA bot then I would assume this would take care of the biggest concerns people have. Basically we need to have a plan in place if the hosted cla-assistant disappeared today without notice.

-Brett
 
 

Mariatta


On Fri, Aug 31, 2018 at 8:26 AM Elvis Pranskevichus <elvis@magic.io> wrote:
On Wednesday, August 29, 2018 6:14:52 PM EDT Mariatta Wijaya wrote:
> - Since the status check will be made required, it means every
> contribution no matter how trivial, requires CLA. Without it, we
> can't merge the pull request. (Maybe only the admins can still
> merge). Sounds like this is a good thing anyway (for Python)

It is possible to set the minimum number of lines changed required to
trigger a CLA check.  There's also a minimum number of files, although
that seems less useful.

                                 Elvis


_______________________________________________
core-workflow mailing list -- core-workflow@python.org
To unsubscribe send an email to core-workflow-leave@python.org
https://mail.python.org/mm3/mailman3/lists/core-workflow.python.org/
This list is governed by the PSF Code of Conduct: https://www.python.org/psf/codeofconduct