Skip to content
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

The infamous CLA check is back #589

Closed
ederag opened this issue Mar 20, 2020 · 8 comments
Closed

The infamous CLA check is back #589

ederag opened this issue Mar 20, 2020 · 8 comments

Comments

@ederag
Copy link
Collaborator

ederag commented Mar 20, 2020

@elbenfreund, it's back
Screenshot_20200320_234148

It's most probably a mistake, as you well know this code is pure GPL,
and can never be re-licensed otherwise.

Please remove it, that scares contributors off,
and increases the risk of someone else putting a CLA on this by mistake.

@ederag
Copy link
Collaborator Author

ederag commented Mar 20, 2020

After 859a89e, the check disappeared from #573.
But it is still there for the other PR.

@matthijskooijman
Copy link
Member

Looking at recent pull requests, I see no CLA checks anymore, so I think this is now properly disabled. I'll close this, we can reopen when this turns out to be false.

@ederag
Copy link
Collaborator Author

ederag commented Apr 28, 2020

A message has to be given in all PR with this wrong check then.
I'll do this, to lift any ambiguity.

@matthijskooijman
Copy link
Member

I'd rather just delete the comments, let me do that.

@matthijskooijman
Copy link
Member

Seems there was just one CLA comment left, I deleted it. I also deleted @ederag's followup comments since they referred already deleted CLA comments, which probably only adds confusion.

@ederag
Copy link
Collaborator Author

ederag commented Apr 28, 2020

More than 15 min later, the checks still appear.
Screenshot_20200428_153039

Don't you dare delete any of my comments again without prior discussion.
That's disrespectful.

@matthijskooijman
Copy link
Member

Ah, I was looking at the CLA assistant comments, but there are indeed still (or again) pending checks. I see this is only the case on some PRs and not new ones, so that probably means that the checks are properly disabled now, but the existing ones still linger. Not sure if they can be removed, maybe @ederag can.

As for deleting comments, sorry for that. It seemed better to delete the CLA comments, and with that also your comment since it would otherwise not make sense. When I found no further CLA comments, I thought you were being overzealous in adding comments everywhere, but it seems I just missed the pending check.

In any case, I reposted some comments, and posted one additional one, so I think all pending checks have not been covered and this can be closed again. Agreed?

@GeraldJansen
Copy link
Contributor

I suppose this can be closed now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants