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

Submit sbt dependencies to GitHub for vulnerability monitoring #32

Merged
merged 3 commits into from
Jan 7, 2025

Conversation

gu-dependency-graph-integrator[bot]
Copy link
Contributor

@gu-dependency-graph-integrator gu-dependency-graph-integrator bot commented Dec 11, 2024

What does this change?

This PR sends your sbt dependencies to GitHub for vulnerability monitoring via Dependabot. The submitted dependencies will appear in the Dependency Graph on merge to main (it might take a few minutes to update).

What do I need to do?

  • Ensure that the version of sbt in the project is v1.5 or above in order for the dependency submission action to run.
  • A run of this action (Update Dependency Graph for sbt) should have been triggered (see the checks below) when the branch sbt-dependency-graph-a2e99e68b739b516 was created. Sense check the output of the step "Log snapshot for user validation", and make sure that your dependencies look okay.
  • When you are happy the action works, remove the branch name trigger sbt-dependency-graph-a2e99e68b739b516 from the file sbt-dependency-graph.yaml (aka delete line 6), approve this PR, and merge.

Why?

If a repository is in production, we need to track its third party dependencies for vulnerabilities. Historically, we have done this using Snyk, but we are now moving to GitHub’s native Dependabot. Scala is not a language that Dependabot supports out of the box, this workflow is required to make it happen. As a result, we have raised this PR on your behalf to add it to the Dependency Graph.

How has it been verified?

We have tested this workflow, and the process of raising a PR on DevX repos, and have verified that it works. However, we have included some instructions above to help you verify that it works for you. Please do not hesitate to contact DevX Security if you have any questions or concerns.

Further information for sbt

See the sbt workflow documentation for further information and configuration options.

@gu-dependency-graph-integrator gu-dependency-graph-integrator bot requested a review from a team as a code owner December 11, 2024 09:30
Copy link

github-actions bot commented Dec 11, 2024

Test Results

2 tests  ±0   2 ✅ ±0   0s ⏱️ ±0s
2 suites ±0   0 💤 ±0 
2 files   ±0   0 ❌ ±0 

Results for commit c0cf83a. ± Comparison against base commit d79161b.

♻️ This comment has been updated with latest results.

Following the PR checklist, I’m removing the test branch trigger as it’s
no longer needed.
Now that we’re sending the dependencies to github, we no longer need to
send them to snyk.
Copy link
Contributor

@emdash-ie emdash-ie left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I have followed the PR checklist and I’m happy to merge. (But I will merge in the morning during normal work hours.)

@emdash-ie emdash-ie merged commit 94dfb51 into main Jan 7, 2025
2 checks passed
@emdash-ie emdash-ie deleted the sbt-dependency-graph-a2e99e68b739b516 branch January 7, 2025 09:05
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.

1 participant