Closed rafaqz closed 4 months ago
Attention: Patch coverage is 78.41410%
with 49 lines
in your changes are missing coverage. Please review.
Project coverage is 84.07%. Comparing base (
f5468f2
) to head (607995f
).
Files | Patch % | Lines |
---|---|---|
src/Dimensions/coord.jl | 0.00% | 11 Missing :warning: |
src/Lookups/lookup_arrays.jl | 83.33% | 11 Missing :warning: |
src/Lookups/selector.jl | 80.39% | 10 Missing :warning: |
src/Dimensions/utils.jl | 0.00% | 4 Missing :warning: |
src/Lookups/predicates.jl | 62.50% | 3 Missing :warning: |
src/Lookups/set.jl | 76.92% | 3 Missing :warning: |
src/Dimensions/dimension.jl | 90.00% | 1 Missing :warning: |
src/Dimensions/predicates.jl | 80.00% | 1 Missing :warning: |
src/Dimensions/show.jl | 50.00% | 1 Missing :warning: |
src/Lookups/indexing.jl | 75.00% | 1 Missing :warning: |
... and 3 more |
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
I've wanted to do this for ages: the two sub-modules are now
Dimensions
andLookups
, notDimensions
andLookupArrays
.Having to type the
Array
part is annoying, and we say "lookup" and the getter islookup
, so the module may as well beLookups
and the supertypeLookup
.Its not actually breaking, as there are still aliases for the old names.