-
Notifications
You must be signed in to change notification settings - Fork 128
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: Revert "fix: upload-artifact and download-artifact v4" #3398
chore: Revert "fix: upload-artifact and download-artifact v4" #3398
Conversation
This reverts commit b595e06.
@laurentsimon @ianlewis . Regarding recent breaking changes, I think this is the only one that we must revert. |
Thanks, let's merge this one in. I think we also need to revert the one on the attestation-name option deprecation. Can you send a similar reverting PR to example-package? |
Pre-submit failing due to using upload-artifact at v3 but download-artifact at v4 which are incompatible. I'll disable that particular check to merge this PR. |
secure-upload-folder pre-submit re-enabled |
…act v4"" (#3499) Reverts #3398 Following up with - slsa-framework/example-trw#3 - slsa-framework/example-package#340 - slsa-framework/slsa-verifier#719 Signed-off-by: Ramon Petgrave <32398091+ramonpetgrave64@users.noreply.github.com>
Reverts #3312
#3393