Skip to content

Conversation

@thc202
Copy link
Member

@thc202 thc202 commented Dec 18, 2025

Rework the task hierarchy to not have unnecessary properties in the Snap release task.

@psiinon
Copy link
Member

psiinon commented Dec 18, 2025

Logo
Checkmarx One – Scan Summary & Detailsf11e062b-2fcc-4cd5-bf13-5a7f696a96d8

Great job! No new security vulnerabilities introduced in this pull request


Use @Checkmarx to reach out to us for assistance.

Just send a PR comment with @Checkmarx followed by a natural language request.

Examples: @Checkmarx how are you able to help me? @Checkmarx rescan this PR

@kingthorin
Copy link
Member

kingthorin commented Dec 19, 2025

I can't do a line comment right now for some reason, but the new file is dated 2020 in the license header

buildSrc/src/main/java/org/zaproxy/gradle/MainReleaseRepositoryDispatch.java

@thc202
Copy link
Member Author

thc202 commented Dec 19, 2025

That's exactly the same code that was written in 2020 except the different class name, but I can change it if it matters much.

Rework the task hierarchy to not have unnecessary properties in the
Snap release task.

Signed-off-by: thc202 <[email protected]>
@kingthorin
Copy link
Member

Sorry I hadn't caught on that it was simply relocated, it was late.

Thanks for tackling this.

@kingthorin kingthorin merged commit 0bf306c into zaproxy:master Dec 19, 2025
5 checks passed
@thc202 thc202 deleted the fix-rel-tasks branch December 19, 2025 10:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants