new error code: TRIO912 unnecessary checkpoints + autofix

python-trio / flake8-async

Highly opinionated linter for Trio code

https://flake8-async.readthedocs.io

MIT License

17 stars 2 forks source link

new error code: TRIO912 unnecessary checkpoints + autofix #183

Open jakkdl opened 1 year ago

jakkdl commented 1 year ago

fixes #70 (merely 5 months after it was opened :grin:)

Early WIP, I only got the basic case down. I started on this before #182, but I'm prioritizing that one.

Remaining TODO:

if statements
- this shouldn't be that complicated, I've mostly got the logic figured out.
loops
- once if is solved this can probably use similar logic, but it'll likely be messy
try/except
- same

jakkdl commented 2 months ago

I picked this up and worked on it somewhat recently, but stalled out due to the complexity. The complexity is mostly because it's hard to resist solving the juicy edge cases and autofixing everything, so next time I pick this up I'll just drop the complex parts, see if the basic cases that are working are worth having as a rule, and clean up the PR so it can be merged.