-
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
docs: fix typo and some coding guidelines #2613
Conversation
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
@XavierChengZW thanks for you contribution. |
…e added @OverRide,a static field should be directly referred by its class name,members of a POJO class wrapper classes
Codecov Report
@@ Coverage Diff @@
## develop #2613 +/- ##
=============================================
- Coverage 51.18% 51.17% -0.02%
+ Complexity 2806 2804 -2
=============================================
Files 554 554
Lines 17772 17772
Branches 2099 2099
=============================================
- Hits 9096 9094 -2
Misses 7819 7819
- Partials 857 859 +2
|
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
…r#proceed
Ⅰ. Describe what this PR did
fix typo for io.seata.rm.tcc.interceptor.ActionInterceptorHandler#proceed
fix some coding guidelines analyze
fix io.seata.core.rpc.netty.RpcServerBootstrap#start A static field should be directly referred by its class name instead of its corresponding object name.
fix io.seata.core.store.db.sql.log.AbstractLogStoreSqls#getQueryGlobalMax,io.seata.core.store.db.sql.log.AbstractLogStoreSqls#getQueryBranchMax.An overridden method from an interface or abstract class must be marked with @OverRide annotation.
fix io.seata.core.store.GlobalTransactionDO. Members of a POJO class must be wrapper classes.
Ⅱ. Does this pull request fix one issue?
Not
Ⅲ. Why don't you add test cases (unit test/integration test)?
No validation required
Ⅳ. Describe how to verify it
No validation required
Ⅴ. Special notes for reviews
Not