-
Notifications
You must be signed in to change notification settings - Fork 229
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
SLSA v1: Mark as Approved specification #829
Conversation
✅ Deploy Preview for slsa ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
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.
We should point to v1.0 as the current version. I also wonder about the version used for provenance.
Yes. I am working on a separate PR for that, which is much bigger. I wanted to separate out the Approval (which is codified in our governance) from changing the default (which is not). |
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.
LGTM
- Mark SLSA v1.0, Provenance v1, and VSA v1 as Approved. - Mark release candidates as Retired and hide them from the dropdown. - Remove `?draft` from `predicateType`. Note: This only marks v1.0 as Approved. A separate PR will set the default to v1.0. Signed-off-by: Mark Lodato <lodato@google.com>
Signed-off-by: Mark Lodato <lodato@google.com>
- Make v1.0 the default nav version. - Update front page to link to v1.0. - Update redirects to point to v1.0. Note: This just sets the default version to v1.0. PR slsa-framework#829 marks it as Approved. Part of slsa-framework#513. Signed-off-by: Mark Lodato <lodato@google.com>
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.
🥳 🍾
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.
LGTM
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.
🎉
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.
🥳
Merging now. Thanks all! |
SLSA v1: Mark as Approved specification
?draft
frompredicateType
.Part of #513.
Important: We cannot merge this until Tuesday, April 18, which is after the end of the mandatory two-week review period. In the meantime, please review and approve so that we can merge on the 18th.
Note: This just marks v1.0 as approved. PR #832 sets the default version to v1.0.