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

chore: update docs for container sbom command #5788

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

thomasschafer
Copy link

@thomasschafer thomasschafer commented Mar 14, 2025

This PR updates the documentation based on the changes to container-cli here. Note that this PR must only be merged after that container-cli PR has been merged, and I'll need to update this one to pull in the latest version of the extension.

Pull Request Submission Checklist

  • Follows CONTRIBUTING guidelines
  • Includes detailed description of changes
  • Contains risk assessment (Low | Medium | High)
  • Highlights breaking API changes (if applicable)
  • Links to automated tests covering new functionality
  • Includes manual testing instructions (if necessary)
  • Updates relevant GitBook documentation (PR link: ___)
  • Includes product update to be announced in the next stable release notes

How should this be manually tested?

To test, build and run the CLI with the arguments

container sbom --format spdx2.3+json nginx:latest --platform=linux/amd64

Test with different values for --platform, e.g. linux/386, and you should see different results

Copy link
Contributor

github-actions bot commented Mar 14, 2025

Warnings
⚠️

Please make changes to snyk help text in Gitbook. Changes will be automatically synchronised to Snyk CLI as a scheduled PR.
For more information, see: help/README.md.

Generated by 🚫 dangerJS against a16bac1

@thomasschafer thomasschafer force-pushed the feat/add-platform-flag-container-sbom branch from ab80e37 to a16bac1 Compare March 18, 2025 15:45
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