Closed knoepfel closed 2 years ago
Hello @knoepfel! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:
There are currently no PEP 8 issues detected in this Pull Request. Cheers! :beers:
Merging #541 (39ac7a2) into master (30951a5) will increase coverage by
0.35%
. The diff coverage is100.00%
.
@@ Coverage Diff @@
## master #541 +/- ##
==========================================
+ Coverage 84.12% 84.48% +0.35%
==========================================
Files 48 48
Lines 3100 3100
Branches 482 482
==========================================
+ Hits 2608 2619 +11
+ Misses 460 450 -10
+ Partials 32 31 -1
Flag | Coverage Δ | |
---|---|---|
python-3.10 | 83.96% <100.00%> (+0.35%) |
:arrow_up: |
python-3.6 | 83.75% <100.00%> (+0.32%) |
:arrow_up: |
python-pypy-3.7 | 83.68% <100.00%> (+0.09%) |
:arrow_up: |
Flags with carried forward coverage won't be shown. Click here to find out more.
Impacted Files | Coverage Δ | |
---|---|---|
...ecisionengine/framework/taskmanager/TaskManager.py | 96.48% <100.00%> (ø) |
|
...rc/decisionengine/framework/modules/SourceProxy.py | 86.95% <0.00%> (+15.94%) |
:arrow_up: |
Continue to review full report at Codecov.
Legend - Click here to learn more
Δ = absolute <relative> (impact)
,ø = not affected
,? = missing data
Powered by Codecov. Last update 30951a5...39ac7a2. Read the comment docs.
This PR tests:
This tests work in the "old" system with the source proxy, and the "new" system where there is no source proxy, but the workflow is assembled through configuration-combination utilities.