-
Notifications
You must be signed in to change notification settings - Fork 8.8k
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
feature: support http put #5070
Conversation
Codecov Report
📣 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 @@
## 2.x #5070 +/- ##
============================================
- Coverage 48.95% 48.95% -0.01%
Complexity 4168 4168
============================================
Files 741 741
Lines 26521 26531 +10
Branches 3294 3295 +1
============================================
+ Hits 12984 12987 +3
- Misses 12129 12140 +11
+ Partials 1408 1404 -4
|
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.
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.
LGTM
@zsp419 thanks for your contribution, can you submit a same pr to branch 2.x? |
Currently can only merge into develop due to code conflicts. |
ok,I'll take care of it soon |
thank you. |
#5377 已重新提交pr |
Ⅰ. Describe what this PR did
使AbstractHttpExecutor类支持PUT方式的请求
Ⅱ. Does this pull request fix one issue?
fixes #5067
Ⅲ. Why don't you add test cases (unit test/integration test)?
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews