Skip to content
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

Merged

Conversation

ramonpetgrave64
Copy link
Collaborator

@ramonpetgrave64 ramonpetgrave64 commented Mar 20, 2024

Reverts #3312
#3393

@ramonpetgrave64
Copy link
Collaborator Author

@laurentsimon @ianlewis . Regarding recent breaking changes, I think this is the only one that we must revert.

@ramonpetgrave64 ramonpetgrave64 enabled auto-merge (squash) March 20, 2024 15:20
@laurentsimon laurentsimon changed the title Revert "fix: upload-artifact and download-artifact v4" chore: Revert "fix: upload-artifact and download-artifact v4" Mar 20, 2024
@laurentsimon
Copy link
Collaborator

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?

auto-merge was automatically disabled March 20, 2024 15:23

Pull request was closed

@laurentsimon laurentsimon reopened this Mar 20, 2024
@laurentsimon laurentsimon enabled auto-merge (squash) March 20, 2024 15:24
@laurentsimon
Copy link
Collaborator

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.

@laurentsimon laurentsimon merged commit 90f2eb1 into main Mar 20, 2024
157 of 161 checks passed
@laurentsimon
Copy link
Collaborator

secure-upload-folder pre-submit re-enabled

ramonpetgrave64 added a commit that referenced this pull request Apr 2, 2024
…#3398)"

This reverts commit 90f2eb1.

Signed-off-by: Ramon Petgrave <32398091+ramonpetgrave64@users.noreply.github.com>
ramonpetgrave64 added a commit that referenced this pull request Apr 3, 2024
…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>
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