-
Notifications
You must be signed in to change notification settings - Fork 8.7k
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
replace Chinese comments #1036
replace Chinese comments #1036
Conversation
Codecov Report
@@ Coverage Diff @@
## develop #1036 +/- ##
=========================================
Coverage 38.5% 38.5%
Complexity 1076 1076
=========================================
Files 220 220
Lines 8774 8774
Branches 1108 1108
=========================================
Hits 3378 3378
Misses 4966 4966
Partials 430 430
Continue to review full report at Codecov.
|
common/src/main/java/io/seata/common/exception/FrameworkErrorCode.java
Outdated
Show resolved
Hide resolved
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
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.
/** | ||
* The Net disconnect. | ||
*/ | ||
NetDisconnect("0103", "seataConnection closed", "网络断开,请检查到对端(client 或seata server)的网络连接"), | ||
NetDisconnect("0103", "Seata Connection closed", "The network is disconnected. Please check the network connection to the client or seata server."), |
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.
Connection -> connection ?
/** | ||
* The Net disconnect. | ||
*/ | ||
NetDisconnect("0103", "seataConnection closed", "网络断开,请检查到对端(client 或seata server)的网络连接"), | ||
NetDisconnect("0103", "Seata Connection closed", "The network is disconnected. Please check the network connection to the client or seata server."), |
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.
Connection -> connection
…xuzhiyiYoh/seata into enhance/remove_chinese_comments
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.
thanks.LGTM.
Ⅰ. Describe what this PR did
Replace some Chinese comments.
Ⅱ. Does this pull request fix one issue?
#732
Ⅲ. Why don't you add test cases (unit test/integration test)?
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews