Repository navigation
Introduce "cyclonedx rename-entity" command - #346
Conversation
Primarily as a practical test case for Bom.WalkThis() and Bom.RenameBomRef() methods introduced in the library, but may be useful to have exposed for end-users. Signed-off-by: Jim Klimov <jimklimov@gmail.com>
Signed-off-by: Jim Klimov <jimklimov@gmail.com>
|
Sorry I merged this a little to fast and didn't see that it relies on a PR in the library. (All the check were green 🤔) |
|
ok |
|
In a larger picture, I keep hoping to get a higher priority at work to revisiting all those posted changes and resync with newer development merged to the upstream. Spec 1.5, etc. introduced some changes... |
|
Re-opening "as is" done, see #396 |
|
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. |
Primarily written as a practical test case for
Bom.WalkThis()andBom.RenameBomRef()methods introduced in the library, but may be useful to have exposed for end-users.Relies on CycloneDX/cyclonedx-dotnet-library#245 for the bulk of work (BomEntity base-class and interface family, etc.) and CycloneDX/cyclonedx-dotnet-library#256 for metadata update of the output document.