Skip to content

Comments

editorial: draft: Improve instructions to verify source VSAs#1498

Merged
TomHennen merged 2 commits intoslsa-framework:mainfrom
TomHennen:fix1445
Oct 27, 2025
Merged

editorial: draft: Improve instructions to verify source VSAs#1498
TomHennen merged 2 commits intoslsa-framework:mainfrom
TomHennen:fix1445

Conversation

@TomHennen
Copy link
Contributor

The prior instructions weren't complete and were somewhat spread out. Tried to consolidate in one spot. Also fixed a typo in the general verification summary docs that incorrectly referenced slsaResult instead of verificationResult.

fixes #1445

The prior instructions weren't complete and were somewhat spread out.
Tried to consolidate in one spot.  Also fixed a typo in the
general verification summary docs that incorrectly referenced
slsaResult instead of verificationResult.

fixes slsa-framework#1445

Signed-off-by: Tom Hennen <tomhennen@google.com>
@netlify
Copy link

netlify bot commented Oct 21, 2025

Deploy Preview for slsa ready!

Name Link
🔨 Latest commit 133bd63
🔍 Latest deploy log https://app.netlify.com/projects/slsa/deploys/68f7e5f92a8a9c000800f064
😎 Deploy Preview https://deploy-preview-1498--slsa.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@TomHennen TomHennen changed the title spec: draft: Improve instructions to verify source VSAs content: draft: Improve instructions to verify source VSAs Oct 21, 2025
@TomHennen TomHennen requested review from adityasaky and arewm October 21, 2025 18:45
@TomHennen TomHennen changed the title content: draft: Improve instructions to verify source VSAs editorial: draft: Improve instructions to verify source VSAs Oct 21, 2025
@TomHennen
Copy link
Contributor Author

Updated the type of this PR to editorial as it doesn't change the meaning of the spec, just tries to clarify what was already there.

Signed-off-by: Tom Hennen <tomhennen@google.com>
@TomHennen TomHennen requested a review from arewm October 21, 2025 20:00
@TomHennen TomHennen added source-track slsa 1.2 Required for SLSA 1.2 release. Please apply it liberally! labels Oct 23, 2025
Copy link
Collaborator

@zachariahcox zachariahcox left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

love it

@TomHennen TomHennen merged commit 34a8a0a into slsa-framework:main Oct 27, 2025
6 checks passed
@github-project-automation github-project-automation bot moved this from 🆕 New to ✅ Done in Issue triage Oct 27, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

slsa 1.2 Required for SLSA 1.2 release. Please apply it liberally! source-track

Projects

Status: Done
Status: Done

Development

Successfully merging this pull request may close these issues.

"How to verify SLSA a source revision..." is incomplete

3 participants