-
Notifications
You must be signed in to change notification settings - Fork 16
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
Consider using typed-astunparse #43
Comments
py-backwards was using it before and there were problems similar to simonpercivall/astunparse#18 and simonpercivall/astunparse#17 |
Wouldn't be easier to make PRs to it to fix those instead of creating a fork? Because having typed unparse is really useful and makes #44 work great. |
Not that much, typed-astunparse is based on astunparse. And pull requests were already submitted to astunparse. Also, are we really need type comments in compiled code? I think it would be better to have a less readable compiled code and something like source maps. |
Hm, it seems they are waiting to be rebased?
Yes, one could automatically generate typing stub files. But comments seemed easier at a time. Otherwise one would have to create whole infrastructure to pass those types around and make those extra files. |
https://github.com/simonpercivall/astunparse/pulls
As I remember, in the current implementation of packager original sources are still in the package. So IDE will see original sources with types. Not sure about mypy. And source maps will be mostly used for stack traces, like in js. |
Now closed simonpercivall/astunparse#19 and simonpercivall/astunparse#20 |
It seems this one supports also preserving type comments: https://github.com/mbdevpl/typed-astunparse (in contrast with https://github.com/nvbn/py-backwards-astunparse).
The text was updated successfully, but these errors were encountered: