Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: add Snyk monitoring #154

Merged
merged 3 commits into from
Oct 11, 2022
Merged

chore: add Snyk monitoring #154

merged 3 commits into from
Oct 11, 2022

Conversation

jfsoul
Copy link
Contributor

@jfsoul jfsoul commented Sep 23, 2022

What does this change?

This PR integrates the repository with a reusable Snyk (GitHub action) workflow which will scan your code’s dependencies and alert you if vulnerabilities are found. This reusable workflow gives us reliability, control and consistency of how all our repositories are integrated with Snyk.

In particular, reliably integrated means we compare the hash of the last commit on your default branch to the one that Snyk last scanned, and make sure that they match.

This integration has already been tested to make sure scanning will work. Manual updates may also have been applied, which should be squash-merged to keep a tidier history.

If you think that this repository doesn’t belong to your team, please adjust the teams who have admin access in GitHub and talk to the DevX Security team

Feel free to review and merge this PR as a team, or the DevX Security team will do this on your behalf soon.

@jfsoul jfsoul requested review from a team as code owners September 23, 2022 13:19
@jfsoul jfsoul changed the title chore: [TEST] Trigger snyk chore: add Snyk monitoring Sep 28, 2022
@kenoir kenoir merged commit 8648d3c into master Oct 11, 2022
@kenoir kenoir deleted the nori/test-snyk-230922 branch October 11, 2022 08:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants