Update code quality workflow

SRGSSR · May 15, 2024 · d8defd5 · d8defd5
1 parent fcc7156
commit d8defd5
Show file tree

Hide file tree

Showing 6 changed files with 52 additions and 53 deletions.
diff --git a/.github/workflows/code_quality.yaml b/.github/workflows/code_quality.yaml
@@ -1,75 +1,58 @@
 name: Code quality
+
 on:
   merge_group:
   pull_request:
-    types: [synchronize, opened, reopened, ready_for_review]
 
 jobs:
   android-lint:
-    if: github.event.pull_request.draft == false
+    name: Android Lint
     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'
-      - uses: reviewdog/action-setup@v1
-      - name: Run android linter
-        uses: gradle/gradle-build-action@v2
-        with:
-          arguments: lint
-      - name: review linter
-        uses: dvdandroid/action-android-lint@master
-        with:
-          github_token: ${{ secrets.RTS_DEVOPS_GITHUB_TOKEN }}
-          lint_xml_file: ./build/reports/android-lint.xml
-          reporter: github-pr-review
-      - name: check linter
-        uses: dvdandroid/action-android-lint@master
-        with:
-          github_token: ${{ secrets.RTS_DEVOPS_GITHUB_TOKEN }}
-          lint_xml_file: ./build/reports/android-lint.xml
-          reporter: github-pr-check
+      - uses: gradle/actions/setup-gradle@v3
+      - name: Run Android Lint
+        run: ./gradlew lint
+      - uses: github/codeql-action/upload-sarif@v3
+        if: success() || failure()
+        with:
+          sarif_file: build/reports/android-lint/
+          category: android-lint
 
   detekt:
-    if: github.event.pull_request.draft == false
+    name: Detekt
     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'
-      - uses: reviewdog/action-setup@v1
-      - name: review detekt
-        uses: alaegin/[email protected]
-        with:
-          github_token: ${{ secrets.RTS_DEVOPS_GITHUB_TOKEN }}
-          detekt_config: config/detekt.yml
-          reviewdog_reporter: github-pr-review
-          detekt_excludes: "**/build/**,**/.idea/**,**/buildSrc/**,**/androidTest/**,**/test/**,**/resources/**"
-      - name: check detekt
-        uses: alaegin/[email protected]
-        with:
-          github_token: ${{ secrets.RTS_DEVOPS_GITHUB_TOKEN }}
-          detekt_config: config/detekt.yml
-          reviewdog_reporter: github-pr-check
-          detekt_excludes: "**/build/**,**/.idea/**,**/buildSrc/**,**/androidTest/**,**/test/**,**/resources/**"
+      - uses: gradle/actions/setup-gradle@v3
+      - name: Run Detekt
+        run: ./gradlew detekt
+      - uses: github/codeql-action/upload-sarif@v3
+        if: success() || failure()
+        with:
+          sarif_file: build/reports/detekt.sarif
+          category: detekt
 
-  android-unit-test:
-    if: github.event.pull_request.draft == false
+  unit-test:
+    name: Unit Test
     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'
-      - name: Run android local unit test
-        uses: gradle/gradle-build-action@v2
-        with:
-          arguments: testDebug
+      - uses: gradle/actions/setup-gradle@v3
+      - name: Run Unit Tests
+        run: ./gradlew testDebug
diff --git a/build.gradle.kts b/build.gradle.kts
@@ -1,5 +1,6 @@
 // Top-level build file where you can add configuration options common to all sub-projects/modules.
 import io.gitlab.arturbosch.detekt.Detekt
+import io.gitlab.arturbosch.detekt.report.ReportMergeTask
 
 plugins {
     alias(libs.plugins.android.application) apply false
@@ -9,6 +10,12 @@ plugins {
     alias(libs.plugins.detekt)
 }
 
+apply(plugin = "android-reporting")
+
+val detektReportMerge by tasks.registering(ReportMergeTask::class) {
+    output.set(rootProject.layout.buildDirectory.file("reports/detekt.sarif"))
+}
+
 allprojects {
     apply(plugin = "io.gitlab.arturbosch.detekt")
 
@@ -28,14 +35,18 @@ allprojects {
             xml.required.set(false)
             html.required.set(true)
             txt.required.set(false)
-            sarif.required.set(false)
+            sarif.required.set(true)
             md.required.set(false)
         }
+        finalizedBy(detektReportMerge)
     }
 
     dependencies {
-        //noinspection UseTomlInstead
-        detektPlugins("io.gitlab.arturbosch.detekt:detekt-formatting:1.22.0")
+        detektPlugins(libs.detekt.formatting)
+    }
+
+    detektReportMerge {
+        input.from(tasks.withType<Detekt>().map { it.sarifReportFile })
     }
 }
 

diff --git a/data/build.gradle.kts b/data/build.gradle.kts
@@ -40,6 +40,8 @@ android {
     lint {
         // https://developer.android.com/reference/tools/gradle-api/4.1/com/android/build/api/dsl/LintOptions
         abortOnError = false
+        sarifReport = true
+        sarifOutput = rootProject.layout.buildDirectory.file("reports/android-lint/${project.name}.sarif").get().asFile
     }
     publishing {
         singleVariant("release") {
@@ -51,7 +53,6 @@ android {
 
 dependencies {
     api(libs.kotlinx.serialization.json)
-    detektPlugins(libs.detekt.formatting)
 
     testImplementation(libs.robolectric)
     testImplementation(libs.junit)

diff --git a/dataprovider-paging/build.gradle.kts b/dataprovider-paging/build.gradle.kts
@@ -39,6 +39,8 @@ android {
     lint {
         // https://developer.android.com/reference/tools/gradle-api/4.1/com/android/build/api/dsl/LintOptions
         abortOnError = false
+        sarifReport = true
+        sarifOutput = rootProject.layout.buildDirectory.file("reports/android-lint/${project.name}.sarif").get().asFile
     }
     publishing {
         singleVariant("release") {

diff --git a/dataprovider-retrofit/build.gradle.kts b/dataprovider-retrofit/build.gradle.kts
@@ -39,6 +39,8 @@ android {
     lint {
         // https://developer.android.com/reference/tools/gradle-api/4.1/com/android/build/api/dsl/LintOptions
         abortOnError = false
+        sarifReport = true
+        sarifOutput = rootProject.layout.buildDirectory.file("reports/android-lint/${project.name}.sarif").get().asFile
     }
     publishing {
         singleVariant("release") {

diff --git a/dataproviderdemo/build.gradle.kts b/dataproviderdemo/build.gradle.kts
@@ -49,8 +49,8 @@ android {
         abortOnError = true
         checkAllWarnings = true
         checkDependencies = false
-        xmlReport = true // Enable for Danger Android Lint
-        xmlOutput = file("${project.rootDir}/build/reports/android-lint.xml")
+        sarifReport = true
+        sarifOutput = rootProject.layout.buildDirectory.file("reports/android-lint/${project.name}.sarif").get().asFile
         disable.add("LogConditional")
     }
 }