Closed abelcastilloavant closed 6 years ago
Well then - that broke a lot of tests.
Merging #16 into master will increase coverage by
0.34%
. The diff coverage is100%
.
@@ Coverage Diff @@
## master #16 +/- ##
==========================================
+ Coverage 88.88% 89.23% +0.34%
==========================================
Files 1 2 +1
Lines 63 65 +2
==========================================
+ Hits 56 58 +2
Misses 7 7
Impacted Files | Coverage Δ | |
---|---|---|
R/accessors.R | 100% <100%> (ø) |
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 69fe784...b3b5698. Read the comment docs.
Addresses https://github.com/kirillseva/cacher/issues/1
So yeah, overloading the
[[
operator is risky, but at least it's doable.Edit: less risky than I originally thought.