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.

Sign up for GitHub

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

Jump to bottom

Add support for go.mod as well as go.sum #76

Merged
merged 12 commits into from
Apr 12, 2022
Merged

Add support for go.mod as well as go.sum #76

merged 12 commits into from
Apr 12, 2022

Conversation

jerbob92
Copy link
Contributor

Fixes #75

@jerbob92 jerbob92 mentioned this pull request Jan 13, 2022
Closed
@jerbob92
Copy link
Contributor Author

@rahul2393 @knqyf263 ping

@knqyf263
Copy link
Collaborator

knqyf263 commented Apr 12, 2022 •
edited
Loading

Hi @jerbob92, I refactored a bit. Please let me know if I did something wrong.
The reason for simplifying test cases is a long list makes it hard to distinguish differences.

@knqyf263 knqyf263 changed the title Use go.mod, not go.sum, and test different cases Add support for go.mod as well as go.sum Apr 12, 2022
@knqyf263 knqyf263 merged commit d0501f9 into aquasecurity:main Apr 12, 2022
This was referenced Apr 12, 2022
Merged
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@knqyf263 knqyf263 knqyf263 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Golang mod parser reports version that are not used in application
2 participants
@jerbob92 @knqyf263