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(xml): requirement descriptions should be unbounded #533

Merged
merged 4 commits into from
Nov 7, 2024

Conversation

hakandilek
Copy link
Contributor

fixes #528

where occurrences of definitions.standards.requirements.descriptions should be unbounded

fixes CycloneDX#528 where occurrences of definitions.standards.requirements.descriptions should be unbounded

Signed-off-by: Hakan Dilek <hakandilek@gmail.com>
@hakandilek hakandilek requested a review from a team as a code owner November 4, 2024 07:48
@jkowalleck jkowalleck added defect CDX 1.6 related to release v1.6 labels Nov 4, 2024
@jkowalleck jkowalleck added this to the 1.6.1 milestone Nov 4, 2024
under definitions.standards, requirement may have
- multiple descriptions and
- a text field

Signed-off-by: Hakan Dilek <hakandilek@gmail.com>
@hakandilek
Copy link
Contributor Author

@jkowalleck This one is ready for a review including the tests mentioned before.

@jkowalleck jkowalleck requested a review from a team November 4, 2024 09:58
@hakandilek
Copy link
Contributor Author

This one Java test is failing because it's referring to the prepacked schema file from within the released (v9.0.5) jar file: jar:file:{user.home}/.m2/repository/org/cyclonedx/cyclonedx-core-java/9.0.5/cyclonedx-core-java-9.0.5.jar!/bom-1.6.xsd

@jkowalleck
Copy link
Member

This one Java test is failing because it's referring to the prepacked schema file from within the released (v9.0.5) jar file: jar:file:{user.home}/.m2/repository/org/cyclonedx/cyclonedx-core-java/9.0.5/cyclonedx-core-java-9.0.5.jar!/bom-1.6.xsd

exactly.

It is on my radar for a long while, to rework these java tests in a way they use the schema files in this repo, instead of some shipped ones in a library...
see #256

@jkowalleck jkowalleck merged commit 8ded79c into CycloneDX:master Nov 7, 2024
7 of 8 checks passed
@hakandilek hakandilek deleted the fix/issue-528 branch November 7, 2024 20:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CDX 1.6 related to release v1.6 defect format: XML
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[bug]: Inconsistent number of occurrences for definitions.standards.requirements.descriptions
2 participants