-
-
Notifications
You must be signed in to change notification settings - Fork 129
Add a workflows to test the ORT to ScanCode.io integration #1886
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
base: main
Are you sure you want to change the base?
Conversation
|
This PR is currently blocked due to #1885. |
d90305a to
218fc4f
Compare
218fc4f to
345155d
Compare
Signed-off-by: Thomas Steenbergen <thomas@aboutcode.org>
Signed-off-by: Thomas Steenbergen <thomas@aboutcode.org>
Signed-off-by: Thomas Steenbergen <thomas@aboutcode.org>
Signed-off-by: Thomas Steenbergen <thomas@aboutcode.org>
ba94ab2 to
1cd5ce0
Compare
1cd5ce0 to
2452675
Compare
|
@tdruez Could you help me debug this PR, have been running into issues with ScanCode.io detecting the right number of packages, vulnerabilities and dependencies? |
plus a few adjustments. Signed-off-by: tdruez <tdruez@aboutcode.org>
Signed-off-by: tdruez <tdruez@aboutcode.org>
Not vulnerability data in SPDX output Signed-off-by: tdruez <tdruez@aboutcode.org>
Signed-off-by: tdruez <tdruez@aboutcode.org>
|
Hey @tsteenbe, sure I can help! Changes:
Now there are a couple This is a data issue. ScanCode.io loads what is provided in the SBOM, see https://github.com/oss-review-toolkit/ort/blob/e40ada3cda7ff156607e88499540e82741b3aaee/plugins/reporters/cyclonedx/src/funTest/resources/cyclonedx-reporter-expected-result.json#L365-L386 Nothing to fix here imo.
Let me know about the expected values. |
See individual commits for details.