Fix #2711 & part of #5343: Workflow cancellation support in CI using concurrency

.github/workflows/build_tests.yml

@@ -10,6 +10,10 @@ on:
       # Push events on develop branch
       - develop
 
+concurrency:
+  group: ${{ github.workflow }}-${{ github.ref }}
+  cancel-in-progress: true
+
 jobs:
   bazel_build_app:
     name: Build Binary with Bazel

.github/workflows/main.yml

@@ -12,6 +12,10 @@ on:
       # Push events on develop branch
       - develop
 
+concurrency:
+  group: ${{ github.workflow }}-${{ github.ref }}
+  cancel-in-progress: true
+
 # This workflow has the following jobs:
 # robolectric_tests: Robolectric tests for all modules except the app module
 # app_tests: Non-flaky Robolectric tests for the app module
@@ -54,45 +58,45 @@ jobs:
         run: sudo ./gradlew --full-stacktrace assembleDebug -Dorg.gradle.java.home=$JAVA_HOME
 
       - name: Utility tests
-        if: always()
+        if: ${{ !cancelled() }}
         # We require 'sudo' to avoid an error of the existing android sdk. See https://github.com/actions/starter-workflows/issues/58
         run:  sudo ./gradlew --full-stacktrace :utility:testDebugUnitTest -Dorg.gradle.java.home=$JAVA_HOME
       - name: Upload Utility Test Reports
         uses: actions/upload-artifact@v2
-        if: ${{ always() }} # IMPORTANT: Upload reports regardless of status
+        if: ${{ !cancelled() }} # IMPORTANT: Upload reports regardless of status
         with:
           name: utility reports
           path: utility/build/reports
 
       - name: Data tests
-        if: always()
+        if: ${{ !cancelled() }}
         # We require 'sudo' to avoid an error of the existing android sdk. See https://github.com/actions/starter-workflows/issues/58
         run: sudo ./gradlew --full-stacktrace :data:testDebugUnitTest -Dorg.gradle.java.home=$JAVA_HOME
       - name: Upload Data Test Reports
         uses: actions/upload-artifact@v2
-        if: ${{ always() }} # IMPORTANT: Upload reports regardless of status
+        if: ${{ !cancelled() }} # IMPORTANT: Upload reports regardless of status
         with:
           name: data reports
           path: data/build/reports
 
       - name: Domain tests
-        if: always()
+        if: ${{ !cancelled() }}
         # We require 'sudo' to avoid an error of the existing android sdk. See https://github.com/actions/starter-workflows/issues/58
         run:  sudo ./gradlew --full-stacktrace :domain:testDebugUnitTest -Dorg.gradle.java.home=$JAVA_HOME
       - name: Upload Domain Test Reports
         uses: actions/upload-artifact@v2
-        if: ${{ always() }} # IMPORTANT: Upload reports regardless of status
+        if: ${{ !cancelled() }} # IMPORTANT: Upload reports regardless of status
         with:
           name: domain reports
           path: domain/build/reports
 
       - name: Testing tests
-        if: always()
+        if: ${{ !cancelled() }}
         # We require 'sudo' to avoid an error of the existing android sdk. See https://github.com/actions/starter-workflows/issues/58
         run:  sudo ./gradlew --full-stacktrace :testing:testDebugUnitTest -Dorg.gradle.java.home=$JAVA_HOME
       - name: Upload Testing Test Reports
         uses: actions/upload-artifact@v2
-        if: ${{ always() }} # IMPORTANT: Upload reports regardless of status
+        if: ${{ !cancelled() }} # IMPORTANT: Upload reports regardless of status
         with:
           name: testing reports
           path: testing/build/reports
@@ -133,15 +137,15 @@ jobs:
           sudo ./gradlew --full-stacktrace :app:testDebugUnitTest --${{ matrix.shard }} -Dorg.gradle.java.home=$JAVA_HOME
       - name: Upload App Test Reports
         uses: actions/upload-artifact@v2
