NVIDIA / spark-rapids-jni

RAPIDS Accelerator JNI For Apache Spark
Apache License 2.0
41 stars 66 forks source link

[FEA] Add checkstyle for building. #2591

Open liurenjie1024 opened 1 week ago

liurenjie1024 commented 1 week ago

Is your feature request related to a problem? Please describe. Currently we are missing format/style check during build.

Describe the solution you'd like checkstyle is among the most popular formatters/linters in java ecosystem, and we should add support for it during build process.

Describe alternatives you've considered

No.

Additional context No.

pxLi commented 6 days ago

JNI repo has the https://github.com/NVIDIA/spark-rapids-jni/blob/branch-24.12/.github/workflows/clang-format.yml

if we would like to have some check for java/scala code, please help provide the command or add the process to maven stages like plugin https://github.com/NVIDIA/spark-rapids/blob/branch-24.12/pom.xml#L1703-L1710 firstly @liurenjie1024 thanks