-
Notifications
You must be signed in to change notification settings - Fork 600
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(deps): update anchore dependencies #2388
Conversation
Signed-off-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Fixing tests on this now so we can make sure we can cut syft/grype tomorrow |
Signed-off-by: Christopher Phillips <[email protected]>
Signed-off-by: Christopher Angelo Phillips <[email protected]>
Signed-off-by: Christopher Phillips <[email protected]>
@@ -804,6 +804,12 @@ func TestMatchByImage(t *testing.T) { | |||
|
|||
// The single match in the actual results is the same in ignoredMatched | |||
// but must the details of the VEX matcher appended | |||
if len(vexedResults.Sorted()) < 1 { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Added this check because of the naked [0]
access below which was panic when we got no vexed results as a consequence of the PURL changes.
Signed-off-by: Christopher Phillips <[email protected]>
Signed-off-by: Christopher Phillips <[email protected]>
Signed-off-by: Christopher Phillips <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 👍
Signed-off-by: Christopher Phillips <[email protected]>
Dependencies changed
v0.0.12
➔v0.0.13
(requestedmain
)v1.18.1
➔v1.18.2-0.20250121205042-7f5dbf9872fd
(requestedmain
)