update

oneapi-src · May 31, 2024 · 9f09bf0 · 9f09bf0
1 parent 83e77b8
commit 9f09bf0
Showing 1 changed file with 13 additions and 12 deletions.
diff --git a/.github/workflows/pr.yml b/.github/workflows/pr.yml
@@ -16,7 +16,6 @@ env:
 jobs:
   unit-tests:
     runs-on: ubuntu-latest
-    if: steps.changes.outputs.src == 'true' 
     # One runner for each domain
     strategy:
       matrix:
@@ -41,16 +40,6 @@ jobs:
     name: unit tests ${{ matrix.config }} CPU
     steps:
     - uses: actions/checkout@44c2b7a8a4ea60a981eaca3cf939b5f4305c123b # v4.1.5
-    - name: Check for changes to this domain
-      id: changes
-      uses: dorny/paths-filter@v2
-      with:
-        filters: |
-          src:
-            - 'src/${{ matrix.domain}}/**'
-            - 'tests/unit_tests/${{ matrix.domain }}/**'
-            - CMakeLists.txt
-            # - '.github/workflows/**'
     - name: Restore netlib from cache
       id: cache-lapack
       uses: actions/cache@0c45773b623bea8c8e75f6c82b208c3cf94ea4f9 # v4.0.2
@@ -68,19 +57,31 @@ jobs:
         # 64 bit int
         cmake ${SHARED_OPT} -DBUILD_INDEX64=on -B lapack/build64
         cmake --build lapack/build64 ${PARALLEL} --target install
+    - name: Check for changes to this domain
+      id: changes
+      uses: dorny/paths-filter@v2
+      with:
+        filters: |
+          src:
+          - 'src/${{ matrix.domain}}/**'
+          - 'tests/unit_tests/${{ matrix.domain }}/**'
+          - CMakeLists.txt
+          # - '.github/workflows/**'
     - name: Install oneapi
+      if: needs.check_changes.outputs.src == 'true'
       uses: rscohn2/setup-oneapi@2ad0cf6b74bc2426bdcee825cf88f9db719dd727 # v0.1.0
       with:
         components: |
           icx@2024.1.0
           mkl@2024.1.0
-
     - name: Configure/Build for a domain
+      if: needs.check_changes.outputs.src == 'true'
       run: |
         source /opt/intel/oneapi/setvars.sh
         cmake -DTARGET_DOMAINS=${{ matrix.domain }} -DENABLE_MKLGPU_BACKEND=off -DCMAKE_VERBOSE_MAKEFILE=on ${{ matrix.build_options }} -B build
         cmake --build build ${PARALLEL}
     - name: Run tests
+      if: needs.check_changes.outputs.src == 'true'
       run: |
         source /opt/intel/oneapi/setvars.sh
         ctest --test-dir build ${{ matrix.test_options }}