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
Fix the API for adding decorated ServiceWithPathMappings
and update…
#1627
Conversation
… documentation Motivation: We recently received a good question about how to decorate a `GrpcService`, which is a `ServiceWithPathMappings`. I found there is not enough explanation about such a question and it is currently not easy to specify multiple decorators for a `ServiceWithPathMappings`. Modifications: - Allow specifying multiple decorators when registering a `ServiceWithPathMappings`. - Add some sections about decorating `ServiceWithPathMappings` and `GrpcService`. Result: - Easier to decorate a `ServiceWithPathMappings` such as `GrpcService`. - Better documentation.
@perlun Looking forward to your feed back on the documentation. |
Codecov Report
@@ Coverage Diff @@
## master #1627 +/- ##
============================================
- Coverage 73.15% 72.71% -0.44%
+ Complexity 7869 7861 -8
============================================
Files 714 716 +2
Lines 31189 31331 +142
Branches 3825 3841 +16
============================================
- Hits 22816 22783 -33
- Misses 6415 6592 +177
+ Partials 1958 1956 -2
Continue to review full report at Codecov.
|
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
LGTM one more. 😄 |
Thanks for reviewing. |
line#1627) Motivation: We recently received a good question about how to decorate a `GrpcService`, which is a `ServiceWithPathMappings`. I found there is not enough explanation about such a question and it is currently not easy to specify multiple decorators for a `ServiceWithPathMappings`. Modifications: - Allowed specifying multiple decorators when registering a `ServiceWithPathMappings`. - Added some sections about decorating `ServiceWithPathMappings` and `GrpcService`. - Miscellaneous: - Added some missing imports. - Made indentation prettier in some examples. Result: - Easier to decorate a `ServiceWithPathMappings` such as `GrpcService`. - Better documentation.
… documentation
Motivation:
We recently received a good question about how to decorate a
GrpcService
, which is aServiceWithPathMappings
.I found there is not enough explanation about such a question and it is
currently not easy to specify multiple decorators for a
ServiceWithPathMappings
.Modifications:
ServiceWithPathMappings
.ServiceWithPathMappings
andGrpcService
.Result:
ServiceWithPathMappings
such asGrpcService
.