21X and 22X, blocking sync calls in async function

jakkdl commented 1 year ago

Checking off the first couple sync calls in #58

Flake8TrioVisitor.__subclasses__ didn't do it anymore once I started using multilevel inheritance, so solved it by creating a decorator for error classes that registers them. (type hinting the decorator function was fun :sweat_smile:)

Wasn't sure if the http calls and process invocation bullet points wanted different error messages for each, and the messages themselves are kinda rough, but went with this for a first pass.

TODO:

[x] changelog
[ ] better readme text & error messages
[x] os.spawn[vl]p?e? should give TRIO220 sometimes
[x] test file shouldn't be called trio210.py - change regex so it can be named trio2xx.py or split into trio21x.py and trio22x.py

Don't bother merging both this and #91, merge one and I'll fix the conflicts in the other.

jakkdl commented 1 year ago

Not gonna bother composing beautiful error messages and readme texts until the split has been decided on.

jakkdl commented 1 year ago

addressed comments, rebased on top of main. This PR is turning out quite meaty indeed.

The documentation and error messages are still mediocre, feel free to suggest changes.

python-trio / flake8-async

21X and 22X, blocking sync calls in async function #94