Skip to content
This repository has been archived by the owner on Aug 18, 2023. It is now read-only.

Update vuln analysis GHAW to remove on.push hook #250

Merged
merged 1 commit into from
Jun 8, 2023

Conversation

atc0005
Copy link
Owner

@atc0005 atc0005 commented Jun 8, 2023

This hook was previously needed for proper operation of the Vulnerability / CodeQL job so that it could compare before/after changes against the base branch.

This requirement was removed as part of recent work in the github/codeql-action project.

I'm keeping the event type check for jobs already using it for the time being.

refs atc0005/todo#60

This hook was previously needed for proper operation of the
`Vulnerability / CodeQL` job so that it could compare before/after
changes against the base branch.

This requirement was removed as part of recent work in the
github/codeql-action project.

I'm keeping the event type check for jobs already using it for the
time being.

refs atc0005/todo#60
@atc0005 atc0005 added enhancement New feature or request CI labels Jun 8, 2023
@atc0005 atc0005 added this to the Next Release milestone Jun 8, 2023
@atc0005 atc0005 self-assigned this Jun 8, 2023
@atc0005 atc0005 merged commit 45fbdd0 into master Jun 8, 2023
@atc0005 atc0005 deleted the remove-push-event-hook-from-vuln-anaylsis-workflow branch June 8, 2023 23:55
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
CI enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant