Skip to content

ci: fix binaries artifacts not uploaded to release #6612

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

Merged
merged 1 commit into from
Apr 11, 2024

Conversation

Hugi-R
Copy link
Contributor

@Hugi-R Hugi-R commented Apr 11, 2024

When GH action download-artifact was updated to v4 in 9f62c01, the default download path changed.
This fixes the file structure of the downloaded artifacts, that can then be uploaded again.

When action download-artifact was updated to v4, the default download path changed.
This fix binaries not being uploaded to releases.
@phymbert
Copy link
Collaborator

Fixes #6604

@EwoutH please review

Copy link
Contributor

@EwoutH EwoutH left a comment

Choose a reason for hiding this comment

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

Yes, think this correctly fixes the configuration.

@Hugi-R did you test this with a release on your own fork?

@Hugi-R
Copy link
Contributor Author

Hugi-R commented Apr 11, 2024

@Hugi-R did you test this with a release on your own fork?

Yes, I tested it yesterday and made a release with binaries https://github.com/Hugi-R/llama.cpp/releases/tag/b2649

@phymbert phymbert merged commit 1bbdaf6 into ggml-org:master Apr 11, 2024
@EwoutH
Copy link
Contributor

EwoutH commented Apr 11, 2024

Perfect, thanks!

tybalex pushed a commit to rubra-ai/tools.cpp that referenced this pull request Apr 17, 2024
When action download-artifact was updated to v4, the default download path changed.
This fix binaries not being uploaded to releases.
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.

3 participants