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

Updated minimum Rust version #131

Closed
wants to merge 1 commit into from
Closed

Updated minimum Rust version #131

wants to merge 1 commit into from

Conversation

nihaals
Copy link
Contributor

@nihaals nihaals commented Jun 23, 2020

Not sure if Rust 1.39+, Rust >=1.39 or something like minimum... is needed.
See PyO3/pyo3#5 (comment).
PyO3/pyo3@7075827 (PyO3/pyo3#989) is blocking this from being merged.

@rth
Copy link
Contributor

rth commented Jun 23, 2020

Thanks! We would need to wait for PyO3 0.11.0 to be released PyO3/pyo3#989 before updating the readme.

It would be good to try building with PyO3 master in CI though and check if anything needs to be updated.

@nihaals
Copy link
Contributor Author

nihaals commented Jun 23, 2020

Oh didn't see the PR, I'll add it to the blocking list

@kngwyu
Copy link
Member

kngwyu commented Jun 23, 2020

Thanks!
Could you please send a pull request for #132?

@kngwyu kngwyu closed this Jun 23, 2020
@kngwyu
Copy link
Member

kngwyu commented Jun 23, 2020

Also, it's better if we have minimum-rustc bagde
minimum rustc 1.39

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