Skip to content

chore: Revert "fix: upload-artifact and download-artifact v4"#3398

Merged
laurentsimon merged 1 commit intomainfrom
revert-3312-ramonpetgrave64-upload-download-artifact-v4
Mar 20, 2024
Merged

chore: Revert "fix: upload-artifact and download-artifact v4"#3398
laurentsimon merged 1 commit intomainfrom
revert-3312-ramonpetgrave64-upload-download-artifact-v4

Conversation

@ramonpetgrave64
Copy link
Contributor

@ramonpetgrave64 ramonpetgrave64 commented Mar 20, 2024

Reverts #3312
#3393

@ramonpetgrave64
Copy link
Contributor 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
@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>
@ramonpetgrave64 ramonpetgrave64 deleted the revert-3312-ramonpetgrave64-upload-download-artifact-v4 branch January 29, 2025 20:09
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