-
Notifications
You must be signed in to change notification settings - Fork 716
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
improve: support js,wasm #683
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@shynome Can you please add test case?
I'm sorry, I don't have time to add test case because I don't use resty anymore. as you can see, those code is copy from golang std library |
@shynome Thanks for getting back. I see; I will take it up later on. |
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## master #683 +/- ##
==========================================
- Coverage 95.73% 95.63% -0.11%
==========================================
Files 11 12 +1
Lines 1571 1603 +32
==========================================
+ Hits 1504 1533 +29
- Misses 41 42 +1
- Partials 26 28 +2
☔ View full report in Codecov by Sentry. |
those code copy from
/go/src/net/http/transport_default_other.go
closes #610