Skip to content

Commit 48857ea

Browse files
committed
Add Detekt to Github Action
1 parent d894456 commit 48857ea

File tree

1 file changed

+25
-3
lines changed

1 file changed

+25
-3
lines changed

.github/workflows/pr.yml

Lines changed: 25 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -72,6 +72,28 @@ jobs:
7272
with:
7373
category: "KTlint"
7474

75+
detekt:
76+
needs: [yamllint, lockfiles]
77+
runs-on: ubuntu-latest
78+
permissions:
79+
security-events: write
80+
steps:
81+
- uses: actions/checkout@v4
82+
83+
- uses: ./.github/actions/setup-build-env
84+
with:
85+
cache-encryption-key: ${{ secrets.GRADLE_ENCRYPTION_KEY }}
86+
mock-google-services: "true"
87+
88+
# Since `build-logic` is added with `includeBuild` we have to call detekt explicitly on `:convention`.
89+
- name: Run detekt
90+
run: ./gradlew detekt :build-logic:convention:detekt --continue
91+
92+
- uses: ./.github/actions/upload-sarif-results
93+
if: always()
94+
with:
95+
category: "Detekt"
96+
7597
lint:
7698
needs: [yamllint, lockfiles]
7799
runs-on: ubuntu-latest
@@ -102,7 +124,7 @@ jobs:
102124

103125
screenshot_test:
104126
name: "Screenshot Tests"
105-
needs: [lint, lockfiles, ktlint]
127+
needs: [lint, lockfiles, ktlint, detekt]
106128
runs-on: ubuntu-latest
107129
steps:
108130
- uses: actions/checkout@v4
@@ -171,7 +193,7 @@ jobs:
171193

172194
unit_tests:
173195
name: "Unit Tests"
174-
needs: [lint, lockfiles, ktlint]
196+
needs: [lint, lockfiles, ktlint, detekt]
175197
runs-on: ubuntu-latest
176198
steps:
177199
- uses: actions/checkout@v4
@@ -195,7 +217,7 @@ jobs:
195217
196218
instrumentation_test:
197219
name: "Instrumentation Tests"
198-
needs: [lint, lockfiles, ktlint]
220+
needs: [lint, lockfiles, ktlint, detekt]
199221
runs-on: ubuntu-latest
200222
strategy:
201223
# We want the result of each device to be reported, so we can't fail-fast

0 commit comments

Comments
 (0)