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 some cdx:python properties #96

Merged
merged 5 commits into from
Aug 31, 2024
Merged

fix some cdx:python properties #96

merged 5 commits into from
Aug 31, 2024

Conversation

jkowalleck
Copy link
Member

@jkowalleck jkowalleck commented Aug 30, 2024

to be backwards-compatible, the "wrong" property names were not removed, but moved where they belong, and were deprecated in favor of the "fixed" ones.

Signed-off-by: Jan Kowalleck <jan.kowalleck@owasp.org>
@jkowalleck jkowalleck added the bug Something isn't working label Aug 30, 2024
@jkowalleck jkowalleck requested a review from a team as a code owner August 30, 2024 07:18
jkowalleck added a commit to CycloneDX/cyclonedx-python that referenced this pull request Aug 30, 2024
in accoedance with CycloneDX/cyclonedx-property-taxonomy#96

Signed-off-by: Jan Kowalleck <jan.kowalleck@gmail.com>
Signed-off-by: Jan Kowalleck <jan.kowalleck@owasp.org>
Signed-off-by: Jan Kowalleck <jan.kowalleck@owasp.org>
Signed-off-by: Jan Kowalleck <jan.kowalleck@owasp.org>
Signed-off-by: Jan Kowalleck <jan.kowalleck@owasp.org>
@jkowalleck jkowalleck merged commit 3d80d6d into main Aug 31, 2024
4 checks passed
@jkowalleck jkowalleck deleted the cdx-python_fixes_1 branch August 31, 2024 06:44
jkowalleck added a commit to CycloneDX/cyclonedx-python that referenced this pull request Sep 20, 2024
populate the newly added/fixed CycloneDX properties
`cdx:python:package:source:vcs:...` in accordance with
<CycloneDX/cyclonedx-property-taxonomy#96> and
<CycloneDX/cyclonedx-property-taxonomy#98>.

the deprecated properties are still used, so no breaking changes exist.

fixes #789

---------

Signed-off-by: Jan Kowalleck <jan.kowalleck@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants