Closed gyermolenko closed 5 years ago
Merging #273 into master will decrease coverage by
0.02%
. The diff coverage is20%
.
@@ Coverage Diff @@
## master #273 +/- ##
==========================================
- Coverage 69.85% 69.82% -0.03%
==========================================
Files 62 62
Lines 2289 2287 -2
==========================================
- Hits 1599 1597 -2
Misses 690 690
Impacted Files | Coverage Δ | |
---|---|---|
patterns/behavioral/specification.py | 83.33% <ø> (ø) |
:arrow_up: |
patterns/behavioral/template.py | 96.66% <ø> (ø) |
:arrow_up: |
patterns/behavioral/state.py | 97.72% <ø> (ø) |
:arrow_up: |
patterns/fundamental/delegation_pattern.py | 0% <ø> (ø) |
:arrow_up: |
patterns/behavioral/command.py | 96.96% <ø> (ø) |
:arrow_up: |
setup.py | 0% <ø> (ø) |
:arrow_up: |
patterns/behavioral/memento.py | 0% <0%> (ø) |
:arrow_up: |
patterns/structural/flyweight.py | 38.33% <0%> (ø) |
:arrow_up: |
patterns/behavioral/iterator.py | 94.44% <100%> (-0.56%) |
: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 e5b253a...0731056. Read the comment docs.
As it turns out
flake8 *py
command checks only files by that mask in current folder, w/o going into subfolders. Fix required to also fix existing warnings.In addition to that:
# noqa
toappend_output.sh
because multiline output string is also checked by flake8 (which is weird but it is current behavior discussed e.g. here)pytest
warning regarding example TestRunner class from facadesetup.py
to correspond to changes in #270 (+py37)