Skip to content

Execution context Refactor - #51

Merged
eadwinCode merged 5 commits into
mainfrom
execution_context
Jan 5, 2023
Merged

Execution context Refactor#51
eadwinCode merged 5 commits into
mainfrom
execution_context

Conversation

@eadwinCode

Copy link
Copy Markdown
Collaborator
  • Separated HostContext from ExecutionContext which reduced complexity in the creation ExecutionContext
  • ExecutionContext class now manages context around context and execution functions
  • HostContext class manages general request context and is available for exception handlers

@codecov-commenter

Copy link
Copy Markdown

Codecov Report

Merging #51 (e3d224d) into main (81d250a) will increase coverage by 0.14%.
The diff coverage is 93.50%.

@@            Coverage Diff             @@
##             main      #51      +/-   ##
==========================================
+ Coverage   92.45%   92.59%   +0.14%     
==========================================
  Files         144      150       +6     
  Lines        5247     5362     +115     
==========================================
+ Hits         4851     4965     +114     
- Misses        396      397       +1     
Impacted Files Coverage Δ
ellar/common/routing/__init__.py 100.00% <ø> (ø)
ellar/core/routing/controller/websocket/handler.py 45.00% <0.00%> (ø)
ellar/core/routing/controller/websocket/route.py 46.15% <0.00%> (ø)
ellar/core/routing/websocket/route.py 80.51% <0.00%> (ø)
ellar/common/decorators/base.py 90.90% <50.00%> (ø)
ellar/core/exceptions/handlers.py 87.17% <50.00%> (ø)
ellar/core/routing/websocket/handler.py 67.10% <50.00%> (ø)
ellar/core/middleware/exceptions.py 89.74% <75.00%> (+0.85%) ⬆️
ellar/core/templating/renderer.py 96.96% <75.00%> (ø)
ellar/common/routing/params.py 82.50% <83.33%> (ø)
... and 34 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@eadwinCode
eadwinCode merged commit 5668f65 into main Jan 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants