From 086494837b7e83c1ee52d3770a97f0f3d5c65a90 Mon Sep 17 00:00:00 2001 From: Robert Stupp Date: Mon, 27 Jul 2026 09:18:52 +0200 Subject: [PATCH] Remove `jmh` from CI and `check` Do not run JMH in CI, it's results are not monitored and JMH results from GH hosted runners aren't reliable either. Also remove JMH invocation from Gradle's `check`. --- .github/workflows/main.yml | 10 ---------- core/build.gradle.kts | 2 -- jackson/build.gradle.kts | 2 -- jackson3/build.gradle.kts | 2 -- 4 files changed, 16 deletions(-) diff --git a/.github/workflows/main.yml b/.github/workflows/main.yml index 521b2397..0b409956 100644 --- a/.github/workflows/main.yml +++ b/.github/workflows/main.yml @@ -43,17 +43,10 @@ jobs: with: cache-read-only: ${{ github.event_name != 'push' || github.ref != 'refs/heads/main' }} - - name: Spotless Check - # Spotless must run in a different invocation, because - # it has some weird Gradle configuration/variant issue - if: ${{ matrix.java-version == '21' }} - run: ./gradlew spotlessCheck --scan - - name: Build run: > ./gradlew --rerun-tasks ${{ env.ADDITIONAL_GRADLE_OPTS }} assemble check publishToMavenLocal - -x jmh -x spotlessCheck -x testNative --scan - name: Quarkus native smoke @@ -65,9 +58,6 @@ jobs: # because these tasks intentionally do not depend on the publishToMavenLocal tasks. run: ./gradlew buildToolIntegrations - - name: Microbenchmarks - run: ./gradlew jmh - - name: Capture test results uses: actions/upload-artifact@043fb46d1a93c77aae656e7c1c64a875d1fc6a0a # v7 if: failure() diff --git a/core/build.gradle.kts b/core/build.gradle.kts index 52175403..9f417444 100644 --- a/core/build.gradle.kts +++ b/core/build.gradle.kts @@ -102,6 +102,4 @@ sourceSets.test { tasks.named("compileJava") { dependsOn(generateCelGrammar) } -tasks.named("check") { dependsOn(tasks.named("jmh")) } - tasks.named("assemble") { dependsOn(tasks.named("jmhJar")) } diff --git a/jackson/build.gradle.kts b/jackson/build.gradle.kts index 20112b20..bfc132a8 100644 --- a/jackson/build.gradle.kts +++ b/jackson/build.gradle.kts @@ -51,6 +51,4 @@ dependencies { jmh { jmhVersion.set(libs.versions.jmh.get()) } -tasks.named("check") { dependsOn(tasks.named("jmh")) } - tasks.named("assemble") { dependsOn(tasks.named("jmhJar")) } diff --git a/jackson3/build.gradle.kts b/jackson3/build.gradle.kts index 3dc4bb58..e99ca0f1 100644 --- a/jackson3/build.gradle.kts +++ b/jackson3/build.gradle.kts @@ -51,6 +51,4 @@ dependencies { jmh { jmhVersion.set(libs.versions.jmh.get()) } -tasks.named("check") { dependsOn(tasks.named("jmh")) } - tasks.named("assemble") { dependsOn(tasks.named("jmhJar")) }