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(ci): update actions/upload|download-artifact from v3 to v4 #486

Closed
wants to merge 1 commit into from

Conversation

skylenet
Copy link
Member

@nflaig
Copy link
Collaborator

nflaig commented Dec 12, 2024

Just bumping the version is not sufficient, we need to adapt the workflows to work with v4 as there are behavior changes.

Last time I tried upgrading to v4 was in #462 but I didn't get it to work and had done a lot of other changes so I skipped upgrading those. They changed how files are copied and how to deal with existing directories, iirc it throws an error if a directory already exists. There is a overwrite flag (see inputs) but the problem there was it would overwrite the whole folder and since we run this as a matrx it would only keep the last release.

There is probably a way to tweak this (ie. by using different folders) to make it work but this needs to be tested in a forked repo first.

@nflaig
Copy link
Collaborator

nflaig commented Dec 12, 2024

@nflaig
Copy link
Collaborator

nflaig commented Dec 12, 2024

I opened a PR #487 which updates the workflows to work with v4

@skylenet skylenet closed this Dec 12, 2024
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