Refactor CI configuration for improved clarity and consistency

Author: VREMSoftwareDevelopment

.github/workflows/android_ci.yml

@@ -1,40 +1,37 @@
-name: Android CI
-on: [push, pull_request]
+name: "Android CI"
+on: [ push, pull_request ]
 jobs:
   test:
     if: "!contains(github.event.head_commit.message, 'skip ci')"
     runs-on: ubuntu-latest
     steps:
-      - uses: actions/checkout@v4
-      - uses: actions/setup-java@v4
+      - name: checkout repository
+        uses: actions/checkout@v4
+      - name: set up JDK 21
+        uses: actions/setup-java@v4
         with:
-          distribution: zulu
-          java-version: 21
-      - run: ./gradlew lintDebug --stacktrace
-        name: Lint
-      - run: ./gradlew testDebugUnitTest --stacktrace
-        name: Unit Tests
-      - if: failure()
+          distribution: 'zulu'
+          java-version: '21'
+      - name: lint
+        run: bash ./gradlew lintDebug --stacktrace
+      - name: unit tests
+        run: bash ./gradlew testDebugUnitTest --stacktrace
+      - name: coverage
+        run: bash ./gradlew jacocoTestCoverageVerification --stacktrace
+      - name: artifact reports
         uses: actions/upload-artifact@v4
-        with:
-          name: test-report
-          path: app/build/reports/tests/testDebugUnitTest/
-      - uses: EnricoMi/publish-unit-test-result-action@v2
-        with:
-          files: app/build/test-results/testDebugUnitTest/*.xml
-      - run: ./gradlew jacocoTestCoverageVerification --stacktrace
-        name: Coverage
-      - uses: actions/upload-artifact@v4
         with:
           name: artifact-reports
           path: app/build/reports
-      - uses: codecov/codecov-action@v4
+      - name: upload coverage to Codecov
+        uses: codecov/codecov-action@v4
         with:
           token: ${{ secrets.CODECOV_TOKEN }}
           file: ./app/build/reports/jacoco/jacocoTestReport/jacocoTestReport.xml
-      - run: ./gradlew assembleDebug --stacktrace
-        name: Build APK
-      - uses: actions/upload-artifact@v4
+      - name: build apk
+        run: bash ./gradlew assembleDebug --stacktrace
+      - name: artifact apk
+        uses: actions/upload-artifact@v4
         with:
           name: artifact-apk
-          path: app/build/outputs/apk/debug
+          path: app/build/outputs/apk/debug

.github/workflows/android_test_ci.yml

@@ -1,25 +1,31 @@
-name: Android Instrumentation Tests
+name: "Android Instrumentation Tests"
 on:
   push:
-    branches: [release]
+    branches:
+      - release
 jobs:
   instrumentation_tests:
     if: "!contains(github.event.head_commit.message, 'skip ci')"
     runs-on: ubuntu-latest
     steps:
-      - uses: actions/checkout@v4
-      - uses: actions/setup-java@v4
+      - name: checkout repository
+        uses: actions/checkout@v4
+      - name: set up JDK 21
+        uses: actions/setup-java@v4
         with:
-          distribution: zulu
-          java-version: 21
-      - run: ./gradlew assembleDebug assembleAndroidTest
-      - uses: emulator-wtf/run-tests@v0
+          distribution: 'zulu'
+          java-version: '21'
+      - name: build apks
+        run: bash ./gradlew assembleDebug assembleAndroidTest
+      - name: run tests
+        uses: emulator-wtf/run-tests@v0
         with:
           api-token: ${{ secrets.EW_API_TOKEN }}
           app: app/build/outputs/apk/debug/app-debug.apk
           test: app/build/outputs/apk/androidTest/debug/app-debug-androidTest.apk
           outputs-dir: build/test-results
-      - uses: mikepenz/action-junit-report@v5
+      - name: publish test results
+        uses: mikepenz/action-junit-report@v5
         if: always()
         with:
-          report_paths: build/test-results/**/*.xml
+          report_paths: 'build/test-results/**/*.xml'

.github/workflows/codeql-analysis.yml

@@ -1,5 +1,5 @@
 name: "CodeQL Analyze"
-on: [push, pull_request]
+on: [ push, pull_request ]
 jobs:
   analyze:
     if: "!contains(github.event.head_commit.message, 'skip ci')"
@@ -10,18 +10,24 @@ jobs:
       contents: read
       security-events: write
     strategy:
+      fail-fast: false
       matrix:
-        language: [java-kotlin]
+        language: [ 'java-kotlin' ]
     steps:
-      - uses: actions/checkout@v4
-      - uses: actions/setup-java@v4
+      - name: Checkout repository
+        uses: actions/checkout@v4
+      - name: set up JDK 21
+        uses: actions/setup-java@v4
         with:
-          distribution: zulu
-          java-version: 21
-      - uses: github/codeql-action/init@v3
+          distribution: 'zulu'
+          java-version: '21'
+      - name: Initialize CodeQL
+        uses: github/codeql-action/init@v3
         with:
           languages: ${{ matrix.language }}
-      - uses: github/codeql-action/autobuild@v3
-      - uses: github/codeql-action/analyze@v3
+      - name: Auto build
+        uses: github/codeql-action/autobuild@v3
+      - name: Perform CodeQL Analysis
+        uses: github/codeql-action/analyze@v3
         with:
           category: "/language:${{matrix.language}}"

.github/workflows/stale.yml

@@ -1,7 +1,7 @@
 name: Mark stale issues and pull requests
 on:
   schedule:
-    - cron: 30 6 1 * *
+    - cron: '30 6 1 * *'
 jobs:
   stale:
     runs-on: ubuntu-latest
@@ -12,9 +12,9 @@ jobs:
       - uses: actions/stale@v9
         with:
           repo-token: ${{ secrets.GITHUB_TOKEN }}
-          stale-issue-message: This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.
-          stale-pr-message: This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.
-          stale-issue-label: stale
-          stale-pr-label: stale
-          exempt-issue-labels: bug,high-priority
-          days-before-stale: 365
+          stale-issue-message: 'This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.'
+          stale-pr-message: 'This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.'
+          stale-issue-label: 'stale'
+          stale-pr-label: 'stale'
+          exempt-issue-labels: 'bug,high-priority'
+          days-before-stale: 365