Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bumped up compat for HTTP.jl #23

Merged
merged 1 commit into from
Aug 18, 2022

Conversation

svilupp
Copy link
Contributor

@svilupp svilupp commented Aug 15, 2022

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, JuliaWeb/HTTP.jl#233)

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

@codecov-commenter
Copy link

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 deyandyankov merged commit 240ba46 into JuliaAI:main Aug 18, 2022
@deyandyankov
Copy link
Collaborator

Thank you for the PR!

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.

3 participants