Closed fghaas closed 5 years ago
Merging #28 into master will increase coverage by
1.82%
. The diff coverage is87.83%
.
@@ Coverage Diff @@
## master #28 +/- ##
==========================================
+ Coverage 85.21% 87.03% +1.82%
==========================================
Files 6 6
Lines 284 324 +40
==========================================
+ Hits 242 282 +40
Misses 42 42
Impacted Files | Coverage Δ | |
---|---|---|
tests/test_cli.py | 100% <100%> (ø) |
:arrow_up: |
tests/test_helpers.py | 100% <100%> (ø) |
:arrow_up: |
olxutils/templates.py | 100% <100%> (ø) |
:arrow_up: |
olxutils/helpers.py | 73.68% <60%> (+0.71%) |
:arrow_up: |
olxutils/cli.py | 68% <83.33%> (+7.5%) |
: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 f490776...fa44ad2. Read the comment docs.
Eventually, we're going to want
olx
to do more than just create a new run. Thus, implement subcommand functionality and make a few more related changes:olx-new-run
intoolx new-run
, retaining the previous syntax as a compatibility alias.new_run.py
, but add a user-facing FutureWarning.