-
Notifications
You must be signed in to change notification settings - Fork 174
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
Vendor syft/migrate witness -> syft direct #716
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tested locally and this worked fine. Like that we are moving to a builder pattern and just fixed a small spelling mistake.
In fact sbom might make more sense and syft as the alias as we did with k9s iirc |
Awesome, thanks for finishing that up. |
@jeff-mccoy I fixed the root examples but we may want to fix the sub command examples as well (looks like this would require looping through the cobra child commands and fixing them manually) |
I think elsewhere we just didn't include the vendored-in examples, I think it might be fine to leave them off as it's more a power-user feature and not really our commands anyway. |
Removed the sub command examples. Left the high-level examples just as a quick start for folks |
* vendor syft/migrate witness -> syft direct * Update examples and docs * dropping the flake-tastic rhel runner test Co-authored-by: Wayne Starr <[email protected]>
Adds
zarf tools syft
command for full access to https://github.com/anchore/syft. Also migrates from 3rd party wrapper (Wtiness) to direct Syft integration. Refactored code to prepare for adding file-based SBOM generation in addition to image-based.