-
Notifications
You must be signed in to change notification settings - Fork 422
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
feat: add adapter for hertz #472
Conversation
c3bb9d8
to
66f0e91
Compare
Codecov Report
@@ Coverage Diff @@
## master #472 +/- ##
=======================================
Coverage 53.17% 53.17%
=======================================
Files 91 91
Lines 5896 5896
=======================================
Hits 3135 3135
Misses 2414 2414
Partials 347 347 Continue to review full report at Codecov.
|
pkg/adapters/hertz/.gitignore
Outdated
@@ -0,0 +1,66 @@ | |||
# JetBrains template |
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.
gitignore is not required, it is already available in the main project
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.
GET
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
Nice work. Thanks for contributing! |
Describe what this PR does / why we need it
cloudwego/hertz#138
Does this pull request fix one issue?
cloudwego/hertz#138
Describe how you did it
The implementation is based on hertz middlewares for http server and client.
They have been tested in
server_test.go
andclient_test.go
Describe how to verify it
The implementation can be verified via
go test
.Special notes for reviews