Merge pull request #167 from carstenhag/task/add-android-lint-pr-check

Adapt CI pipeline to add android lint comments/checks
EmmaTellblom · Nov 3, 2024 · 7cb78ad · 7cb78ad
2 parents 185b3f4 + 0711e13
commit 7cb78ad
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 2 deletions.
diff --git a/.github/workflows/push.yml b/.github/workflows/push.yml
@@ -11,12 +11,19 @@ jobs:
     name: Build
     runs-on: ubuntu-latest
     steps:
-      - uses: actions/checkout@v3
+      - uses: actions/checkout@v4
       - name: set up JDK 17
-        uses: actions/setup-java@v3
+        uses: actions/setup-java@v4
         with:
           java-version: '17'
           distribution: 'temurin'
           cache: gradle
       - name: Build with Gradle
         run: chmod +x gradlew && ./gradlew build
+
+      # Explained at https://medium.com/bumble-tech/android-lint-and-detekt-warnings-in-github-pull-requests-2880df5d32af
+      - uses: github/codeql-action/upload-sarif@v2
+        if: success() || failure()
+        with:
+          sarif_file: app/build/reports/lint-results-debug.sarif
+          category: lint
diff --git a/app/build.gradle.kts b/app/build.gradle.kts
@@ -54,6 +54,9 @@ android {
         // Disables dependency metadata when building Android App Bundles.
         includeInBundle = false
     }
+    lint {
+        sarifReport = true
+    }
 }
 
 dependencies {