-
-
Notifications
You must be signed in to change notification settings - Fork 349
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
Add pylint missing-await checker #700
Conversation
3e2552f
to
7c84dac
Compare
Codecov Report
@@ Coverage Diff @@
## master #700 +/- ##
==========================================
- Coverage 99.52% 99.42% -0.11%
==========================================
Files 102 105 +3
Lines 12192 12272 +80
Branches 904 916 +12
==========================================
+ Hits 12134 12201 +67
- Misses 36 48 +12
- Partials 22 23 +1
|
It looks like there's a convention of having standalone
Maybe we should follow, with a In other news, it looks like py38-dev and astroid have become incompatible, so our CI is currently broken... maybe we should just disable the static analysis tests when |
7c84dac
to
115d156
Compare
fixed false positive from async generators |
115d156
to
5b38a11
Compare
6b123c5
to
9d5c405
Compare
typical usage: $ pylint --load-plugins=trio.testing.pylint-missing-await --enable=missing-await DIR/
c28ec3f
to
ae3b396
Compare
ae3b396
to
69fba08
Compare
|
This is like five years old and the project switched to ruff since. Closing. |
pylint support for #671
TODO:
This runs fine on trio codebase and my own codebase. I haven't looked hard for false negatives.
Run on trio codebase: