-
Notifications
You must be signed in to change notification settings - Fork 97
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
Release v0.2.3 with aarch64 support #279
Conversation
19d4ed0
to
472f3bf
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great, thanks lot! I left some minor comments, otherwise this looks good to me once the CI passes.
2d561a0
to
1a2f3b2
Compare
1a2f3b2
to
fcd55f7
Compare
I have added the pip release into the github release page, because in china This however makes the release page look a bit crowded. I think its understandable in my opinion, and we can always later if we receive bad feedback about it. But open for discussion. Also I have tested the |
I think that we can release |
fcd55f7
to
e98de8f
Compare
This commit replace previously manually generated pypi release with the config generated by: `maturin generate-ci github --zig`.
The name generator uses a thread-local random number generator internally, which makes it `!Send`. So we should not keep it across await points because our `multiple_daemons` example needs to spawn it in the tokio runtime, which requires `Send`.
0a4e541
to
51b35aa
Compare
This Pull Request change:
maturin generate-ci github --zig
. This have the added benefit to supportaarch64
.aarch64
for our binaries in our release CI.