JuliaAI / MLFlowClient.jl

Julia client for MLFlow.
https://juliaai.github.io/MLFlowClient.jl/
MIT License
42 stars 8 forks source link

Bumped up compat for HTTP.jl #23

Closed svilupp closed 1 year ago

svilupp commented 1 year ago

Would it be possible to increase the compat for HTTP.jl to 1.2? I've re-run the testing suite and all passes

Motivation: MLFlow-based workflows are more likely to have connection timeouts, which leads to a lot of HTTP.jl errors like IOError(EOFError()) (eg, https://github.com/JuliaWeb/HTTP.jl/issues/233)

There have been a lot of fixes in HTTP.lj since version 0.9 (see https://github.com/JuliaWeb/HTTP.jl/issues/903), so we would be able to benefit from that.

codecov-commenter commented 1 year ago

Codecov Report

Merging #23 (6fc2046) into main (f715b9e) will not change coverage. The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #23   +/-   ##
=======================================
  Coverage   70.25%   70.25%           
=======================================
  Files           4        4           
  Lines         279      279           
=======================================
  Hits          196      196           
  Misses         83       83           

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

deyandyankov commented 1 year ago

Thank you for the PR!