-
-
Notifications
You must be signed in to change notification settings - Fork 71
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
feat: environment licenses as proper SPDX #576
Conversation
Signed-off-by: Jan Kowalleck <[email protected]>
Signed-off-by: Jan Kowalleck <[email protected]>
Signed-off-by: Jan Kowalleck <[email protected]>
Signed-off-by: Jan Kowalleck <[email protected]>
Signed-off-by: Jan Kowalleck <[email protected]>
Signed-off-by: Jan Kowalleck <[email protected]>
Signed-off-by: Jan Kowalleck <[email protected]>
Signed-off-by: Jan Kowalleck <[email protected]>
Signed-off-by: Jan Kowalleck <[email protected]>
I had just a quick looking at the result: https://github.com/andife/cyclonedx-python/actions/runs/6036254421 But I've some question: I do not know, but if we have a valid "id", do we also have to mention the "name"? The explanation for "name": According to the code, I thought |
impossible. according to CycloneDX specification it is EITHER
how comes? cyclonedx-python/cyclonedx_py/_internal/license_trove_classifier.py Lines 82 to 84 in 306e8a8
|
so for py-serializable we know the "Apache 2.0", it's an "id"... there should not be a "name" "Apache Software License"...? For this unique mapping in general we have already the solution => "Apache 2.0"
you are right... maybe one has to ask Github per API... |
where does this solution come from?
why would you ask githhub for a package that you have installed locally? |
followup of #576 Signed-off-by: Jan Kowalleck <[email protected]>
followup of #576 Signed-off-by: Jan Kowalleck <[email protected]>
fixes #459