Closed jcpunk closed 2 years ago
Hello @jcpunk! 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 #548 (d6e04ab) into master (814669d) will decrease coverage by
0.16%
. The diff coverage is100.00%
.
@@ Coverage Diff @@
## master #548 +/- ##
==========================================
- Coverage 84.48% 84.31% -0.17%
==========================================
Files 48 48
Lines 3100 3093 -7
Branches 482 481 -1
==========================================
- Hits 2619 2608 -11
- Misses 450 452 +2
- Partials 31 33 +2
Flag | Coverage Δ | |
---|---|---|
python-3.10 | 84.09% <100.00%> (+0.15%) |
:arrow_up: |
python-3.6 | 83.97% <100.00%> (+0.28%) |
:arrow_up: |
python-pypy-3.7 | ? |
Flags with carried forward coverage won't be shown. Click here to find out more.
Impacted Files | Coverage Δ | |
---|---|---|
...gine/framework/dataspace/datasources/postgresql.py | 85.36% <100.00%> (-0.41%) |
:arrow_down: |
src/decisionengine/framework/dataspace/maintain.py | 99.04% <0.00%> (-0.96%) |
:arrow_down: |
.../decisionengine/framework/engine/DecisionEngine.py | 89.75% <0.00%> (-0.74%) |
:arrow_down: |
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 814669d...d6e04ab. Read the comment docs.
It is sad to drop, but alas....