Closed chrisjsewell closed 3 years ago
Merging #112 (34dd190) into develop (a2561a4) will not change coverage. The diff coverage is
100.00%
.:exclamation: Current head 34dd190 differs from pull request most recent head a92c695. Consider uploading reports for the commit a92c695 to get more accurate results
@@ Coverage Diff @@
## develop #112 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 7 7
Lines 1541 1544 +3
=========================================
+ Hits 1541 1544 +3
Impacted Files | Coverage Δ | |
---|---|---|
disk_objectstore/__init__.py | 100.00% <100.00%> (ø) |
|
disk_objectstore/container.py | 100.00% <100.00%> (ø) |
|
disk_objectstore/examples/example_objectstore.py | 100.00% <100.00%> (ø) |
|
disk_objectstore/examples/profile_zeros.py | 100.00% <100.00%> (ø) |
|
disk_objectstore/models.py | 100.00% <100.00%> (ø) |
|
disk_objectstore/utils.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 a2561a4...a92c695. Read the comment docs.
One general question I still have is why we are changing yapf to black in this repo.
Oh they will all be getting changed: yapf was picked for whatever reason historically, but now black is easily the de facto standard; https://python.libhunt.com/compare-yapf-vs-black (and from a technical standpoint, I can tell you the way black parses the code is a lot more sound)
setup.cfg
andpyproject.toml
Manifest.in
,tox.ini
requirements.txt
/dev-requirements.txt
withrequirements.lock
dev
extra todev
&examples