Skip to content

Commit

Permalink
ci: prevent PR merges when analysis is incomplete (#521)
Browse files Browse the repository at this point in the history
This change takes advantage of the new `--fail-incomplete` flag to be
more strict in how the Phylum analysis check is handled. Before this
change it was possible to merge a PR with new/changed dependencies even
before the Phylum analysis was complete. With this change, the status
check will report as a failure, blocking the ability to merge, when the
analysis is incomplete. This provides a degree of strictness to prevent
adding *any* new dependency until it passes established policy.
  • Loading branch information
maxrake authored Jan 2, 2025
1 parent 5317fd9 commit 157d9dc
Showing 1 changed file with 1 addition and 0 deletions.
1 change: 1 addition & 0 deletions .github/workflows/phylum_analyze_pr.yml
Original file line number Diff line number Diff line change
Expand Up @@ -19,3 +19,4 @@ jobs:
uses: phylum-dev/phylum-analyze-pr-action@f428af5c1ee8a705740d51b67424106012740f38 # v2.2.0
with:
phylum_token: ${{ secrets.PHYLUM_TOKEN }}
cmd: phylum-ci -vv --fail-incomplete

0 comments on commit 157d9dc

Please sign in to comment.