-
Notifications
You must be signed in to change notification settings - Fork 60
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
Async rendering support #393
Conversation
Codecov Report
@@ Coverage Diff @@
## master #393 +/- ##
==========================================
+ Coverage 96.02% 96.36% +0.33%
==========================================
Files 3 3
Lines 151 165 +14
Branches 18 19 +1
==========================================
+ Hits 145 159 +14
Misses 6 6
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
Please fix the merge conflict |
OK, went a bit weird for a moment. All sorted now. |
Awesome, I'll prepare a new release with this feature. @Dreamsorcerer would you get write access to the repo (GitHub Maintainer role precisely). I don't expect much work here but don't want to be a bus factor and shuffle off routine procedures on somebody else :) |
Yep, go ahead. Happy to keep things going. |
Great! |
-- This is branched from the typing PR, so merge that first to reduce the diff --
This adds
render_string_async()
andrender_template_async()
functions.It also modifies the
@template
decorator to use the async rendering whenenable_async=True
has been passed to the setup.Fixes #333