Skip to content

Sign CLA#5

Closed
ritorhymes wants to merge 1 commit intozaproxy:mainfrom
ritorhymes:main
Closed

Sign CLA#5
ritorhymes wants to merge 1 commit intozaproxy:mainfrom
ritorhymes:main

Conversation

@ritorhymes
Copy link
Copy Markdown

I have read the CLA Document and I hereby sign the CLA

@kingthorin
Copy link
Copy Markdown
Member

For the website PRs the DCO on the commits is enough.

@kingthorin kingthorin closed this Sep 18, 2025
@ritorhymes
Copy link
Copy Markdown
Author

We can consider it preemptive in case I make other contributions in the future elsewhere

@thc202
Copy link
Copy Markdown
Member

thc202 commented Sep 18, 2025

I'm not sure there's the need but If you want you should to it properly, see cla.json.

@ritorhymes
Copy link
Copy Markdown
Author

I'm not sure what that means. I followed the instructions in this repo's README.

@thc202
Copy link
Copy Markdown
Member

thc202 commented Sep 18, 2025

That's wrong, we don't have the CLA enabled here but we can do I guess.

@ritorhymes
Copy link
Copy Markdown
Author

It seems to be correct. Bottom of the README here says:

"If you would like to agree to the ZAP CLA but do not have a PR ready to submit to the ZAP repos then you can submit a dummy one to this repo - we don't need the PR to be merged so can then just close it."

That is what I did, using the verbatim text for agreement in the PR body, as would be used elsewhere to agree to the CLA.

@ritorhymes
Copy link
Copy Markdown
Author

I appear to be the only one who's done that so far tho lol

@kingthorin kingthorin mentioned this pull request Sep 18, 2025
@thc202 thc202 reopened this Jan 17, 2026
@ritorhymes
Copy link
Copy Markdown
Author

@thc202 Why is this reopened?

@thc202
Copy link
Copy Markdown
Member

thc202 commented Jan 17, 2026

Don't you want to sign the CLA?

@ritorhymes
Copy link
Copy Markdown
Author

I have read the CLA Document and I hereby sign the CLA

I thought posting this initial message was the required maneuver to do so, what else do I need to do?

@ritorhymes
Copy link
Copy Markdown
Author

no problem, lmk

@thc202 thc202 closed this Jan 23, 2026
@thc202 thc202 reopened this Jan 23, 2026
@github-actions github-actions bot locked and limited conversation to collaborators Jan 23, 2026
@zaproxy zaproxy unlocked this conversation Jan 23, 2026
@thc202
Copy link
Copy Markdown
Member

thc202 commented Jan 23, 2026

Can you try push again?

Signed-off-by: Rito <83614463+ritorhymes@users.noreply.github.com>
@github-actions
Copy link
Copy Markdown

github-actions bot commented Jan 23, 2026

All contributors have signed the CLA ✍️ ✅
Posted by the CLA Assistant Lite bot.

@ritorhymes
Copy link
Copy Markdown
Author

I have read the CLA Document and I hereby sign the CLA

@kingthorin
Copy link
Copy Markdown
Member

kingthorin commented Jan 23, 2026

Thanks
image

@kingthorin kingthorin closed this Jan 23, 2026
@github-actions github-actions bot locked and limited conversation to collaborators Jan 23, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants