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: Fix release workflow #419

Closed
wants to merge 1 commit into from
Closed

Conversation

saulecabrera
Copy link
Member

This commit attempts to address
#418, which is currently blocking release. This change pretty much includes all the suggestions in the linked issue.

Additionally, this change makes use of @actions/download-artifact@v4. Given that the new cross-gem action gives us control over how to upload the gem artifacts, download-artifact should now be compatible. It's also worth noting that download-artifact@v3 was deprecated on Nov 30th 2024 https://github.com/actions/download-artifact?tab=readme-ov-file#actionsdownload-artifact

This commit attempts to address
#418, which is
currently blocking release. This change pretty much includes all the
suggestions in the linked issue.

Additionally, this change makes use of @actions/download-artifact@v4.
Given that the new cross-gem action gives us control over how to upload
the gem artifacts, download-artifact should now be compatible. It's also
worth noting that download-artifact@v3 was deprecated on Nov 30th 2024
https://github.com/actions/download-artifact?tab=readme-ov-file#actionsdownload-artifact
Copy link
Collaborator

@jbourassa jbourassa left a comment

Choose a reason for hiding this comment

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

I tried roughly this and can't get it to work1, for some reason the process dies when compiling. That's what I was trying to describe in #418.

Do we have a reason to believe this version wouldn't run into the same issue?

@@ -42,18 +42,23 @@ jobs:

- uses: oxidize-rb/actions/setup-ruby-and-rust@v1
with:
ruby-version: "3.3"
ruby-version: "3.3.5"
Copy link
Collaborator

Choose a reason for hiding this comment

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

I don't think this should be necessary, but I don't know if it hurst either.

@saulecabrera
Copy link
Member Author

I tried roughly this and can't get it to work1, for some reason the process dies when compiling. That's what I was trying to describe in #418.

Do we have a reason to believe this version wouldn't run into the same issue?

Oh I totally missed that, in that case, I don't think there's anything fundamentally different in this version that would cause it to succeed.

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