Skip to content

Conversation

@migmartri
Copy link
Member

The latest integration form @aarlint introduced default values. This patch updates the CLI to takes these into account and show it to the user

┌──────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────┐
│ Attachment inputs                                                                                                        │
├──────────────────┬─────────┬─────────────────────┬───────────────────────────────────────────────────────────────────────┤
│ FIELD            │ TYPE    │ REQUIRED            │ DESCRIPTION                                                           │
├──────────────────┼─────────┼─────────────────────┼───────────────────────────────────────────────────────────────────────┤
│ send_attestation │ boolean │ No (default: true)  │ Send attestation                                                      │
│ send_sbom        │ boolean │ No (default: false) │ Additionally send CycloneDX or SPDX Software Bill Of Materials (SBOM) │
└──────────────────┴─────────┴─────────────────────┴───────────────────────────────────────────────────────────────────────┘

Signed-off-by: Miguel Martinez <miguel@chainloop.dev>
@migmartri migmartri requested review from javirln and jiparis January 3, 2025 16:31
@migmartri migmartri merged commit 033f21a into chainloop-dev:main Jan 3, 2025
13 checks passed
@migmartri migmartri deleted the set-default-values branch January 3, 2025 17:34
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