Repository navigation
basic Asyncio support - #210
Conversation
|
Looks good so far! Will re-review when you think it's done 👍 |
Zac-HD
left a comment
There was a problem hiding this comment.
let's rebase on master, add ASYNC102 support, and then we're probably done?
…ssing on asyncio, though not necessarily for good reasons
…rk with which library, and minor updates. 102/103/104 now sees asyncio.exceptions.CancelledError as a critical exception. Fix import/arg detection of asyncio. add asyncio102_asyncio. add async103_all_imported. Finally make test_anyio_from_config autodetect the correct line number. Fix BASE_LIBRARY marker being interpreted as a bool. Make #NOTRIO/#NOASYNCIO/#NOANYIO run the visitor but ignore the result, instead of skipping, to check it doesn't crash. Generalize error-message-library-check.
|
Uh, I ended up doing A Lot:tm:. But the diff didn't actually end up being all that big. The rebase might break viewing only new changes though, so you might want to review commit-by-commit. Only ones listed in #208 not supported now is:
So I'm now comfortable saying this has decent TODO: Oh I did rename the repository, so should update links. |
Zac-HD
left a comment
There was a problem hiding this comment.
LGTM; agree that we could release this and should release whatever we have on the weekend (when I set up release automation again).
I'll let you merge after any docs/links updates you want to make.
Co-authored-by: Zac Hatfield-Dodds <zac.hatfield.dodds@gmail.com>
|
ah oops, I realized I forgot to ask your opinion on |
|
I'd probably go with the slash, but don't really care so long as we're consistent. |
first steps on #208
106, 200, 21x and 9xx are supported, though no autofix.
This is rebased on top of #209, so if reviewing you should only look at the latest commit(s).
todo:
--libraryflag instead.