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

http headers comment #1083

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

randomizedcoder
Copy link

Additional comments to help with issue: #726

Copy link

google-cla bot commented Dec 17, 2023

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

@brianquinlan
Copy link
Collaborator

Could you sign the contributer license agreement? https://cla.developers.google.com/.

@randomizedcoder
Copy link
Author

randomizedcoder commented Jan 4, 2024 via email

@brianquinlan
Copy link
Collaborator

Thanks Brian. I already did. Does it need "signing" per pull request?

No but I re-ran the presubmit and still see this error. Could you try again?

@randomizedcoder
Copy link
Author

randomizedcoder commented Jan 4, 2024 via email

@randomizedcoder
Copy link
Author

randomizedcoder commented Jan 4, 2024 via email

@brianquinlan
Copy link
Collaborator

I think that the problem is that there are two contributors to this PR:
PR Opener: @randomizedcoder
Author: <se***n​@siden.io>

The se***n​@siden.io address has not signed the CLA.

@randomizedcoder
Copy link
Author

randomizedcoder commented Jan 4, 2024 via email

@brianquinlan
Copy link
Collaborator

The CLA check is still failing :-(

@randomizedcoder
Copy link
Author

I don't know mate. It's definitely signed.

I really don't care about attribution or anything, I was just trying to help some other person in future.

Maybe it would take you a couple of moments to just add this comment next time your pushing a change?

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

Successfully merging this pull request may close these issues.

2 participants