-
-
Notifications
You must be signed in to change notification settings - Fork 834
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
Document supported async libraries #387
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks good to me. 👍 For me it definitely solidifies the idea that we should be auto-detecting which async library, but that's something for another day!
540901a
to
7a706ed
Compare
@sethmlarson I added an extra admonition with installation instructions for trio; I think your comment (which it seems you removed?) was relevant. |
Eh, I'm wondering if it's needed cuz it's not possible to run trio without trio installed. |
Yeah, fair point. Does that mean we don't actually need the |
Probably not! |
7a706ed
to
c4d3739
Compare
c4d3739
to
c3425c5
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks great
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great, thanks yall!
Following the addition of trio support via #276.