Merge pull request twisted#1648 from twisted/10246-gha-concurency

Merge pull request twisted#123 from twisted/4356-branch-name-with-trac-id Author: adiroiban Reviewer: altendky Fixes: ticket:10246 Enable GitHub Actions concurrency for the test workflow.
MichalKuzma21 · Aug 11, 2021 · fcd8f3c · fcd8f3c
2 parents 1d439dd + 08480d0
commit fcd8f3c
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 3 deletions.
diff --git a/.github/workflows/test.yaml b/.github/workflows/test.yaml
@@ -11,6 +11,15 @@ on:
     branches: [ trunk ]
 
 
+# Only have a run a single parallel for each branch.
+# Runs for trunk are queues.
+# Older runs for non-trunk branches are cancelled and the jobs are executed
+# only for the latest push to the branch.
+concurrency:
+  group: ${{ github.ref }}
+  cancel-in-progress: ${{ github.ref != 'refs/heads/trunk' }}
+
+
 defaults:
   run:
     shell: bash
@@ -110,7 +119,7 @@ jobs:
         tox ${{ matrix.trial-target }}
 
     - name: Prepare coverage
-      if: always() && contains(matrix['tox-env'], 'withcov')
+      if: ${{ !cancelled() && contains(matrix['tox-env'], 'withcov') }}
       run: |
         # sub-process coverage are generated in separate files so we combine them
         # to get an unified coverage for the local run.
@@ -120,15 +129,15 @@ jobs:
         python -m coverage report --skip-covered
 
     - uses: codecov/codecov-action@v1
-      if: always() && contains(matrix['tox-env'], 'withcov')
+      if: ${{ !cancelled() && contains(matrix['tox-env'], 'withcov') }}
       with:
         files: coverage.xml
         name: lnx-${{ matrix.python-version }}-${{ matrix.tox-env }}${{ matrix.noipv6 }}
         fail_ci_if_error: true
         functionalities: gcov,search
 
     - name: Publish to Coveralls
-      if: always() && contains(matrix['tox-env'], 'withcov')
+      if: ${{ !cancelled() && contains(matrix['tox-env'], 'withcov') }}
       continue-on-error: true
       run: |
         python -m coveralls -v

diff --git a/src/twisted/newsfragments/10246.misc b/src/twisted/newsfragments/10246.misc