-
Notifications
You must be signed in to change notification settings - Fork 135
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
Show example of customing the attestation output #353
Comments
Just to clarify, where would you like to customize it? on the workflow run or on the release? There is an example where it's used in the release step of the second example but I'd like to understand the use case you're thinking of. |
In the workflow itself. In my case, I already had an existing workflow which assumed the name of the Go generator and I wanted to retain the same name. I wasn't sure how to configure that output value because the README didn't show this, so I ended up guessing wrong a couple times and just updated my workflow to use the default name instead. |
Right now there isn't an input that would allow you to change the name. I suppose we could add one but I think we want to keep inputs to a minimum if we can. The workflow itself is temporary so feels a bit arbitrary to me, but I can definitely see the need for customization on the release. Do you have an example somewhere we could look at? |
We added an input to customize the attestation file name a while back and I created a PR to document it in #618. I think that should allow us to close this issue. wdut? @chipzoller |
Yes, that looks good, and thank you. |
It'd be nice in the README here if you could show an example of how to use the workflow output
attestation-name
to customize the name of the uploaded attestation. The input is shown (because it's required), but the output isn't and the default output file appears to beattestation.intoto.jsonl
.The text was updated successfully, but these errors were encountered: