Skip to content
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.

Sign up for GitHub

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

Jump to bottom

remove async patching system #1390

Closed
davidism opened this issue Apr 10, 2021 · 0 comments · Fixed by #1392
Closed

remove async patching system #1390

davidism opened this issue Apr 10, 2021 · 0 comments · Fixed by #1392
Assignees
@davidism
Labels
async
Milestone
3.0.0

Comments

@davidism
Copy link
Member

davidism commented Apr 10, 2021 •
edited
Loading

It's no longer needed since we only support Python 3.6+.

#765 observed that import asyncio takes a long time, so keep those imports at function scope to avoid longer startup time.

A future task is to figure out how to remove the use of asyncio so that other libraries such as Trio could be used instead.
@davidism davidism added the async label Apr 10, 2021
@davidism davidism added this to the 3.0.0 milestone Apr 10, 2021
@davidism davidism self-assigned this Apr 10, 2021
@davidism davidism mentioned this issue Apr 10, 2021
Merged
6 tasks
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Apr 27, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@davidism davidism

Labels
async
Projects
None yet
Milestone
3.0.0
Development

Successfully merging a pull request may close this issue.

async support doesn't require patching pallets/jinja
1 participant
@davidism