lizining1231 opened a new pull request, #3607:
URL: https://github.com/apache/dubbo-go/pull/3607
### Description
Fixes #3598 (task 24)
Unify the comment style in the Triple mainline( protocol/triple/client.go ,
protocol/triple/server.go ,
protocol/triple/triple_protocol/handler.go ). This is a comment-only change
with no behavior impact.
### Changes
- Remove stale and duplicate draft TODOs: two dangling client TODOs
(already implemented), a duplicated opentracing TODO, and a TODO
referencing the non-existent handleService.
- Fix a malformed TODO: // todo:// open tracing -> // TODO: support
OpenTracing.
- Rewrite the "Enrich http2/http3 transport config" TODOs into real
transport config descriptions.
- Comment capitalization: capitalize the first letter of inline comments
(including end-of-line comments) across the three files.
- TODO capitalization: normalize todo to all-caps TODO with a colon,
keeping author tags (e.g. TODO(DMwangnima):).
- Term capitalization: idl -> IDL, tls -> TLS, opentracing -> OpenTracing.
- Comment formatting fixes: missing space after //,
tailer -> trailer typo, and consistent Compatibility: notes.
### Checklist
- [x] I confirm the target branch is `develop`
- [x] Code has passed local testing
- [ ] I have added tests that prove my fix is effective or that my feature
works
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
