Closed ablaom closed 4 years ago
Merging #79 into master will increase coverage by
1.59%
. The diff coverage is78.37%
.
@@ Coverage Diff @@
## master #79 +/- ##
==========================================
+ Coverage 88.49% 90.09% +1.59%
==========================================
Files 8 8
Lines 226 222 -4
==========================================
Hits 200 200
+ Misses 26 22 -4
Impacted Files | Coverage Δ | |
---|---|---|
src/ScientificTypes.jl | 85.71% <ø> (ø) |
:arrow_up: |
src/autotype.jl | 97.91% <100%> (ø) |
:arrow_up: |
src/conventions/mlj/finite.jl | 93.75% <100%> (+9.96%) |
:arrow_up: |
src/schema.jl | 81.39% <100%> (ø) |
:arrow_up: |
src/conventions/mlj/mlj.jl | 90.32% <57.14%> (+0.32%) |
:arrow_up: |
src/scitype.jl | 75% <61.53%> (ø) |
: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 cdc585b...25e70f6. Read the comment docs.
Context: #78
Note: According to local testing, these changes do not break tests for current dev branches of MLJBase, MLJModels, or MLJ. So these could safely update their ScientificTypes version requirements to "^0.3, ^0.4" and make patch releases (MLJ, MLJBase should also re-export
Textual
.)I have not tested MLJTutorials.