JuliaAI / MLJScikitLearnInterface.jl

MLJ Interface for ScikitLearn.jl
Other
12 stars 6 forks source link

Fix CI failure #51

Closed OkonSamuel closed 1 year ago

OkonSamuel commented 1 year ago

Note: Remember to squash merge.

closes #42

codecov-commenter commented 1 year ago

Codecov Report

Merging #51 (04370c8) into dev (36882f1) will not change coverage. The diff coverage is n/a.

:exclamation: Current head 04370c8 differs from pull request most recent head 0a8242d. Consider uploading reports for the commit 0a8242d to get more accurate results

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more

@@           Coverage Diff           @@
##              dev      #51   +/-   ##
=======================================
  Coverage   73.30%   73.30%           
=======================================
  Files          12       12           
  Lines         206      206           
=======================================
  Hits          151      151           
  Misses         55       55           

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

OkonSamuel commented 1 year ago

@ablaom. This should fix the ci failure. We no longer need the hack as the Julia issue linked in #42 has been closed. The failures left are unrelated to ci and are fixed in #50