Closed hhsecond closed 4 years ago
Merging #37 into master will increase coverage by
0.43%
. The diff coverage is92.98%
.
@@ Coverage Diff @@
## master #37 +/- ##
==========================================
+ Coverage 91.53% 91.96% +0.43%
==========================================
Files 21 21
Lines 685 759 +74
Branches 51 55 +4
==========================================
+ Hits 627 698 +71
Misses 47 47
- Partials 11 14 +3
Impacted Files | Coverage Δ | |
---|---|---|
stockroom/storages/data.py | 83.33% <ø> (ø) |
|
stockroom/core.py | 82.85% <84.37%> (+5.77%) |
:arrow_up: |
stockroom/cli.py | 84.00% <87.50%> (-1.92%) |
:arrow_down: |
stockroom/storages/model.py | 84.61% <95.00%> (+1.48%) |
:arrow_up: |
stockroom/storages/experiment.py | 85.36% <100.00%> (+1.15%) |
:arrow_up: |
tests/conftest.py | 100.00% <100.00%> (ø) |
|
tests/test_cli.py | 100.00% <100.00%> (ø) |
|
tests/test_data.py | 100.00% <100.00%> (ø) |
|
tests/test_model.py | 100.00% <100.00%> (ø) |
|
tests/test_multiple_instances.py | 100.00% <100.00%> (ø) |
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 9a21522...584d665. Read the comment docs.
This pull request introduces 1 alert when merging 2f0fa8459ea187cfd3631c7452d83ca46cb4c845 into 9a21522296254e70f32347891e098d1724093dc1 - view on LGTM.com
new alerts:
Description
Why is this change required? What problem does it solve? Describe your changes in detail:
Managing the write and read enable stock object is not seamless. This is a reasonable attempt to solve the UX
Types of changes
What types of changes does your code introduce? Put an
x
in all the boxes that apply:Checklist
List of items to check before submitting your PR. Note that this are only for helping you remember what you might (also) need to do and are not mandatory requirements