Skip to content

Commit 4ba6f86

Browse files
authored
ci: run clirr on auth-library, sdk-platform-java, handwritten libraries (#12894)
Fixes #12893
1 parent 1e633d7 commit 4ba6f86

4 files changed

Lines changed: 77 additions & 2 deletions

File tree

.github/workflows/ci.yaml

Lines changed: 21 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -246,8 +246,28 @@ jobs:
246246
JOB_TYPE: test
247247
JOB_NAME: units-8-runtime-${{matrix.java}}
248248
working-directory: ${{matrix.package}}
249+
split-clirr:
250+
runs-on: ubuntu-latest
251+
needs: changes
252+
strategy:
253+
matrix:
254+
package: ${{ fromJSON(needs.changes.outputs.packages) }}
255+
steps:
256+
- name: Get current week within the year
257+
id: date
258+
run: echo "::set-output name=week_of_year::$(date +'%W' --utc)"
259+
- uses: actions/checkout@v4
260+
- uses: actions/setup-java@v4
261+
with:
262+
distribution: temurin
263+
java-version: 11
264+
- run: .kokoro/build.sh
265+
env:
266+
BUILD_SUBDIR: ${{matrix.package}}
267+
JOB_TYPE: clirr
268+
JOB_NAME: clirr-${{matrix.package}}
249269
required:
250-
needs: [ changes, split-units ]
270+
needs: [ changes, split-units, split-clirr ]
251271
name: conditional-required-check
252272
if: ${{ always() }} # Always run even if any "needs" jobs fail
253273
runs-on: ubuntu-22.04

.github/workflows/google-auth-library-java-ci.yaml

Lines changed: 17 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -55,8 +55,24 @@ jobs:
5555
BUILD_SUBDIR: google-auth-library-java
5656
JOB_TYPE: test
5757
SUREFIRE_JVM_OPT: "-P '!slf4j2x,slf4j2x-test'"
58+
clirr:
59+
needs: filter
60+
if: ${{ needs.filter.outputs.library == 'true' }}
61+
runs-on: ubuntu-latest
62+
steps:
63+
- uses: actions/checkout@v3
64+
- uses: actions/setup-java@v3
65+
with:
66+
distribution: temurin
67+
java-version: 11
68+
- run: java -version
69+
- run: .kokoro/build.sh
70+
env:
71+
JOB_TYPE: clirr
72+
BUILD_SUBDIR: google-auth-library-java
73+
5874
required:
59-
needs: [ units-logging ]
75+
needs: [ units-logging, clirr ]
6076
name: conditional-required-check
6177
if: ${{ always() }} # Always run even if any "needs" jobs fail
6278
runs-on: ubuntu-22.04

.github/workflows/sdk-platform-java-ci.yaml

Lines changed: 16 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -53,6 +53,22 @@ jobs:
5353
GOOGLE_SDK_JAVA_LOGGING: true
5454
working-directory: sdk-platform-java
5555

56+
clirr:
57+
needs: filter
58+
if: ${{ needs.filter.outputs.library == 'true' }}
59+
runs-on: ubuntu-22.04
60+
steps:
61+
- uses: actions/checkout@v3
62+
- uses: actions/setup-java@v3
63+
with:
64+
distribution: temurin
65+
java-version: 11
66+
- run: java -version
67+
- run: .kokoro/build.sh
68+
env:
69+
JOB_TYPE: clirr
70+
BUILD_SUBDIR: sdk-platform-java
71+
5672
sdk-platform-java-8:
5773
needs: filter
5874
if: ${{ needs.filter.outputs.library == 'true' }}

.kokoro/build.sh

Lines changed: 23 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -305,6 +305,29 @@ case ${JOB_TYPE} in
305305
popd
306306
fi
307307
;;
308+
clirr)
309+
if [[ -n "${BUILD_SUBDIR}" ]]
310+
then
311+
echo "Compiling and building all modules for ${BUILD_SUBDIR}"
312+
install_modules "${BUILD_SUBDIR}"
313+
echo "Running in subdir: ${BUILD_SUBDIR}"
314+
pushd "${BUILD_SUBDIR}"
315+
EXTRA_PROFILE_OPTS=()
316+
else
317+
install_modules "sdk-platform-java"
318+
fi
319+
mvn -B -ntp \
320+
-Dfmt.skip=true \
321+
-Denforcer.skip=true \
322+
-Dcheckstyle.skip=true \
323+
clirr:check
324+
RETURN_CODE=$?
325+
if [[ -n "${BUILD_SUBDIR}" ]]
326+
then
327+
echo "restoring directory"
328+
popd
329+
fi
330+
;;
308331
*) ;;
309332

310333
esac

0 commit comments

Comments
 (0)