Skip to content

Commit ba110f2

Browse files
committed
[KYUUBI #7287] Fix checkstyle workflow
### Why are the changes needed? Fix checkstyle workflow by adding `-am` on compiling special module ### How was this patch tested? Pass GHA. ### Was this patch authored or co-authored using generative AI tooling? No. Closes #7287 from pan3793/style. Closes #7287 a8ec1f3 [Cheng Pan] java17 65ae8cf [Cheng Pan] Fix checkstyle workflow Authored-by: Cheng Pan <chengpan@apache.org> Signed-off-by: Cheng Pan <chengpan@apache.org> (cherry picked from commit 0ef4201) Signed-off-by: Cheng Pan <chengpan@apache.org>
1 parent 8ec6539 commit ba110f2

1 file changed

Lines changed: 9 additions & 9 deletions

File tree

.github/workflows/style.yml

Lines changed: 9 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -40,11 +40,11 @@ jobs:
4040
- uses: actions/checkout@v4
4141
with:
4242
fetch-depth: 0
43-
- name: Setup JDK 8
43+
- name: Setup JDK 17
4444
uses: actions/setup-java@v4
4545
with:
4646
distribution: temurin
47-
java-version: 8
47+
java-version: 17
4848
cache: 'maven'
4949
- name: Setup Maven
5050
uses: ./.github/actions/setup-maven
@@ -63,13 +63,13 @@ jobs:
6363
# for more details, please visit: https://github.com/actions/toolkit/issues/399#issuecomment-666147717
6464
if: steps.modules-check.conclusion == 'success' && steps.modules-check.outcome == 'failure'
6565
run: |
66-
export MAVEN_ARGS="-DskipTests -Dorg.slf4j.simpleLogger.defaultLogLevel=warn -Drat.skip=true -Dmaven.scalastyle.skip=true -Dspotless.check.skip=true"
67-
build/mvn clean install -Pflink-provided,hive-provided,spark-provided,tpcds
68-
build/mvn clean install -pl extensions/spark/kyuubi-extension-spark-3-3 -Pspark-3.3
69-
build/mvn clean install -pl extensions/spark/kyuubi-extension-spark-3-4 -Pspark-3.4
70-
build/mvn clean install -pl extensions/spark/kyuubi-extension-spark-3-5,extensions/spark/kyuubi-spark-connector-hive -Pspark-3.5
71-
build/mvn clean install -pl extensions/spark/kyuubi-extension-spark-4-0 -Pspark-4.0 -Pscala-2.13
72-
build/mvn clean install -pl extensions/spark/kyuubi-extension-spark-4-1 -Pspark-4.1 -Pscala-2.13
66+
export MAVEN_ARGS="-DskipTests -Dorg.slf4j.simpleLogger.defaultLogLevel=warn -Drat.skip=true -Dmaven.scalastyle.skip=true -Dspotless.check.skip=true -Pflink-provided,hive-provided,spark-provided"
67+
build/mvn clean install -Ptpcds
68+
build/mvn clean install -pl extensions/spark/kyuubi-extension-spark-3-3 -am -Pspark-3.3
69+
build/mvn clean install -pl extensions/spark/kyuubi-extension-spark-3-4 -am -Pspark-3.4
70+
build/mvn clean install -pl extensions/spark/kyuubi-extension-spark-3-5,extensions/spark/kyuubi-spark-connector-hive -am -Pspark-3.5
71+
build/mvn clean install -pl extensions/spark/kyuubi-extension-spark-4-0 -am -Pspark-4.0 -Pscala-2.13
72+
build/mvn clean install -pl extensions/spark/kyuubi-extension-spark-4-1 -am -Pspark-4.1 -Pscala-2.13
7373
7474
- name: Scalastyle with maven
7575
id: scalastyle-check

0 commit comments

Comments
 (0)