-
Notifications
You must be signed in to change notification settings - Fork 309
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
type check with mypy as well #987
Comments
Until this is added, I think we should revert the |
mypy output when I run locally
|
Considering the popularity of |
@loferris I think this might be a good way to get familiar with the |
Smaller sub-task. Check our code samples and system tests with mypy (after hacking in a py.typed file). That should be sufficient for adding py.typed file in released package. It'd still be good if the whole package passed mypy, though. |
Is this currently being worked on? I have some free cycles this week and can look at this. |
@plamut I don't believe it is. That'd be great! I wonder if we get it working on main, but add the py.typed file to the v3 branch just so we communicate we might break folks whose type checkers all of the sudden start doing something with our package? |
@tswast You mean making it work (on It sounds better than suddenly introducing Sounds like a plan to me! |
Yes, that's what I meant. |
https://issues.apache.org/jira/browse/BEAM-12975 -- #976 appears to have broken Apache BEAM, which uses mypy for type checking.
This would also help with Mac developers, as I've been unable to run pytype locally due to some missing wheels.
The text was updated successfully, but these errors were encountered: