Skip to content

MergeCommand: use Bom.BomMetadataUpdate()… - #340

Open
jimklimov wants to merge 1 commit into
CycloneDX:mainfrom
jimklimov:bom-self-metadata
Open

MergeCommand: use Bom.BomMetadataUpdate()…#340
jimklimov wants to merge 1 commit into
CycloneDX:mainfrom
jimklimov:bom-self-metadata

Conversation

@jimklimov

Copy link
Copy Markdown
Contributor

…and BomMetadataReferThisToolkit() methods in OOP fashion

Depends on library changes from CycloneDX/cyclonedx-dotnet-library#256

  • No idea how to facilitate this with C# recipes to pass CI before that PR gets merged...

…lkit() methods in OOP fashion

Signed-off-by: Jim Klimov <jimklimov@gmail.com>
@jimklimov

Copy link
Copy Markdown
Contributor Author

FYI: This PR is part of a series I've opened 3 years ago, and stalled in review for whatever reasons including the under-the-hood use of reflection to implement inspection of arbitrary BOM entities to merge them, including the ones still open at this time:

I (and my dayjob's budget) have recently employed Claude AI to pick up where I left long ago, and rewrite those changes in idiomatic C#, rebased over current upstream achievements.

After some internal dev-testing shows that this rewrite is successful, I hope these old PRs will be supplanted by a new series with new technological base and same or better feature set as what I was stuck with using (slowly bit-rotting over the years). Hopefully the new set of PRs would be less questionable for an upstream merge :)

I'll add this note to all impacted PRs listed above.

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.

1 participant