Open jishnub opened 1 year ago
Merging #323 (37b2701) into master (5bf955c) will increase coverage by
1.35%
. The diff coverage is100.00%
.:exclamation: Current head 37b2701 differs from pull request most recent head 13eeb03. Consider uploading reports for the commit 13eeb03 to get more accurate results
@@ Coverage Diff @@
## master #323 +/- ##
==========================================
+ Coverage 96.45% 97.81% +1.35%
==========================================
Files 5 5
Lines 451 457 +6
==========================================
+ Hits 435 447 +12
+ Misses 16 10 -6
Impacted Files | Coverage Δ | |
---|---|---|
src/OffsetArrays.jl | 97.27% <100.00%> (-1.02%) |
:arrow_down: |
src/axes.jl | 98.92% <100.00%> (-1.08%) |
:arrow_down: |
src/precompile.jl | 100.00% <0.00%> (+100.00%) |
:arrow_up: |
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more
This PR mainly forwards
Base.IteratorSize(::IdOffsetRange)
to the parent, which allows us to potentially support infinite ranges as well. Infinite ranges are a bit of a sticky area, because currently inInifniteArrays
, the last element of such a range is infinite, which usually has a different type from theeltype
of the range. Nonetheless, it might be possible to support some operations.I've also removed the
test/Project.toml
, as it's not clear if this was much of a help