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

verify the spdx licenses passed in to the license allowlist #729

Merged
merged 3 commits into from
Jan 2, 2024

Conversation

josieang
Copy link
Collaborator

@josieang josieang commented Jan 1, 2024

No description provided.

Copy link
Collaborator Author

@josieang josieang Jan 1, 2024

Choose a reason for hiding this comment

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

the prompt for bard "code for reading spdx expressions from https://github.com/spdx/license-list-data/blob/main/json/licenses.json into a file using go generate"

@codecov-commenter
Copy link

codecov-commenter commented Jan 1, 2024

Codecov Report

Attention: 6 lines in your changes are missing coverage. Please review.

Comparison is base (c698060) 78.70% compared to head (bb837b4) 78.17%.

Files Patch % Lines
cmd/osv-scanner/main.go 25.00% 4 Missing and 2 partials ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #729      +/-   ##
==========================================
- Coverage   78.70%   78.17%   -0.53%     
==========================================
  Files          84       85       +1     
  Lines        6047     6057      +10     
==========================================
- Hits         4759     4735      -24     
- Misses       1085     1117      +32     
- Partials      203      205       +2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link
Collaborator

@another-rex another-rex left a comment

Choose a reason for hiding this comment

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

LGTM! Thanks!

@another-rex
Copy link
Collaborator

Can you add a test for the unrecognized code path?

@josieang josieang enabled auto-merge (squash) January 2, 2024 01:39
@josieang josieang merged commit 16caa1a into google:main Jan 2, 2024
11 checks passed
@josieang josieang deleted the verify-spdx branch January 2, 2024 02:07
@josieang
Copy link
Collaborator Author

josieang commented Jan 2, 2024

Can you add a test for the unrecognized code path?

done!

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