Separate out test workflows for changed files check vs. full test sui…

…te (#216)
mesh-adaptation · Sep 15, 2024 · 6827403 · 6827403
1 parent b6ec485
commit 6827403
Show file tree

Hide file tree

Showing 3 changed files with 12 additions and 10 deletions.
diff --git a/.github/workflows/citation_check.yml b/.github/workflows/citation_check.yml
@@ -1,10 +1,14 @@
 name: 'Citation Check'
 
 on:
-  # Run only when commits are pushed to a branch or a PR that modifies a .cff file
+  # Run when a branch that modifies a .cff file is merged into main
   push:
+    branches:
+      - main
     paths:
       - '**/*.cff'
+
+  # Run when commits are pushed to a branch or a PR that modifies a .cff file
   pull_request:
     paths:
       - '**/*.cff'
@@ -16,9 +20,6 @@ jobs:
   citation_check:
     name: 'Check Citation'
     runs-on: ubuntu-latest
-    container:
-      image: ghcr.io/mesh-adaptation/firedrake-parmmg:latest
-      options: --user root
 
     steps:
       - name: 'Check out the repo'
@@ -31,5 +32,5 @@ jobs:
 
       - name: 'Check citation'
         run: |
-          . /home/firedrake/firedrake/bin/activate
+          pip install cffconvert
           make check_citation
diff --git a/.github/workflows/test_suite.yml b/.github/workflows/test_suite.yml
@@ -34,6 +34,7 @@ jobs:
 
       - name: 'Determine files differing from target branch'
         id: changed-files
+        if: ${{ github.event_name != 'push' && github.ref != 'refs/heads/main' }}
         uses: tj-actions/changed-files@v44
         with:
           files: |
@@ -49,26 +50,26 @@ jobs:
           rm -rf build
 
       - name: 'Setup Python'
-        if: ${{ (steps.changed-files.outputs.any_changed == 'true') || ( github.event_name == 'schedule') }}
+        if: ${{ (github.event_name == 'push' && github.ref == 'refs/heads/main') || (steps.changed-files.outcome == 'success' && steps.changed-files.outputs.any_changed == 'true') || github.event_name == 'schedule' }}
         uses: actions/setup-python@v2
         with:
           python-version: 3.8
 
       - name: 'Install Goalie'
-        if: ${{ (steps.changed-files.outputs.any_changed == 'true') || ( github.event_name == 'schedule') }}
+        if: ${{ (github.event_name == 'push' && github.ref == 'refs/heads/main') || (steps.changed-files.outcome == 'success' && steps.changed-files.outputs.any_changed == 'true') || github.event_name == 'schedule' }}
         run: |
           . /home/firedrake/firedrake/bin/activate
           python -m pip uninstall -y goalie
           python -m pip install -e .
 
       - name: 'Lint'
-        if: ${{ (steps.changed-files.outputs.any_changed == 'true') || ( github.event_name == 'schedule') }}
+        if: ${{ (github.event_name == 'push' && github.ref == 'refs/heads/main') || (steps.changed-files.outcome == 'success' && steps.changed-files.outputs.any_changed == 'true') || github.event_name == 'schedule' }}
         run: |
           . /home/firedrake/firedrake/bin/activate
           make lint
 
       - name: 'Test Goalie'
-        if: ${{ (steps.changed-files.outputs.any_changed == 'true') || ( github.event_name == 'schedule') }}
+        if: ${{ (github.event_name == 'push' && github.ref == 'refs/heads/main') || (steps.changed-files.outcome == 'success' && steps.changed-files.outputs.any_changed == 'true') || github.event_name == 'schedule' }}
         run: |
           . /home/firedrake/firedrake/bin/activate
           python $(which firedrake-clean)

diff --git a/CITATION.cff b/CITATION.cff
@@ -1,5 +1,5 @@
 cff-version: 1.2.0
-message: If you use this software in a publication, please cite it using these metadata.
+message: If you use this software in a publication then please cite it using these metadata.
 title: Goalie
 abstract: Goal-oriented error estimation and mesh adaptation for finite element problems solved using Firedrake.
 authors: