Skip to content

Conversation

@ranocha
Copy link
Contributor

@ranocha ranocha commented Aug 26, 2022

This extends #138. Please see that PR first and help me understand things better if I am not following the correct approach.

@codecov
Copy link

codecov bot commented Aug 26, 2022

Codecov Report

Merging #139 (b261c27) into master (5606269) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master     #139   +/-   ##
=======================================
  Coverage   97.29%   97.29%           
=======================================
  Files           4        4           
  Lines         775      775           
=======================================
  Hits          754      754           
  Misses         21       21           
Impacted Files Coverage Δ
src/extract.jl 98.11% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@ranocha
Copy link
Contributor Author

ranocha commented Aug 30, 2022

I suggest the labels latency and backport-1.8 for this PR.

@ranocha
Copy link
Contributor Author

ranocha commented Sep 1, 2022

Bump. Can we get this into Julia v1.8.1?

1 similar comment
@ranocha
Copy link
Contributor Author

ranocha commented Sep 5, 2022

Bump. Can we get this into Julia v1.8.1?

@staticfloat staticfloat merged commit c887de7 into JuliaIO:master Sep 6, 2022
@ranocha ranocha deleted the patch-2 branch September 6, 2022 17:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants