Improve CI scripts based on reviewer feedback

Move testing to own step in Flang CI Script Rename CI script where only gfortran is used to be consistent with name of CI script where only Flang is used
BerkeleyLab · Jan 17, 2025 · d6c88b5 · d6c88b5
1 parent b68ef9d
commit d6c88b5
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 3 deletions.
diff --git a/.github/workflows/CI.yml b/.github/workflows/CI.yml
@@ -1,4 +1,4 @@
-name: CI
+name: Build with gfortran
 
 on: [push, pull_request]
 

diff --git a/.github/workflows/build-with-flang.yml b/.github/workflows/build-with-flang.yml
@@ -23,16 +23,19 @@ jobs:
       with:
         github-token: ${{ secrets.GITHUB_TOKEN }}
 
-    - name: Build and Test with LLVM Flang
+    - name: Build with LLVM Flang
       if: contains(matrix.os, 'ubuntu')
       run: |
         fpm --version
         $FC --version
         $CC --version
         export FPM_FC=$FC
         export FPM_CC=$CC
-        export GASNET_PSHM_NODES=8
         apt update
         apt install -y build-essential pkg-config make
         ./install.sh
+
+    - name: Run unit tests
+      run: |
+        export GASNET_PSHM_NODES=8
         ./build/run-fpm.sh test -- -d