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

Introduce "cyclonedx rename-entity" command #346

Merged
merged 2 commits into from
Aug 11, 2024

Conversation

jimklimov
Copy link
Contributor

Primarily written 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.

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.

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>
@mtsfoni mtsfoni merged commit f43675e into CycloneDX:main Aug 11, 2024
3 checks passed
mtsfoni added a commit that referenced this pull request Aug 11, 2024
@mtsfoni
Copy link
Contributor

mtsfoni commented Aug 11, 2024

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 🤔)
Could you open it again?

@jimklimov
Copy link
Contributor Author

ok

@jimklimov
Copy link
Contributor Author

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...

@jimklimov
Copy link
Contributor Author

Re-opening "as is" done, see #396

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.

2 participants