Skip to content
This repository was archived by the owner on Aug 6, 2026. It is now read-only.

chore: update to download-artifact@v4 - #336

Merged
laurentsimon merged 1 commit into
slsa-framework:mainfrom
ramonpetgrave64:ramonpetgrave64-upload-download-artifact-v4
Mar 13, 2024
Merged

chore: update to download-artifact@v4#336
laurentsimon merged 1 commit into
slsa-framework:mainfrom
ramonpetgrave64:ramonpetgrave64-upload-download-artifact-v4

Conversation

@ramonpetgrave64

@ramonpetgrave64 ramonpetgrave64 commented Mar 11, 2024

Copy link
Copy Markdown
Contributor

Signed-off-by: Ramon Petgrave <ramon.petgrave64@gmail.com>
@ramonpetgrave64

Copy link
Copy Markdown
Contributor Author

@laurentsimon

@laurentsimon laurentsimon left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Thanks!

I'll wait till the corresponding PR in the slsa-github-generator is merged before merging this one.

laurentsimon added a commit to slsa-framework/slsa-github-generator that referenced this pull request Mar 13, 2024
# Summary

- Fixes #3068 to use upload-artifact and download-artifact v4
- following up in
slsa-framework/example-package#336

## Testing Process

This change is tested with our existing PR Check workflows that use both
directly and indirectly call upload-artifact and download-artifact.
- One test for `secure-upload-folder` should fail in this PR because it
will use `secure-upload-artifact@main`. There's no workaround to
dynamically use the PR's ref instead of `@main`, but after merging this
PR, the test should start passing.

## Checklist

- [x] Review the contributing [guidelines](./../CONTRIBUTING.md)
- [x] Add a reference to related issues in the PR description.
- [x] Update documentation if applicable.
- [x] Add unit tests if applicable.
- [x] Add changes to the [CHANGELOG](./../CHANGELOG.md) if applicable.

---------

Signed-off-by: Ramon Petgrave <ramon.petgrave64@gmail.com>
Signed-off-by: laurentsimon <64505099+laurentsimon@users.noreply.github.com>
Co-authored-by: laurentsimon <64505099+laurentsimon@users.noreply.github.com>
@laurentsimon
laurentsimon merged commit 9978a2a into slsa-framework:main Mar 13, 2024
@ramonpetgrave64 ramonpetgrave64 changed the title update to download-artifact@v4 chore: update to download-artifact@v4 Jul 15, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants