Skip to content

Commit

Permalink
Merge pull request #1220 from GSA/fix-snyk
Browse files Browse the repository at this point in the history
add explanation
  • Loading branch information
FuhuXia authored Jan 19, 2024
2 parents aac0acd + 40c9181 commit b41b029
Showing 1 changed file with 9 additions and 3 deletions.
12 changes: 9 additions & 3 deletions .github/workflows/snyk.yml
Original file line number Diff line number Diff line change
Expand Up @@ -41,12 +41,18 @@ jobs:
# Update requirements.in with the snyk fix suggestions
python tools/snyk-update.py
# Remove unnecessary file so that it doesn't get added to the repo
rm scan.json
# Update requirements.txt
make update-dependencies
# Check if there are any changes
if [ -z "$(git status --porcelain)" ]; then
echo "Found vulnerable issues but no upgrade or patch available"
cat scan.json | jq '[.vulnerabilities[] | .id] | unique[]'
else
echo "Changes made to add into PR: "
git diff
fi
# Fail so that PR is created
exit 1
- name: Create Pull Request
Expand Down

0 comments on commit b41b029

Please sign in to comment.