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

Commit

Permalink
Update vuln analysis GHAW to remove on.push hook
Browse files Browse the repository at this point in the history
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
  • Loading branch information
atc0005 committed Jun 8, 2023
1 parent d4f77d4 commit d1ab426
Showing 1 changed file with 0 additions and 3 deletions.
3 changes: 0 additions & 3 deletions .github/workflows/project-analysis.yml
Original file line number Diff line number Diff line change
Expand Up @@ -8,9 +8,6 @@
name: Project Analysis

on:
push:
# branches: [master]

pull_request:
# `synchronized` seems to equate to pushing new commits to a linked branch
# (whether force-pushed or not)
Expand Down

0 comments on commit d1ab426

Please sign in to comment.