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

Support HTTP 1 #18

Merged
merged 1 commit into from
Aug 16, 2023
Merged

Support HTTP 1 #18

merged 1 commit into from
Aug 16, 2023

Conversation

devmotion
Copy link
Member

This PR adds support for HTTP 1. Or rather, it bumps the compat entry but requires someone familiar with HTTP to carefully check whether HTTP 1 is actually supported or something in the code base has to be checked. It seems that unfortunately there are no actual tests, so one must not rely on CI tests passing successfully.

This updated was missed by the CompatHelper action since it was disabled due to inactivity: https://github.com/JuliaHealth/EHRAuthentication.jl/actions/workflows/CompatHelper.yml

@codecov
Copy link

codecov bot commented Aug 14, 2023

Codecov Report

Merging #18 (8165b7b) into main (ba6c4ea) will increase coverage by 5.00%.
The diff coverage is n/a.

@@           Coverage Diff            @@
##            main     #18      +/-   ##
========================================
+ Coverage   0.00%   5.00%   +5.00%     
========================================
  Files          1       3       +2     
  Lines         38      40       +2     
========================================
+ Hits           0       2       +2     
  Misses        38      38              

see 2 files with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@andreasnoack andreasnoack merged commit 6e19221 into JuliaHealth:main Aug 16, 2023
6 checks passed
@devmotion devmotion deleted the dw/http branch August 16, 2023 09:33
@devmotion
Copy link
Member Author

devmotion commented Aug 16, 2023

@DilumAluthge @andreasnoack Could you re-enable the CompatHelper action?

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