-
Notifications
You must be signed in to change notification settings - Fork 358
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
base: master
Are you sure you want to change the base?
http headers comment #1083
Conversation
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. |
Could you sign the contributer license agreement? https://cla.developers.google.com/. |
Thanks Brian. I already did. Does it need "signing" per pull request?
…On Wed, Jan 3, 2024, 18:13 Brian Quinlan ***@***.***> wrote:
Could you sign the contributer license agreement?
https://cla.developers.google.com/.
—
Reply to this email directly, view it on GitHub
<#1083 (comment)>, or
unsubscribe
<https://github.com/notifications/unsubscribe-auth/APMCHTW47SUKDDIKOIISEIDYMYF6DAVCNFSM6AAAAABAYVNZFCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNZWGIYTKNRRGE>
.
You are receiving this because you authored the thread.Message ID:
***@***.***>
|
No but I re-ran the presubmit and still see this error. Could you try again? |
It thinks I have signed
[image: image.png]
But gives an error when I click I agree
[image: image.png]
…On Wed, Jan 3, 2024 at 9:30 PM Brian Quinlan ***@***.***> wrote:
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?
—
Reply to this email directly, view it on GitHub
<#1083 (comment)>, or
unsubscribe
<https://github.com/notifications/unsubscribe-auth/APMCHTWLGWFL6QVPDOSD7E3YMY47TAVCNFSM6AAAAABAYVNZFCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNZWGM2DGOJXGM>
.
You are receiving this because you authored the thread.Message ID:
***@***.***>
--
Regards,
Dave Seddon
+1 415 857 5102
|
Although maybe it did work, cos the "Date Signed" seems to be updated.
[image: image.png]
…On Wed, Jan 3, 2024 at 9:43 PM dave seddon ***@***.***> wrote:
It thinks I have signed
[image: image.png]
But gives an error when I click I agree
[image: image.png]
On Wed, Jan 3, 2024 at 9:30 PM Brian Quinlan ***@***.***>
wrote:
> 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?
>
> —
> Reply to this email directly, view it on GitHub
> <#1083 (comment)>,
> or unsubscribe
> <https://github.com/notifications/unsubscribe-auth/APMCHTWLGWFL6QVPDOSD7E3YMY47TAVCNFSM6AAAAABAYVNZFCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNZWGM2DGOJXGM>
> .
> You are receiving this because you authored the thread.Message ID:
> ***@***.***>
>
--
Regards,
Dave Seddon
+1 415 857 5102
--
Regards,
Dave Seddon
+1 415 857 5102
|
I think that the problem is that there are two contributors to this PR: The se***n@siden.io address has not signed the CLA. |
Ohhh. Maybe my git config is my work email. Sorry. Here you go.
[image: image.png]
…On Thu, Jan 4, 2024 at 8:36 AM Brian Quinlan ***@***.***> wrote:
I think that the problem is that there are two contributors to this PR:
PR Opener: @randomizedcoder <https://github.com/randomizedcoder>
Author: ***@***.***>
The ***@***.*** address has not signed the CLA.
—
Reply to this email directly, view it on GitHub
<#1083 (comment)>, or
unsubscribe
<https://github.com/notifications/unsubscribe-auth/APMCHTWFNDO3XE5SMEMW2GDYM3LBRAVCNFSM6AAAAABAYVNZFCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNZXGQYTAMRWGE>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
--
Regards,
Dave Seddon
+1 415 857 5102
|
The CLA check is still failing :-( |
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? |
Additional comments to help with issue: #726