Skip to content

Commit

Permalink
Apply suggestions from code review
Browse files Browse the repository at this point in the history
verbose logging and serial test execution for more readable logs. build time dominates so parallel test is less important.

Co-authored-by: Dai-Ni Hsieh <[email protected]>
  • Loading branch information
rscohn2 and dnhsieh-intel authored May 16, 2024
1 parent a32e30e commit 1acabc9
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions .github/workflows/pr.yml
Original file line number Diff line number Diff line change
Expand Up @@ -49,10 +49,10 @@ jobs:
- name: Configure/Build for a domain
run: |
source /opt/intel/oneapi/setvars.sh
cmake -DREF_BLAS_ROOT=${PWD}/lapack/install -DREF_LAPACK_ROOT=${PWD}/lapack/install -DENABLE_MKLGPU_BACKEND=off -DTARGET_DOMAINS=${{ matrix.domain }} -B build
cmake -DREF_BLAS_ROOT=${PWD}/lapack/install -DREF_LAPACK_ROOT=${PWD}/lapack/install -DENABLE_MKLGPU_BACKEND=off -DTARGET_DOMAINS=${{ matrix.domain }} -DCMAKE_VERBOSE_MAKEFILE=on -B build
cmake --build build ${PARALLEL}
- name: Run tests
run: |
source /opt/intel/oneapi/setvars.sh
# Run tests, skip GPU tests
ctest --test-dir build ${PARALLEL} -E 'ct$'
ctest --test-dir build -j 1 -E 'ct$'

0 comments on commit 1acabc9

Please sign in to comment.