Closed harshangrjn closed 5 years ago
@kaarthiksundar Does this look good or anything I am missing?
The fix is fine.
Merging #124 into master will not change coverage. The diff coverage is
87.5%
.
@@ Coverage Diff @@
## master #124 +/- ##
=======================================
Coverage 77.26% 77.26%
=======================================
Files 15 15
Lines 3132 3132
=======================================
Hits 2420 2420
Misses 712 712
Impacted Files | Coverage Δ | |
---|---|---|
src/operators.jl | 74.82% <87.5%> (ø) |
: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 b197004...7ebe18f. Read the comment docs.
Merging #124 into master will not change coverage. The diff coverage is
87.5%
.
@@ Coverage Diff @@
## master #124 +/- ##
=======================================
Coverage 77.26% 77.26%
=======================================
Files 15 15
Lines 3132 3132
=======================================
Hits 2420 2420
Misses 712 712
Impacted Files | Coverage Δ | |
---|---|---|
src/operators.jl | 74.82% <87.5%> (ø) |
: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 b197004...7ebe18f. Read the comment docs.
Fix for #123