Skip to content

fix hugegraph-api code checkstyle - #1851

Merged
javeme merged 1 commit into
apache:masterfrom
chuanxo:fix-style2
Apr 26, 2022
Merged

fix hugegraph-api code checkstyle#1851
javeme merged 1 commit into
apache:masterfrom
chuanxo:fix-style2

Conversation

@chuanxo

@chuanxo chuanxo commented Apr 25, 2022

Copy link
Copy Markdown
Contributor

a new PR from #1845
fix only the obvious code style issues :)

imbajin
imbajin approved these changes Apr 25, 2022
View reviewed changes

@imbajin imbajin left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks a lot for quick split

@chuanxo chuanxo mentioned this pull request Apr 25, 2022
Closed
@codecov

codecov Bot commented Apr 25, 2022 •
edited
Loading

Copy link
Copy Markdown

Codecov Report

Merging #1851 (59e641a) into master (85b01b2) will increase coverage by 6.29%.
The diff coverage is 84.61%.

@@             Coverage Diff              @@
##             master    #1851      +/-   ##
============================================
+ Coverage     64.45%   70.75%   +6.29%     
- Complexity      914      982      +68     
============================================
  Files           447      447              
  Lines         38040    38041       +1     
  Branches       5389     5389              
============================================
+ Hits          24520    26915    +2395     
+ Misses        11000     8493    -2507     
- Partials       2520     2633     +113
Impacted Files Coverage Δ
.../java/com/baidu/hugegraph/api/auth/ProjectAPI.java 82.05% <ø> (ø)
...va/com/baidu/hugegraph/auth/HugeAuthenticator.java 40.21% <ø> (ø)
...a/com/baidu/hugegraph/license/LicenseVerifier.java 0.00% <0.00%> (ø)
.../java/com/baidu/hugegraph/metrics/MetricsUtil.java 62.50% <66.66%> (ø)
...api/src/main/java/com/baidu/hugegraph/api/API.java 74.11% <70.00%> (-0.89%) ⬇️
.../com/baidu/hugegraph/server/ApplicationConfig.java 93.47% <75.00%> (ø)
...n/java/com/baidu/hugegraph/api/graph/BatchAPI.java 27.41% <85.71%> (ø)
...va/com/baidu/hugegraph/api/gremlin/GremlinAPI.java 75.00% <100.00%> (ø)
...n/java/com/baidu/hugegraph/api/job/GremlinAPI.java 50.72% <100.00%> (ø)
.../com/baidu/hugegraph/api/schema/IndexLabelAPI.java 76.25% <100.00%> (ø)
... and 97 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 85b01b2...59e641a. Read the comment docs.

@javeme
javeme merged commit be7c083 into apache:master Apr 26, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@javeme javeme javeme approved these changes
@imbajin imbajin imbajin approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@chuanxo @javeme @imbajin