Skip to content

Commit

Permalink
Move static analysis to reusable workflow
Browse files Browse the repository at this point in the history
  • Loading branch information
cmelchior committed Jun 16, 2022
1 parent 9a35a8d commit be45ad5
Show file tree
Hide file tree
Showing 2 changed files with 76 additions and 53 deletions.
72 changes: 72 additions & 0 deletions .github/workflows/include_static_analysis.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,72 @@
name: Static Analysis

on:
workflow_call:

jobs:
static-analysis:
runs-on: ubuntu-latest
steps:

- name: Checkout code
uses: actions/checkout@v3
with:
submodules: "recursive"

- name: Setup Gradle and task/dependency caching
uses: gradle/gradle-build-action@v2
with:
cache-read-only: false # TODO How to configure caching here?

- name: Run Ktlint
run: ./gradlew ktlintCheck

- name: Stash Ktlint results
if: always()
run: |
rm -rf /tmp/ktlint
rm -rf /tmp/detekt
mkdir /tmp/ktlint
mkdir /tmp/detekt
rsync -a --delete --ignore-errors examples/kmm-sample/androidApp/build/reports/ktlint/ /tmp/ktlint/example/ || true
rsync -a --delete --ignore-errors test/build/reports/ktlint/ /tmp/ktlint/test/ || true
rsync -a --delete --ignore-errors packages/cinterop/build/reports/ktlint/ /tmp/ktlint/cinterop/ || true
rsync -a --delete --ignore-errors packages/library-base/build/reports/ktlint/ /tmp/ktlint/library-base/ || true
rsync -a --delete --ignore-errors packages/library-sync/build/reports/ktlint/ /tmp/ktlint/library-sync/ || true
rsync -a --delete --ignore-errors packages/plugin-compiler/build/reports/ktlint/ /tmp/ktlint/plugin-compiler/ || true
rsync -a --delete --ignore-errors packages/gradle-plugin/build/reports/ktlint/ /tmp/ktlint/plugin-gradle/ || true
rsync -a --delete --ignore-errors benchmarks/build/reports/ktlint/ /tmp/ktlint/benchmarks/ || true
- name: Publish Ktlint results
uses: jwgmeligmeyling/checkstyle-github-action@master
if: always()
with:
name: Ktlint
title: Ktlint Analyzer report
path: '/tmp/ktlint/**/*.xml'

- name: Run Detekt
run: ./gradlew detekt


- name: Stash Ktlint results
if: always()
run: |
rm -rf /tmp/detekt
mkdir /tmp/detekt
rsync -a --delete --ignore-errors examples/kmm-sample/androidApp/build/reports/detekt/ /tmp/detekt/example/ || true
rsync -a --delete --ignore-errors test/build/reports/detekt/ /tmp/detekt/test/ || true
rsync -a --delete --ignore-errors packages/cinterop/build/reports/detekt/ /tmp/detekt/cinterop/ || true
rsync -a --delete --ignore-errors packages/library-base/build/reports/detekt/ /tmp/detekt/library-base/ || true
rsync -a --delete --ignore-errors packages/library-sync/build/reports/detekt/ /tmp/detekt/library-sync/ || true
rsync -a --delete --ignore-errors packages/plugin-compiler/build/reports/detekt/ /tmp/detekt/plugin-compiler/ || true
rsync -a --delete --ignore-errors packages/gradle-plugin/build/reports/detekt/ /tmp/detekt/plugin-gradle/ || true
rsync -a --delete --ignore-errors benchmarks/build/reports/detekt/ /tmp/detekt/benchmarks/ || true
- name: Publish Detekt results
uses: jwgmeligmeyling/checkstyle-github-action@master
if: always()
with:
name: Detekt
title: Detekt Analyzer report
path: '/tmp/detekt/**/*.xml'
57 changes: 4 additions & 53 deletions .github/workflows/pr.yml
Original file line number Diff line number Diff line change
Expand Up @@ -6,8 +6,12 @@ on:
env:
REALM_DISABLE_ANALYTICS: true
jobs:
static-analysis:
uses: ./.github/workflows/include_static_analysis.yml

build-packages:
runs-on: ubuntu-latest
needs: static-analysis
steps:
- name: Checkout code
uses: actions/checkout@v3
Expand Down Expand Up @@ -55,59 +59,6 @@ jobs:
with:
ndk-version: r23c

- name: Run Ktlint
run: ./gradlew ktlintCheck

- name: Stash Ktlint results
if: always()
run: |
rm -rf /tmp/ktlint
rm -rf /tmp/detekt
mkdir /tmp/ktlint
mkdir /tmp/detekt
rsync -a --delete --ignore-errors examples/kmm-sample/androidApp/build/reports/ktlint/ /tmp/ktlint/example/ || true
rsync -a --delete --ignore-errors test/build/reports/ktlint/ /tmp/ktlint/test/ || true
rsync -a --delete --ignore-errors packages/cinterop/build/reports/ktlint/ /tmp/ktlint/cinterop/ || true
rsync -a --delete --ignore-errors packages/library-base/build/reports/ktlint/ /tmp/ktlint/library-base/ || true
rsync -a --delete --ignore-errors packages/library-sync/build/reports/ktlint/ /tmp/ktlint/library-sync/ || true
rsync -a --delete --ignore-errors packages/plugin-compiler/build/reports/ktlint/ /tmp/ktlint/plugin-compiler/ || true
rsync -a --delete --ignore-errors packages/gradle-plugin/build/reports/ktlint/ /tmp/ktlint/plugin-gradle/ || true
rsync -a --delete --ignore-errors benchmarks/build/reports/ktlint/ /tmp/ktlint/benchmarks/ || true
- name: Publish Ktlint results
uses: jwgmeligmeyling/checkstyle-github-action@master
if: always()
with:
name: Ktlint
title: Ktlint Analyzer report
path: '/tmp/ktlint/**/*.xml'

- name: Run Detekt
run: ./gradlew detekt


- name: Stash Ktlint results
if: always()
run: |
rm -rf /tmp/detekt
mkdir /tmp/detekt
rsync -a --delete --ignore-errors examples/kmm-sample/androidApp/build/reports/detekt/ /tmp/detekt/example/ || true
rsync -a --delete --ignore-errors test/build/reports/detekt/ /tmp/detekt/test/ || true
rsync -a --delete --ignore-errors packages/cinterop/build/reports/detekt/ /tmp/detekt/cinterop/ || true
rsync -a --delete --ignore-errors packages/library-base/build/reports/detekt/ /tmp/detekt/library-base/ || true
rsync -a --delete --ignore-errors packages/library-sync/build/reports/detekt/ /tmp/detekt/library-sync/ || true
rsync -a --delete --ignore-errors packages/plugin-compiler/build/reports/detekt/ /tmp/detekt/plugin-compiler/ || true
rsync -a --delete --ignore-errors packages/gradle-plugin/build/reports/detekt/ /tmp/detekt/plugin-gradle/ || true
rsync -a --delete --ignore-errors benchmarks/build/reports/detekt/ /tmp/detekt/benchmarks/ || true
- name: Publish Detekt results
uses: jwgmeligmeyling/checkstyle-github-action@master
if: always()
with:
name: Detekt
title: Detekt Analyzer report
path: '/tmp/detekt/**/*.xml'


# - name: Build packages
# working-directory: packages
Expand Down

0 comments on commit be45ad5

Please sign in to comment.