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

fix(julia): fix parsing Julia v1.0 Manifests #292

Closed

Conversation

Octogonapus
Copy link
Contributor

Julia v1.0 manifests are now parsed correctly.

This fixes the problem raised in aquasecurity/trivy#5635

@Octogonapus Octogonapus changed the title Fix parsing Julia v1.0 Manifests fix(julia): fix parsing Julia v1.0 Manifests Feb 9, 2024
pkg/julia/manifest/parse.go Outdated Show resolved Hide resolved
pkg/julia/manifest/parse_testcase.go Outdated Show resolved Hide resolved
Copy link
Collaborator

@DmitriyLewen DmitriyLewen left a comment

Choose a reason for hiding this comment

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

@Octogonapus Thanks!

LGTM.
@knqyf263 take a look, when you have time, please.

@knqyf263
Copy link
Collaborator

knqyf263 commented Feb 13, 2024

Can we directly apply this fix to Trivy after this PR gets merged? I want to freeze go-dep-parser now.

@Octogonapus
Copy link
Contributor Author

Indeed I could merge it into my Trivy PR if that's what you want.

@knqyf263
Copy link
Collaborator

Yes,I'll let you know when we merge aquasecurity/trivy#6094.

@Octogonapus
Copy link
Contributor Author

This has been merged into aquasecurity/trivy#5635

@Octogonapus Octogonapus deleted the fix_julia_v1.0_parsing branch February 20, 2024 15:01
@knqyf263
Copy link
Collaborator

Thanks!

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.

3 participants