-
Notifications
You must be signed in to change notification settings - Fork 8.9k
feature: support http put #5377
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
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 #5377 +/- ##
============================================
- Coverage 48.50% 48.48% -0.03%
+ Complexity 4303 4302 -1
============================================
Files 775 775
Lines 27397 27407 +10
Branches 3413 3414 +1
============================================
- Hits 13290 13289 -1
- Misses 12701 12713 +12
+ Partials 1406 1405 -1
|
slievrly
left a comment
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
slievrly
left a comment
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.
Have completed! |
good job. |
Ⅰ. 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