fix detekt merged report name

uzzu · Dec 17, 2023 · c41e249 · c41e249
1 parent 575e24f
commit c41e249
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/.github/workflows/pull-request.yml b/.github/workflows/pull-request.yml
@@ -33,7 +33,7 @@ jobs:
       - name: Run reviewdog
         if: always()
         run: |
-          find . -type f -name "ktlint*Check.xml" -exec sh -c "cat {} | reviewdog -f=checkstyle -name='ktlint' -reporter=github-pr-review" \;
+          cat build/reports/detekt/detekt-merged-report.xml | reviewdog -f=checkstyle -name="detekt" -reporter="github-check" -level="warning" -filter-mode="nofilter" -fail-on-error
       - name: Gradle publish to mavenLocal
         run: |
           ./gradlew publishToMavenLocal

diff --git a/build.gradle.kts b/build.gradle.kts
@@ -46,7 +46,7 @@ fun Project.detektConfigurationShared(
     }
 
     val reportMerge by tasks.registering(ReportMergeTask::class) {
-        output.set(rootProject.layout.buildDirectory.file("reports/detekt/merge.xml"))
+        output.set(rootProject.layout.buildDirectory.file("reports/detekt/detekt-merged-report.xml"))
     }
 
     tasks.withType<Detekt>().configureEach {