-
Notifications
You must be signed in to change notification settings - Fork 98
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
server: rewrite package comments in English #96
Conversation
server/server.go
Outdated
@@ -3,7 +3,8 @@ | |||
// If you have downloaded a copy of the tRPC source code from Tencent, | |||
// please note that tRPC source code is licensed under the Apache 2.0 License that can be found in the LICENSE file. | |||
|
|||
// Package server 服务端,包括网络通信 名字服务 监控统计 链路跟踪等各个组件基础接口,具体实现由第三方 middleware 注册进来 | |||
// Package server provides a framework for managing multiple services within a single process. | |||
// A server process may listen on multiple ports, providing different business services on different ports. |
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.
Don't mention business at all.
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.
fixed
Codecov ReportAll modified lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #96 +/- ##
==========================================
+ Coverage 85.59% 85.63% +0.04%
==========================================
Files 188 188
Lines 16114 16114
==========================================
+ Hits 13793 13800 +7
+ Misses 1760 1756 -4
+ Partials 561 558 -3
Flags with carried forward coverage won't be shown. Click here to find out more.
☔ View full report in Codecov by Sentry. |
and fix CONTRIBUTING.zh_CN.md
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
No description provided.