refactor CI configuration for consistency and clarity

Author: VREMSoftwareDevelopment

.github/workflows/android_ci.yml

@@ -1,48 +1,40 @@
-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:
-      - name: checkout repository
-        uses: actions/checkout@v4
-      - name: set up JDK 21
-        uses: actions/setup-java@v4
+      - uses: actions/checkout@v4
+      - uses: actions/setup-java@v4
         with:
-          distribution: 'zulu'
-          java-version: '21'
-      - name: lint
-        run: bash ./gradlew lintDebug --stacktrace
-      - name: unit tests
-        run: bash ./gradlew testDebugUnitTest --stacktrace
-      - name: artifact test reports
-        if: failure()
+          distribution: zulu
+          java-version: 21
+      - run: ./gradlew lintDebug --stacktrace
+        name: Lint
+      - run: ./gradlew testDebugUnitTest --stacktrace
+        name: Unit Tests
+      - if: failure()
         uses: actions/upload-artifact@v4
         with:
           name: test-report
           path: app/build/reports/tests/testDebugUnitTest/
-      - name: publish unit test results
-        if: always()
-        uses: EnricoMi/publish-unit-test-result-action@v2
+      - uses: EnricoMi/publish-unit-test-result-action@v2
         with:
           files: app/build/test-results/testDebugUnitTest/*.xml
-      - name: coverage
-        run: bash ./gradlew jacocoTestCoverageVerification --stacktrace
-      - name: artifact reports
-        uses: actions/upload-artifact@v4
+      - run: ./gradlew jacocoTestCoverageVerification --stacktrace
+        name: Coverage
+      - uses: actions/upload-artifact@v4
         with:
           name: artifact-reports
           path: app/build/reports
-      - name: upload coverage to Codecov
-        uses: codecov/codecov-action@v4
+      - uses: codecov/codecov-action@v4
         with:
           token: ${{ secrets.CODECOV_TOKEN }}
           file: ./app/build/reports/jacoco/jacocoTestReport/jacocoTestReport.xml
-      - name: build apk
-        run: bash ./gradlew assembleDebug --stacktrace
-      - name: artifact apk
-        uses: actions/upload-artifact@v4
+      - run: ./gradlew assembleDebug --stacktrace
+        name: Build 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,31 +1,25 @@
-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:
-      - name: checkout repository
-        uses: actions/checkout@v4
-      - name: set up JDK 21
-        uses: actions/setup-java@v4
+      - uses: actions/checkout@v4
+      - uses: actions/setup-java@v4
         with:
-          distribution: 'zulu'
-          java-version: '21'
-      - name: build apks
-        run: bash ./gradlew assembleDebug assembleAndroidTest
-      - name: run tests
-        uses: emulator-wtf/run-tests@v0
+          distribution: zulu
+          java-version: 21
+      - run: ./gradlew assembleDebug assembleAndroidTest
+      - 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
-      - name: publish test results
-        uses: mikepenz/action-junit-report@v5
+      - 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,24 +10,18 @@ jobs:
       contents: read
       security-events: write
     strategy:
-      fail-fast: false
       matrix:
-        language: [ 'java-kotlin' ]
+        language: [java-kotlin]
     steps:
-      - name: Checkout repository
-        uses: actions/checkout@v4
-      - name: set up JDK 21
-        uses: actions/setup-java@v4
+      - uses: actions/checkout@v4
+      - uses: actions/setup-java@v4
         with:
-          distribution: 'zulu'
-          java-version: '21'
-      - name: Initialize CodeQL
-        uses: github/codeql-action/init@v3
+          distribution: zulu
+          java-version: 21
+      - uses: github/codeql-action/init@v3
         with:
           languages: ${{ matrix.language }}
-      - name: Auto build
-        uses: github/codeql-action/autobuild@v3
-      - name: Perform CodeQL Analysis
-        uses: github/codeql-action/analyze@v3
+      - uses: github/codeql-action/autobuild@v3
+      - 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