-
Notifications
You must be signed in to change notification settings - Fork 831
Coordinating near future work #151
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
Comments
sounds reasonable. sorry, I can not help. I am busy with other project at the moment. |
Great, thanks!
No problem! actix-web is a really cool project :) |
@konstin: I'll try to see if I come up with something sensible concerning the documentation, but I can't really help with proc macros otherwise. I'll try to review as well whenever needed, don't hesitate to request my reviews on a PR. |
we have to upgrade syn to 0.13 version. this is big task, but it prerequisite for stable procmacro and it will help significantly with errors |
To clarify my initial post, those bullet points are the things I'm going to implement.
Just bumping the versions of syn and quote results in over 300 compilation errors. Doing that upgrade properly would mean rewriting most of the code generation. I'd love to have all the syn 0.13 features, but I have neither the time nor the motivation to do this huge rewrite. |
syn 0.13 upgrade is enevitable. Most annoying part of pyo3 is compiler error reporting. With syn 0.11 it is not possible to show proper errors |
rustfmt is blocked on rust-lang/rustfmt#2824. The clippy errors should be reduced to a fourth of the initial value. The dunder methods should be integrated with std's traits before they are documented, and the types would need some proper testing before enumerating them. |
My wishlist and plan for pyo3 would be the following, mostly in that order:
::pyo3
and::std
everywhere), remove the unnecessary underscores and fix the unused imports.__methods__
and other supported python features.@fafhrd91 @althonos Any thoughts?
The text was updated successfully, but these errors were encountered: