search

apache / incubator-hugegraph-commons

(Archived Warning)Please visit "apache/hugegraph" repo instead
https://github.com/apache/hugegraph/tree/master/hugegraph-commons
Apache License 2.0
29 stars 45 forks source link

refact: address some code alert #115

Closed imbajin closed 1 year ago

imbajin commented 1 year ago

this PR address all the warning from check-code-style plugin's

Other contributors could also handle them (just run maven compile and check the warning message) like below:

image

TODO: fix it also in toolchain/server

codecov[bot] commented 1 year ago

Codecov Report

Merging #115 (68bbfa3) into master (952975a) will not change coverage. The diff coverage is 100.00%.

@@            Coverage Diff            @@
##             master     #115   +/-   ##
=========================================
  Coverage     93.15%   93.15%           
  Complexity       65       65           
=========================================
  Files             9        9           
  Lines           263      263           
  Branches         22       22           
=========================================
  Hits            245      245           
  Misses            8        8           
  Partials         10       10
Impacted Files Coverage Δ
...n/java/org/apache/hugegraph/config/RpcOptions.java 97.22% <ø> (ø)
...va/org/apache/hugegraph/rpc/RpcClientProvider.java 90.90% <ø> (ø)
...va/org/apache/hugegraph/rpc/RpcConsumerConfig.java 91.48% <100.00%> (ø)

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more