Closed nickrobinson251 closed 3 years ago
Merging #108 (0a08047) into master (dbf7be6) will increase coverage by
0.11%
. The diff coverage is100.00%
.:exclamation: Current head 0a08047 differs from pull request most recent head 7c75b90. Consider uploading reports for the commit 7c75b90 to get more accurate results
@@ Coverage Diff @@
## master #108 +/- ##
==========================================
+ Coverage 97.56% 97.67% +0.11%
==========================================
Files 6 6
Lines 205 215 +10
==========================================
+ Hits 200 210 +10
Misses 5 5
Impacted Files | Coverage Δ | |
---|---|---|
src/JLSOFile.jl | 93.33% <100.00%> (+2.85%) |
:arrow_up: |
src/file_io.jl | 94.44% <100.00%> (+0.10%) |
: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 dbf7be6...7c75b90. Read the comment docs.
keys
,haskey
,get
andget!
forJLSOFile
(but notiterate
as i didn't have a use-cae for it); closes #107JLSOFile()
to return aJLSOFile
, whereas onmaster
it errrors:save(io, ::JLSOFile)
methodTogether, this allows users to write code like
or
The main advantage is this allows working with a
JLSOFile
similar to working with the Dict that would be returned byJLSO.load(filename)
, but withJLSOFile
lazily deserialising (unlikeload
)