codecov-commenter
commented
1 year ago Codecov Report
Merging #91 (51b67a1) into main (c4da56c) will decrease coverage by
0.97%. The diff coverage is95.37%.
@@ Coverage Diff @@
## main #91 +/- ##
==========================================
- Coverage 97.88% 96.92% -0.97%
==========================================
Files 70 67 -3
Lines 2225 2243 +18
Branches 416 427 +11
==========================================
- Hits 2178 2174 -4
- Misses 38 59 +21
- Partials 9 10 +1 | Impacted Files | Coverage Δ | |
|---|---|---|
| docs_src/autowiring.py | 0.00% <0.00%> (ø) |
|
| di/_container.py | 93.48% <93.48%> (ø) |
|
| docs_src/async_constructor.py | 100.00% <100.00%> (ø) |
|
| docs_src/async_init_dependency.py | 100.00% <100.00%> (ø) |
|
| docs_src/bind_as_a_dep.py | 100.00% <100.00%> (ø) |
|
| docs_src/bind_hooks.py | 100.00% <100.00%> (ø) |
|
| docs_src/default_scope.py | 94.44% <100.00%> (ø) |
|
| docs_src/headers_example.py | 97.56% <100.00%> (ø) |
|
| docs_src/injectable_class.py | 100.00% <100.00%> (ø) |
|
| docs_src/invalid_scope_dependence.py | 100.00% <100.00%> (ø) |
|
| ... and 25 more |
There are backwards compatibility shims on Container.execute_{sync,async} so this should, in theory, not break any code, but it is nontheless a pretty large API change so I'm marking it as a breaking change.
This is (hopefully, time permitting) going to be a part of a larger effort to simplify the APIs and implementation of this library