apache / incubator-hugegraph

A graph database that supports more than 100+ billion data, high performance and scalability (Include OLTP Engine & REST-API & Backends)
https://hugegraph.apache.org
Apache License 2.0
2.63k stars 517 forks source link

chore: migrate the `hg-style.xml` to `.editorconfig` #2561

Closed FrostyHec closed 3 months ago

FrostyHec commented 3 months ago

Purpose of the PR

Migrate the format defined in hg-style.xml to editorconfig for code style controlling. Editorconfig offers enhanced capabilities for defining coding conventions and can be automatically recognized and imported by IDEA.

Main Changes

Removed hugegraph-style.xml and define the code format control in editorconfig with same functionality.

Verifying these changes

image

Does this PR potentially affect the following parts?

Documentation Status

codecov[bot] commented 3 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 4.92%. Comparing base (1e97521) to head (34385c7). Report is 1 commits behind head on master.

:exclamation: Current head 34385c7 differs from pull request most recent head f65d056

Please upload reports for the commit f65d056 to get more accurate results.

:exclamation: There is a different number of reports uploaded between BASE (1e97521) and HEAD (34385c7). Click for more details.

HEAD has 5 uploads less than BASE | Flag | BASE (1e97521) | HEAD (34385c7) | |------|------|------| ||7|2|
Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #2561 +/- ## ============================================ - Coverage 56.89% 4.92% -51.98% + Complexity 827 479 -348 ============================================ Files 612 590 -22 Lines 49672 47154 -2518 Branches 6681 6233 -448 ============================================ - Hits 28261 2321 -25940 - Misses 18603 44646 +26043 + Partials 2808 187 -2621 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

FrostyHec commented 3 months ago

Looks fine~ Thanks for your contribution

PS: Do u test it in the IDEA env?

Thanks~, I have test some cases in IDEA and use the checkstyle in project for verification.