Skip to content

Conversation

lrennels
Copy link
Collaborator

@ckingdon95 can you approve this it's jus little fixes I found while working on the remove deprecations branch

@lrennels lrennels requested a review from corakingdon June 13, 2020 01:25
@codecov
Copy link

codecov bot commented Jun 13, 2020

Codecov Report

Merging #714 into master will not change coverage.
The diff coverage is 100.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #714   +/-   ##
=======================================
  Coverage   79.73%   79.73%           
=======================================
  Files          38       38           
  Lines        2877     2877           
=======================================
  Hits         2294     2294           
  Misses        583      583           
Flag Coverage Δ
#unittests 79.73% <100.00%> (ø)
Impacted Files Coverage Δ
src/core/time_arrays.jl 78.12% <100.00%> (ø)

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 c6647a6...6fd8e7c. Read the comment docs.

@lrennels lrennels merged commit a8eee27 into master Jun 15, 2020
@lrennels lrennels deleted the bugfix branch June 15, 2020 18:13
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