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

Update Snyk workflow #74

Merged
merged 1 commit into from
Oct 2, 2023
Merged

Update Snyk workflow #74

merged 1 commit into from
Oct 2, 2023

Conversation

evansims
Copy link
Member

@evansims evansims commented Sep 29, 2023

Changes

This pull request updates the Snyk workflow. It replaces the Snyk action with the Snyk CLI to workaround some issues we've encountered with the Docker container from the Snyk actions not being able to access the configured JVM environment elsewhere in the workflow.

(Note that the CI for this PR will fail until it is merged into the main branch.)

References

N/A

Testing

This pull request updates our continuous integration tests.

Checklist

@evansims evansims added the review:tiny Tiny review label Sep 29, 2023
@evansims evansims temporarily deployed to internal September 29, 2023 06:24 — with GitHub Actions Inactive
@evansims evansims temporarily deployed to internal September 29, 2023 06:24 — with GitHub Actions Inactive
@evansims evansims marked this pull request as ready for review September 29, 2023 06:59
@evansims evansims requested a review from a team as a code owner September 29, 2023 06:59
@poovamraj poovamraj merged commit 4291d23 into master Oct 2, 2023
10 of 11 checks passed
@poovamraj poovamraj deleted the test/update-snyk-workflow branch October 2, 2023 14:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
review:tiny Tiny review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants