Skip to content
New issue

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

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

Already on GitHub? # to your account

Feat: Add subgraph filter #104

Merged
merged 1 commit into from
Mar 29, 2023
Merged

Conversation

alexandru-dragomir
Copy link
Contributor

@alexandru-dragomir alexandru-dragomir commented Mar 1, 2023

  • Ready for review
  • Follows CONTRIBUTING rules
  • Reviewed by Snyk internal team

What does this PR do?

Adds a new function to filter out packages that are unchanged between two graphs. Allows cheaper calculation of remediation information when only newly introduced issues are relevant.

@alexandru-dragomir alexandru-dragomir force-pushed the fix/add-subgraph-filter branch from 00f6554 to 5b1aaa9 Compare March 3, 2023 16:08
@alexandru-dragomir alexandru-dragomir marked this pull request as ready for review March 3, 2023 16:13
@alexandru-dragomir alexandru-dragomir requested a review from a team as a code owner March 3, 2023 16:13
@darscan
Copy link
Contributor

darscan commented Mar 21, 2023

This is a feat not a fix

@darscan
Copy link
Contributor

darscan commented Mar 21, 2023

Branch name is fine, but please edit the commit prefix

@darscan darscan changed the title Fix/add subgraph filter Feat: Add subgraph filter Mar 24, 2023
darscan
darscan previously approved these changes Mar 29, 2023
@darscan darscan dismissed their stale review March 29, 2023 09:54

Dismiss

This adds a function that creates a subgraph of only new
and changed package nodes in a graph.
@alexandru-dragomir alexandru-dragomir merged commit cf57280 into master Mar 29, 2023
@alexandru-dragomir alexandru-dragomir deleted the fix/add-subgraph-filter branch March 29, 2023 10:45
@snyksec
Copy link

snyksec commented Mar 29, 2023

🎉 This PR is included in version 2.6.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants