Skip to content
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

fix hugegraph-api code checkstyle #1851

Merged
merged 1 commit into from
Apr 26, 2022
Merged

fix hugegraph-api code checkstyle #1851

merged 1 commit into from
Apr 26, 2022

Conversation

Fizell
Copy link
Contributor

@Fizell Fizell commented Apr 25, 2022

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

Copy link
Member

@imbajin imbajin left a comment

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

@codecov
Copy link

codecov bot commented Apr 25, 2022

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
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants