search

python-trio / flake8-async

Highly opinionated linter for Trio code
https://flake8-async.readthedocs.io
MIT License
17 stars 2 forks source link

ASYNC100 no longer triggers on context managers containing a `yield` #228

Closed jakkdl closed 6 months ago

jakkdl commented 6 months ago

I .. don't know why I didn't open a PR when pushing this branch. fixes #224

jakkdl commented 6 months ago

oh that's why, it wasn't finished x)

jakkdl commented 6 months ago

eval_files/noqa.py was using yields + async100 + async911 to test noqa functionality, relying on context managers with yields in them to raise async100. So changed it to using returns (and async910) instead