Slim down CI config (#152)

Author: alexstyl

.github/workflows/ci.yml

@@ -1,30 +1,25 @@
-name: PR Unit Tests
+name: CI
 
 on:
+  push:
+    branches: [main]
   pull_request:
-    types: [ opened, reopened, synchronize ]
 
 permissions:
   contents: read
 
 jobs:
-  pr_checks:
+  tests:
     runs-on: ubuntu-latest
     steps:
-      - name: Checkout
-        uses: actions/checkout@v4
-
-      - name: Validate Gradle Wrapper
-        uses: gradle/actions/wrapper-validation@v3
-
-      - name: Set up JDK 17
-        uses: actions/setup-java@v4
+      - uses: actions/checkout@v4
+      - uses: gradle/actions/wrapper-validation@v3
+      - uses: actions/setup-java@v4
         with:
           distribution: temurin
           java-version: 17
 
-      - name: Cache Gradle
-        uses: actions/cache@v4
+      - uses: actions/cache@v4
         with:
           path: |
             ~/.gradle/caches
@@ -33,39 +28,11 @@ jobs:
           restore-keys: |
             gradle-${{ runner.os }}-
 
-      - name: Run Detekt
-        run: ./gradlew --no-daemon --console=plain :core:detekt
-      - name: Show Detekt summary
-        if: always()
-        run: |
-          SUMMARY_MD=core/build/reports/detekt/detekt.md
-          if [ -f "$SUMMARY_MD" ]; then
-            echo -e "## Detekt Report\n" >> "$GITHUB_STEP_SUMMARY"
-            cat "$SUMMARY_MD" >> "$GITHUB_STEP_SUMMARY"
-          else
-            echo "Detekt report not found."
-          fi
-
-      - name: Run unit tests
-        run: ./gradlew --no-daemon jvmTest
+      - name: Compile
+        run: ./gradlew --no-daemon --console=plain :core:assemble
 
-      - name: Verify test reports exist
-        run: |
-          set -e
-          count=$(find . -type f -path '*build/test-results/*/TEST-*.xml' 2>/dev/null | wc -l || true)
-          echo "Discovered $count test report files."
-          if [ "$count" -eq 0 ]; then
-            echo "Check locally that all works okay, you can use this command to check ./gradlew jvmTest"
-            exit 1
-          fi
+      - name: Detekt
+        run: ./gradlew --no-daemon --console=plain :core:detekt
 
-      - name: Upload test reports
-        if: always()
-        uses: actions/upload-artifact@v4
-        with:
-          name: test-reports
-          path: |
-            **/build/test-results
-            **/build/reports/tests
-          if-no-files-found: ignore
-          retention-days: 7
+      - name: Test
+        run: ./gradlew --no-daemon --console=plain :core:jvmTest

core/build.gradle.kts

@@ -139,10 +139,8 @@ val detektSourceDirs = listOf(
 detekt {
     buildUponDefaultConfig = true
     config.setFrom(files(rootProject.file("detekt.yml")))
-    ignoreFailures = true
     parallel = true
     source.setFrom(detektSourceDirs)
-    baseline = file("$projectDir/detekt-baseline.xml")
 }
 
 tasks.withType<Detekt>().configureEach {

core/detekt-baseline.xml

@@ -18,6 +18,9 @@ else
     git rebase "$latest_tag"
 fi
 
+echo "Running detekt..."
+./gradlew core:detekt
+
 echo "Compiling Kotlin..."
 ./gradlew core:assemble