-        if: ${{ always() }} # IMPORTANT: Upload reports regardless of status
+        if: ${{ !cancelled() }} # IMPORTANT: Upload reports regardless of status
         with:
           name: app reports ${{ matrix.shard }}
           path: app/build/reports
 
   app_tests:
     name: App Module Robolectric Tests
     needs: run_app_module_test
-    if: ${{ always() }}
+    if: ${{ !cancelled() }}
     runs-on: ${{ matrix.os }}
     strategy:
       matrix:

.github/workflows/static_checks.yml

@@ -9,6 +9,10 @@ on:
     branches:
       - develop
 
+concurrency:
+  group: ${{ github.workflow }}-${{ github.ref }}
+  cancel-in-progress: true
+
 jobs:
   check_codeowners:
     name: Check CODEOWNERS & Repository files
@@ -151,39 +155,39 @@ jobs:
         shell: bash
 
       - name: Regex Patterns Validation Check
-        if: always()
+        if: ${{ !cancelled() }}
         run: |
           bazel run //scripts:regex_pattern_validation_check -- $(pwd)
 
       - name: XML Syntax Validation Check
-        if: always()
+        if: ${{ !cancelled() }}
         run: |
           bazel run //scripts:xml_syntax_check -- $(pwd)
 
       - name: Testfile Presence Check
-        if: always()
+        if: ${{ !cancelled() }}
         run: |
           bazel run //scripts:test_file_check -- $(pwd)
 
       - name: Accessibility label Check
-        if: always()
+        if: ${{ !cancelled() }}
         run: |
           bazel run //scripts:accessibility_label_check -- $(pwd) scripts/assets/accessibility_label_exemptions.pb app/src/main/AndroidManifest.xml
 
       - name: KDoc Validation Check
-        if: always()
+        if: ${{ !cancelled() }}
         run: |
           bazel run //scripts:kdoc_validity_check -- $(pwd) scripts/assets/kdoc_validity_exemptions.pb
 
       - name: Todo Check
-        if: always()
+        if: ${{ !cancelled() }}
         env:
           GITHUB_TOKEN: ${{ github.token }}
         run: |
           bazel run //scripts:todo_open_check -- $(pwd) scripts/assets/todo_open_exemptions.pb
 
       - name: String Resource Validation Check
-        if: always()
+        if: ${{ !cancelled() }}
         run: |
           bazel run //scripts:string_resource_validation_check -- $(pwd)
 
@@ -202,16 +206,16 @@ jobs:
           version: 6.5.0
 
       - name: Maven Repin Check
-        if: always()
+        if: ${{ !cancelled() }}
         run: |
           REPIN=1 bazel run @unpinned_maven//:pin
 
       - name: Maven Dependencies Update Check
-        if: always()
+        if: ${{ !cancelled() }}
         run: |
           bazel run //scripts:maven_dependencies_list_check -- $(pwd) third_party/maven_install.json scripts/assets/maven_dependencies.pb
 
       - name: License Texts Check
-        if: always()
+        if: ${{ !cancelled() }}
         run: |
           bazel run //scripts:license_texts_check -- $(pwd)/app/src/main/res/values/third_party_dependencies.xml

.github/workflows/unit_tests.yml

@@ -12,6 +12,10 @@ on:
       # Push events on develop branch
       - develop
 
+concurrency:
+  group: ${{ github.workflow }}-${{ github.ref }}
+  cancel-in-progress: true
+
 jobs:
   bazel_compute_affected_targets:
     name: Compute affected tests
@@ -320,7 +324,7 @@ jobs:
   check_test_results:
       name: Check Bazel Test Results
       needs: [bazel_compute_affected_targets, bazel_run_test]
-      if: ${{ always() }}
+      if: ${{ !cancelled() }}
       runs-on: ubuntu-20.04
       steps:
         # This step will be skipped if there are no tests to run, so the overall job should pass.