Skip to content

MergeCommand.cs: ensure a timestamp and use new cyclonedx-dotnet-library features… #334

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

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

jimklimov
Copy link
Contributor

…to CleanupMetadataComponent() and CleanupEmptyLists() as a finishing touch, to avoid inducing a spec violation with a duplicate bom-ref

The new cleanup features depend on library PR CycloneDX/cyclonedx-dotnet-library#245

…MetadataComponent() and CleanupEmptyLists() as a finishing touch, to avoid inducing a spec violation with a duplicate bom-ref

Signed-off-by: Jim Klimov <[email protected]>
@jimklimov jimklimov force-pushed the flatmerge-dedup-squash branch from bdf6013 to cea2b51 Compare August 11, 2023 00:10
Copy link

@ProlucidPB ProlucidPB left a comment

Choose a reason for hiding this comment

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

Would love to see this merged in soon!

@Nowcide
Copy link

Nowcide commented Jul 4, 2025

I'd love to see these changes merge, to enable SBOMs to comply with NTIA, NIST and CRA (Cyber Resiliant Act) regulations, which require time stamping as a minimum requirement for SBOMs.

@AlexandreEXFO
Copy link

#438

@AlexandreEXFO
Copy link

@mtsfoni : Would you have time to review this PR? Please.

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.

4 participants