This is an automated email from the ASF dual-hosted git repository.
dongjoon pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/spark.git
The following commit(s) were added to refs/heads/master by this push:
new 7f81bd7436fe [SPARK-54242][BUILD] Skip `Checkstyle` if
`NOLINT_ON_COMPILE` is true
7f81bd7436fe is described below
commit 7f81bd7436feb31ed6e4c61bc71ad8580fa5ad66
Author: Dongjoon Hyun <[email protected]>
AuthorDate: Fri Nov 7 09:59:32 2025 -0800
[SPARK-54242][BUILD] Skip `Checkstyle` if `NOLINT_ON_COMPILE` is true
### What changes were proposed in this pull request?
This PR aims to skip `Checkstyle` if `NOLINT_ON_COMPILE` is true.
### Why are the changes needed?
Like `Scalastyle`, `Checkstyle` is also a kind of linting. So, we had
better skip it when a user set `NOLINT_ON_COMPILE` to `true`.
### Does this PR introduce _any_ user-facing change?
No Spark behavior change because this is only a build change.
### How was this patch tested?
Pass the CIs and manually.
```
$ NOLINT_ON_COMPILE=true build/sbt package | grep checkstyle
```
### Was this patch authored or co-authored using generative AI tooling?
No.
Closes #52943 from dongjoon-hyun/SPARK-54242.
Authored-by: Dongjoon Hyun <[email protected]>
Signed-off-by: Dongjoon Hyun <[email protected]>
---
project/SparkBuild.scala | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/project/SparkBuild.scala b/project/SparkBuild.scala
index e06ddf826f38..52701d886c8e 100644
--- a/project/SparkBuild.scala
+++ b/project/SparkBuild.scala
@@ -393,7 +393,8 @@ object SparkBuild extends PomBuild {
/* Enable shared settings on all projects */
(allProjects ++ optionallyEnabledProjects ++ assemblyProjects ++
copyJarsProjects ++ Seq(spark, tools))
.foreach(enable(sharedSettings ++ DependencyOverrides.settings ++
- ExcludedDependencies.settings ++ Checkstyle.settings ++
ExcludeShims.settings))
+ ExcludedDependencies.settings ++ (if (noLintOnCompile) Nil else
Checkstyle.settings) ++
+ ExcludeShims.settings))
/* Enable tests settings for all projects except examples, assembly and
tools */
(allProjects ++
optionallyEnabledProjects).foreach(enable(TestSettings.settings))
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
