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

Pin maturin version in release workflow #11

Closed
wants to merge 1 commit into from

Conversation

seddonym
Copy link
Collaborator

@seddonym seddonym commented Sep 25, 2024

The latest release was failing, apparently due to a bug in Maturin v1.7.2.

This pins the Maturin version to v1.7.1 in our release workflow.

https://github.com/kraken-tech/python-rustfluent/actions/runs/11029344761/job/30632470084

(I performed the release by tagging this branch, and it worked.)

Issue raised with maturin in PyO3/maturin#2228

@seddonym seddonym marked this pull request as ready for review September 25, 2024 09:23
@seddonym seddonym requested a review from a team as a code owner September 25, 2024 09:23
@seddonym seddonym requested review from dooferlad and meshy September 25, 2024 09:23
@@ -157,4 +157,5 @@ jobs:
MATURIN_PYPI_TOKEN: ${{ secrets.PYPI_API_TOKEN }}
with:
command: upload
maturin-version: v1.7.1
Copy link

Choose a reason for hiding this comment

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

This seems fine, but perhaps we should upgrade to 1.7.3, which is supposed to fix this bug.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Oh they fixed that quick! Great I will try without this change and see if it fixes it.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

It did! Good shout.

@seddonym seddonym closed this Sep 25, 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