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

build: revert bump actions/upload-artifact from 3 to 4, for #5648 #5662

Merged

Conversation

stasadev
Copy link
Member

The Issue

Go to https://github.com/ddev/ddev/actions/runs/7340705862?pr=5661 and click on the artifact and see:

Failed to generate URL to download artifact.

How This PR Solves The Issue

There is no update guide at the moment, go back to actions/upload-artifact@v3 for now.

See

Manual Testing Instructions

Automated Testing Overview

Related Issue Link(s)

Release/Deployment Notes

@stasadev stasadev requested a review from a team as a code owner December 27, 2023 18:30
@stasadev stasadev merged commit bc5a4b8 into ddev:master Dec 27, 2023
11 checks passed
@stasadev stasadev deleted the 20231227_build_revert_upload_artifact branch December 27, 2023 18:32
@stasadev
Copy link
Member Author

stasadev commented Dec 27, 2023

I checked the build of the master branch, it works https://github.com/ddev/ddev/actions/runs/7340970014

Before merging #5648, I checked in a separate repository that the artifacts appeared on the Checks page and the bot comment looked legitimate, but I didn't click the link to check the artifact itself.

I will be more careful about such changes in the future.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant