From 204dea7ec47298c0e4d2b538282a2ee0328b0305 Mon Sep 17 00:00:00 2001 From: Edoardo Zoni Date: Mon, 12 May 2025 18:08:06 -0700 Subject: [PATCH 1/5] Implement AMReX solution for skipped required workflows --- .github/workflows/check_changes.yml | 31 ++++++++++++++++++++++++++ .github/workflows/clang_sanitizers.yml | 12 +++++----- .github/workflows/clang_tidy.yml | 10 +++++---- .github/workflows/cuda.yml | 16 ++++++++----- .github/workflows/hip.yml | 13 ++++++----- .github/workflows/insitu.yml | 16 ++++++++----- .github/workflows/intel.yml | 13 ++++++----- .github/workflows/jupyter.yml | 10 +++++---- .github/workflows/macos.yml | 10 +++++---- .github/workflows/ubuntu.yml | 22 +++++++++++------- .github/workflows/windows.yml | 13 ++++++----- 11 files changed, 114 insertions(+), 52 deletions(-) create mode 100644 .github/workflows/check_changes.yml diff --git a/.github/workflows/check_changes.yml b/.github/workflows/check_changes.yml new file mode 100644 index 00000000000..aebf704f1f4 --- /dev/null +++ b/.github/workflows/check_changes.yml @@ -0,0 +1,31 @@ +name: Check Changes + +on: + workflow_call: + outputs: + has_docs_changes: + value: ${{ jobs.check.outputs.has_docs_changes }} + has_non_docs_changes: + value: ${{ jobs.check.outputs.has_non_docs_changes }} + +jobs: + check: + runs-on: ubuntu-latest + outputs: + has_non_docs_changes: ${{ steps.set-output.outputs.has_non_docs_changes }} + steps: + - uses: actions/checkout@v4 + - uses: dorny/paths-filter@v3 + id: changes + with: + filters: | + docs: + - 'Docs/**' + - '**.rst' + others: + - '!Docs/**' + - '!**.rst' + - id: set-output + run: | + echo "has_docs_changes=${{ steps.changes.outputs.docs }}" >> $GITHUB_OUTPUT + echo "has_non_docs_changes=${{ steps.changes.outputs.others }}" >> $GITHUB_OUTPUT diff --git a/.github/workflows/clang_sanitizers.yml b/.github/workflows/clang_sanitizers.yml index c82c4a2cb46..729068efdcf 100644 --- a/.github/workflows/clang_sanitizers.yml +++ b/.github/workflows/clang_sanitizers.yml @@ -5,19 +5,21 @@ on: branches: - "development" pull_request: - paths-ignore: - - "Docs/**" - - "**.rst" concurrency: group: ${{ github.ref }}-${{ github.head_ref }}-clangsanitizers cancel-in-progress: true jobs: + + check_changes: + uses: ./.github/workflows/check_changes.yml + build_UB_sanitizer: name: Clang UB sanitizer runs-on: ubuntu-24.04 - if: github.event.pull_request.draft == false + needs: check_changes + if: ${{ github.event.pull_request.draft == false && needs.check_changes.outputs.has_non_docs_changes == 'true' }} env: CC: clang CXX: clang++ @@ -79,7 +81,7 @@ jobs: build_thread_sanitizer: name: Clang thread sanitizer runs-on: ubuntu-24.04 - if: github.event.pull_request.draft == false + if: ${{ github.event.pull_request.draft == false && needs.check_changes.outputs.has_non_docs_changes == 'true' }} env: CC: clang CXX: clang++ diff --git a/.github/workflows/clang_tidy.yml b/.github/workflows/clang_tidy.yml index b348c08420d..22adbce57df 100644 --- a/.github/workflows/clang_tidy.yml +++ b/.github/workflows/clang_tidy.yml @@ -5,15 +5,16 @@ on: branches: - "development" pull_request: - paths-ignore: - - "Docs/**" - - "**.rst" concurrency: group: ${{ github.ref }}-${{ github.head_ref }}-clangtidy cancel-in-progress: true jobs: + + check_changes: + uses: ./.github/workflows/check_changes.yml + run_clang_tidy: strategy: matrix: @@ -21,7 +22,8 @@ jobs: name: clang-tidy-${{ matrix.dim }}D runs-on: ubuntu-24.04 timeout-minutes: 250 - if: github.event.pull_request.draft == false + needs: check_changes + if: ${{ github.event.pull_request.draft == false && needs.check_changes.outputs.has_non_docs_changes == 'true' }} steps: - uses: actions/checkout@v4 - name: install dependencies diff --git a/.github/workflows/cuda.yml b/.github/workflows/cuda.yml index 5a17a99e278..d219a20f731 100644 --- a/.github/workflows/cuda.yml +++ b/.github/workflows/cuda.yml @@ -5,19 +5,21 @@ on: branches: - "development" pull_request: - paths-ignore: - - "Docs/**" - - "**.rst" concurrency: group: ${{ github.ref }}-${{ github.head_ref }}-cuda cancel-in-progress: true jobs: + + check_changes: + uses: ./.github/workflows/check_changes.yml + build_nvcc: name: NVCC SP runs-on: ubuntu-22.04 - if: github.event.pull_request.draft == false + needs: check_changes + if: ${{ github.event.pull_request.draft == false && needs.check_changes.outputs.has_non_docs_changes == 'true' }} env: CXXFLAGS: "-Werror" CMAKE_GENERATOR: Ninja @@ -98,7 +100,8 @@ jobs: build_nvcc_gnumake: name: NVCC GNU Make runs-on: ubuntu-22.04 - if: github.event.pull_request.draft == false + needs: check_changes + if: ${{ github.event.pull_request.draft == false && needs.check_changes.outputs.has_non_docs_changes == 'true' }} steps: - uses: actions/checkout@v4 - name: install dependencies @@ -132,7 +135,8 @@ jobs: build_nvhpc: name: NVHPC runs-on: ubuntu-24.04 - if: github.event.pull_request.draft == false + needs: check_changes + if: ${{ github.event.pull_request.draft == false && needs.check_changes.outputs.has_non_docs_changes == 'true' }} #env: # # For NVHPC, Ninja is slower than the default: # CMAKE_GENERATOR: Ninja diff --git a/.github/workflows/hip.yml b/.github/workflows/hip.yml index cf679f67ca7..47cfb70601b 100644 --- a/.github/workflows/hip.yml +++ b/.github/workflows/hip.yml @@ -5,22 +5,24 @@ on: branches: - "development" pull_request: - paths-ignore: - - "Docs/**" - - "**.rst" concurrency: group: ${{ github.ref }}-${{ github.head_ref }}-hip cancel-in-progress: true jobs: + + check_changes: + uses: ./.github/workflows/check_changes.yml + build_hip_3d_sp: name: HIP 3D SP runs-on: ubuntu-24.04 env: CXXFLAGS: "-Werror -Wno-deprecated-declarations -Wno-error=pass-failed" CMAKE_GENERATOR: Ninja - if: github.event.pull_request.draft == false + needs: check_changes + if: ${{ github.event.pull_request.draft == false && needs.check_changes.outputs.has_non_docs_changes == 'true' }} steps: - uses: actions/checkout@v4 - name: install dependencies @@ -74,7 +76,8 @@ jobs: env: CXXFLAGS: "-Werror -Wno-deprecated-declarations -Wno-error=pass-failed" CMAKE_GENERATOR: Ninja - if: github.event.pull_request.draft == false + needs: check_changes + if: ${{ github.event.pull_request.draft == false && needs.check_changes.outputs.has_non_docs_changes == 'true' }} steps: - uses: actions/checkout@v4 - name: install dependencies diff --git a/.github/workflows/insitu.yml b/.github/workflows/insitu.yml index 35f18e46ddf..cf4eaf3ea5d 100644 --- a/.github/workflows/insitu.yml +++ b/.github/workflows/insitu.yml @@ -5,19 +5,21 @@ on: branches: - "development" pull_request: - paths-ignore: - - "Docs/**" - - "**.rst" concurrency: group: ${{ github.ref }}-${{ github.head_ref }}-insituvis cancel-in-progress: true jobs: + + check_changes: + uses: ./.github/workflows/check_changes.yml + sensei: name: SENSEI runs-on: ubuntu-24.04 - if: github.event.pull_request.draft == false + needs: check_changes + if: ${{ github.event.pull_request.draft == false && needs.check_changes.outputs.has_non_docs_changes == 'true' }} env: CXX: clang++ CC: clang @@ -43,7 +45,8 @@ jobs: ascent: name: Ascent runs-on: ubuntu-24.04 - if: github.event.pull_request.draft == false + needs: check_changes + if: ${{ github.event.pull_request.draft == false && needs.check_changes.outputs.has_non_docs_changes == 'true' }} env: CXX: g++ CC: gcc @@ -83,7 +86,8 @@ jobs: catalyst: name: Catalyst runs-on: ubuntu-24.04 - if: github.event.pull_request.draft == false + needs: check_changes + if: ${{ github.event.pull_request.draft == false && needs.check_changes.outputs.has_non_docs_changes == 'true' }} env: CXX: g++ CC: gcc diff --git a/.github/workflows/intel.yml b/.github/workflows/intel.yml index 9bc61b6c8f5..d3fc77d7c48 100644 --- a/.github/workflows/intel.yml +++ b/.github/workflows/intel.yml @@ -5,15 +5,16 @@ on: branches: - "development" pull_request: - paths-ignore: - - "Docs/**" - - "**.rst" concurrency: group: ${{ github.ref }}-${{ github.head_ref }}-intel cancel-in-progress: true jobs: + + check_changes: + uses: ./.github/workflows/check_changes.yml + build_icpx: name: oneAPI ICX SP runs-on: ubuntu-24.04 @@ -24,7 +25,8 @@ jobs: CXXFLAGS: "-Werror -Wno-error=pass-failed -Wno-tautological-constant-compare" # For oneAPI, Ninja is slower than the default: # CMAKE_GENERATOR: Ninja - if: github.event.pull_request.draft == false + needs: check_changes + if: ${{ github.event.pull_request.draft == false && needs.check_changes.outputs.has_non_docs_changes == 'true' }} steps: - uses: actions/checkout@v4 - name: install dependencies @@ -89,7 +91,8 @@ jobs: CXXFLAGS: "-Werror -Wno-tautological-constant-compare" # For oneAPI, Ninja is slower than the default: # CMAKE_GENERATOR: Ninja - if: github.event.pull_request.draft == false + needs: check_changes + if: ${{ github.event.pull_request.draft == false && needs.check_changes.outputs.has_non_docs_changes == 'true' }} steps: - uses: actions/checkout@v4 - name: install dependencies diff --git a/.github/workflows/jupyter.yml b/.github/workflows/jupyter.yml index 3c770859554..9bd7dfb7a60 100644 --- a/.github/workflows/jupyter.yml +++ b/.github/workflows/jupyter.yml @@ -5,19 +5,21 @@ on: branches: - "development" pull_request: - paths-ignore: - - "Docs/**" - - "**.rst" concurrency: group: ${{ github.ref }}-${{ github.head_ref }}-jupyter cancel-in-progress: true jobs: + + check_changes: + uses: ./.github/workflows/check_changes.yml + jupyter_notebooks: name: Notebooks runs-on: ubuntu-latest - if: github.event.pull_request.draft == false + needs: check_changes + if: ${{ github.event.pull_request.draft == false && needs.check_changes.outputs.has_non_docs_changes == 'true' }} steps: - uses: actions/checkout@v4 - name: Set up Python diff --git a/.github/workflows/macos.yml b/.github/workflows/macos.yml index e5fea8cd5cf..a9b281f6e8f 100644 --- a/.github/workflows/macos.yml +++ b/.github/workflows/macos.yml @@ -5,19 +5,21 @@ on: branches: - "development" pull_request: - paths-ignore: - - "Docs/**" - - "**.rst" concurrency: group: ${{ github.ref }}-${{ github.head_ref }}-macos cancel-in-progress: true jobs: + + check_changes: + uses: ./.github/workflows/check_changes.yml + build_appleclang: name: AppleClang runs-on: macos-latest - if: github.event.pull_request.draft == false + needs: check_changes + if: ${{ github.event.pull_request.draft == false && needs.check_changes.outputs.has_non_docs_changes == 'true' }} env: HOMEBREW_NO_INSTALLED_DEPENDENTS_CHECK: TRUE # For macOS, Ninja is slower than the default: diff --git a/.github/workflows/ubuntu.yml b/.github/workflows/ubuntu.yml index 40e7eb40c65..6f9d101da61 100644 --- a/.github/workflows/ubuntu.yml +++ b/.github/workflows/ubuntu.yml @@ -5,19 +5,21 @@ on: branches: - "development" pull_request: - paths-ignore: - - "Docs/**" - - "**.rst" concurrency: group: ${{ github.ref }}-${{ github.head_ref }}-ubuntu cancel-in-progress: true jobs: + + check_changes: + uses: ./.github/workflows/check_changes.yml + build_cxxminimal: name: GCC Minimal w/o MPI runs-on: ubuntu-24.04 - if: github.event.pull_request.draft == false + needs: check_changes + if: ${{ github.event.pull_request.draft == false && needs.check_changes.outputs.has_non_docs_changes == 'true' }} env: CXXFLAGS: "-Werror" steps: @@ -55,7 +57,8 @@ jobs: build_1D_2D: name: GCC 1D & 2D w/ MPI, QED tools runs-on: ubuntu-24.04 - if: github.event.pull_request.draft == false + needs: check_changes + if: ${{ github.event.pull_request.draft == false && needs.check_changes.outputs.has_non_docs_changes == 'true' }} env: CXXFLAGS: "-Werror" CXX: "g++-12" @@ -102,7 +105,8 @@ jobs: build_3D_sp: name: GCC 3D & RZ w/ MPI, single precision runs-on: ubuntu-22.04 - if: github.event.pull_request.draft == false + needs: check_changes + if: ${{ github.event.pull_request.draft == false && needs.check_changes.outputs.has_non_docs_changes == 'true' }} env: CXX: "g++-12" CC: "gcc-12" @@ -149,7 +153,8 @@ jobs: build_gcc_ablastr: name: GCC ABLASTR w/o MPI runs-on: ubuntu-24.04 - if: github.event.pull_request.draft == false + needs: check_changes + if: ${{ github.event.pull_request.draft == false && needs.check_changes.outputs.has_non_docs_changes == 'true' }} env: CMAKE_GENERATOR: Ninja CXXFLAGS: "-Werror" @@ -185,7 +190,8 @@ jobs: build_pyfull: name: Clang pywarpx runs-on: ubuntu-24.04 - if: github.event.pull_request.draft == false + needs: check_changes + if: ${{ github.event.pull_request.draft == false && needs.check_changes.outputs.has_non_docs_changes == 'true' }} env: CC: clang CXX: clang++ diff --git a/.github/workflows/windows.yml b/.github/workflows/windows.yml index 877290f5844..6e85f0e7f82 100644 --- a/.github/workflows/windows.yml +++ b/.github/workflows/windows.yml @@ -5,21 +5,23 @@ on: branches: - "development" pull_request: - paths-ignore: - - "Docs/**" - - "**.rst" concurrency: group: ${{ github.ref }}-${{ github.head_ref }}-windows cancel-in-progress: true jobs: + + check_changes: + uses: ./.github/workflows/check_changes.yml + build_win_msvc: name: MSVC C++17 w/o MPI runs-on: windows-latest # disabled due to issues in #5230 if: 0 - #if: github.event.pull_request.draft == false + #needs: check_changes + #if: ${{ github.event.pull_request.draft == false && needs.check_changes.outputs.has_non_docs_changes == 'true' }} steps: - uses: actions/checkout@v4 - uses: actions/setup-python@v5 @@ -70,7 +72,8 @@ jobs: build_win_clang: name: Clang C++17 w/ OMP w/o MPI runs-on: windows-2019 - if: github.event.pull_request.draft == false + needs: check_changes + if: ${{ github.event.pull_request.draft == false && needs.check_changes.outputs.has_non_docs_changes == 'true' }} steps: - uses: actions/checkout@v4 - uses: actions/setup-python@v5 From 061291924efd5af8f2b94ee7c7611106507d78d8 Mon Sep 17 00:00:00 2001 From: Edoardo Zoni <59625522+EZoni@users.noreply.github.com> Date: Mon, 12 May 2025 20:48:19 -0700 Subject: [PATCH 2/5] Add missing dependent job --- .github/workflows/clang_sanitizers.yml | 1 + 1 file changed, 1 insertion(+) diff --git a/.github/workflows/clang_sanitizers.yml b/.github/workflows/clang_sanitizers.yml index 729068efdcf..9a07729042f 100644 --- a/.github/workflows/clang_sanitizers.yml +++ b/.github/workflows/clang_sanitizers.yml @@ -81,6 +81,7 @@ jobs: build_thread_sanitizer: name: Clang thread sanitizer runs-on: ubuntu-24.04 + needs: check_changes if: ${{ github.event.pull_request.draft == false && needs.check_changes.outputs.has_non_docs_changes == 'true' }} env: CC: clang From 3766c471135b32e272f2eba0f571b1d0a4c2f85e Mon Sep 17 00:00:00 2001 From: Edoardo Zoni Date: Mon, 12 May 2025 20:58:19 -0700 Subject: [PATCH 3/5] Improve workflow job names --- .github/workflows/check_changes.yml | 1 + .github/workflows/clang_sanitizers.yml | 1 + .github/workflows/clang_tidy.yml | 1 + .github/workflows/cuda.yml | 1 + .github/workflows/hip.yml | 1 + .github/workflows/insitu.yml | 1 + .github/workflows/intel.yml | 1 + .github/workflows/jupyter.yml | 1 + .github/workflows/macos.yml | 1 + .github/workflows/ubuntu.yml | 1 + .github/workflows/windows.yml | 1 + 11 files changed, 11 insertions(+) diff --git a/.github/workflows/check_changes.yml b/.github/workflows/check_changes.yml index aebf704f1f4..f526f162ddb 100644 --- a/.github/workflows/check_changes.yml +++ b/.github/workflows/check_changes.yml @@ -10,6 +10,7 @@ on: jobs: check: + name: Check changes runs-on: ubuntu-latest outputs: has_non_docs_changes: ${{ steps.set-output.outputs.has_non_docs_changes }} diff --git a/.github/workflows/clang_sanitizers.yml b/.github/workflows/clang_sanitizers.yml index 9a07729042f..2796085a34a 100644 --- a/.github/workflows/clang_sanitizers.yml +++ b/.github/workflows/clang_sanitizers.yml @@ -13,6 +13,7 @@ concurrency: jobs: check_changes: + name: Analyze uses: ./.github/workflows/check_changes.yml build_UB_sanitizer: diff --git a/.github/workflows/clang_tidy.yml b/.github/workflows/clang_tidy.yml index 22adbce57df..b8e7bdc0de8 100644 --- a/.github/workflows/clang_tidy.yml +++ b/.github/workflows/clang_tidy.yml @@ -13,6 +13,7 @@ concurrency: jobs: check_changes: + name: Analyze uses: ./.github/workflows/check_changes.yml run_clang_tidy: diff --git a/.github/workflows/cuda.yml b/.github/workflows/cuda.yml index d219a20f731..b8a6c6db145 100644 --- a/.github/workflows/cuda.yml +++ b/.github/workflows/cuda.yml @@ -13,6 +13,7 @@ concurrency: jobs: check_changes: + name: Analyze uses: ./.github/workflows/check_changes.yml build_nvcc: diff --git a/.github/workflows/hip.yml b/.github/workflows/hip.yml index 47cfb70601b..b7f83351d4e 100644 --- a/.github/workflows/hip.yml +++ b/.github/workflows/hip.yml @@ -13,6 +13,7 @@ concurrency: jobs: check_changes: + name: Analyze uses: ./.github/workflows/check_changes.yml build_hip_3d_sp: diff --git a/.github/workflows/insitu.yml b/.github/workflows/insitu.yml index cf4eaf3ea5d..27473f9c90a 100644 --- a/.github/workflows/insitu.yml +++ b/.github/workflows/insitu.yml @@ -13,6 +13,7 @@ concurrency: jobs: check_changes: + name: Analyze uses: ./.github/workflows/check_changes.yml sensei: diff --git a/.github/workflows/intel.yml b/.github/workflows/intel.yml index d3fc77d7c48..215799f57c0 100644 --- a/.github/workflows/intel.yml +++ b/.github/workflows/intel.yml @@ -13,6 +13,7 @@ concurrency: jobs: check_changes: + name: Analyze uses: ./.github/workflows/check_changes.yml build_icpx: diff --git a/.github/workflows/jupyter.yml b/.github/workflows/jupyter.yml index 9bd7dfb7a60..49564efdb67 100644 --- a/.github/workflows/jupyter.yml +++ b/.github/workflows/jupyter.yml @@ -13,6 +13,7 @@ concurrency: jobs: check_changes: + name: Analyze uses: ./.github/workflows/check_changes.yml jupyter_notebooks: diff --git a/.github/workflows/macos.yml b/.github/workflows/macos.yml index a9b281f6e8f..9014b359c50 100644 --- a/.github/workflows/macos.yml +++ b/.github/workflows/macos.yml @@ -13,6 +13,7 @@ concurrency: jobs: check_changes: + name: Analyze uses: ./.github/workflows/check_changes.yml build_appleclang: diff --git a/.github/workflows/ubuntu.yml b/.github/workflows/ubuntu.yml index 6f9d101da61..61ceed91577 100644 --- a/.github/workflows/ubuntu.yml +++ b/.github/workflows/ubuntu.yml @@ -13,6 +13,7 @@ concurrency: jobs: check_changes: + name: Analyze uses: ./.github/workflows/check_changes.yml build_cxxminimal: diff --git a/.github/workflows/windows.yml b/.github/workflows/windows.yml index 6e85f0e7f82..e7762fef9df 100644 --- a/.github/workflows/windows.yml +++ b/.github/workflows/windows.yml @@ -13,6 +13,7 @@ concurrency: jobs: check_changes: + name: Analyze uses: ./.github/workflows/check_changes.yml build_win_msvc: From 4137ca6c5674e830f3029fb3bf0567ab7bb75fd4 Mon Sep 17 00:00:00 2001 From: Edoardo Zoni Date: Tue, 20 May 2025 14:03:05 -0700 Subject: [PATCH 4/5] Update rules to check changes --- .github/workflows/check_changes.yml | 12 ++++-------- 1 file changed, 4 insertions(+), 8 deletions(-) diff --git a/.github/workflows/check_changes.yml b/.github/workflows/check_changes.yml index f526f162ddb..1e90dfb2bab 100644 --- a/.github/workflows/check_changes.yml +++ b/.github/workflows/check_changes.yml @@ -3,8 +3,6 @@ name: Check Changes on: workflow_call: outputs: - has_docs_changes: - value: ${{ jobs.check.outputs.has_docs_changes }} has_non_docs_changes: value: ${{ jobs.check.outputs.has_non_docs_changes }} @@ -20,13 +18,11 @@ jobs: id: changes with: filters: | - docs: - - 'Docs/**' - - '**.rst' - others: + non_docs: - '!Docs/**' + - '!**.md' - '!**.rst' + predicate-quantifier: 'every' - id: set-output run: | - echo "has_docs_changes=${{ steps.changes.outputs.docs }}" >> $GITHUB_OUTPUT - echo "has_non_docs_changes=${{ steps.changes.outputs.others }}" >> $GITHUB_OUTPUT + echo "has_non_docs_changes=${{ steps.changes.outputs.non_docs }}" >> $GITHUB_OUTPUT From 1d8e14642371e697b46d1dc888782a1ae3d0c72c Mon Sep 17 00:00:00 2001 From: Edoardo Zoni Date: Tue, 20 May 2025 14:06:41 -0700 Subject: [PATCH 5/5] Update PETSc workflows --- .github/workflows/petsc.yml | 20 +++++++++++++------- 1 file changed, 13 insertions(+), 7 deletions(-) diff --git a/.github/workflows/petsc.yml b/.github/workflows/petsc.yml index b6a06ae40e6..8af4d00ab9c 100644 --- a/.github/workflows/petsc.yml +++ b/.github/workflows/petsc.yml @@ -5,19 +5,22 @@ on: branches: - "development" pull_request: - paths-ignore: - - "Docs/**" - - "**.rst" concurrency: group: ${{ github.ref }}-${{ github.head_ref }}-petsc cancel-in-progress: true jobs: + + check_changes: + name: Analyze + uses: ./.github/workflows/check_changes.yml + test-petsc-gcc: name: GCC runs-on: ubuntu-latest - if: github.event.pull_request.draft == false + needs: check_changes + if: ${{ github.event.pull_request.draft == false && needs.check_changes.outputs.has_non_docs_changes == 'true' }} env: CXXFLAGS: "-Werror" steps: @@ -72,7 +75,8 @@ jobs: test-petsc-mpich: name: GCC-12 + MPICH runs-on: ubuntu-latest - if: github.event.pull_request.draft == false + needs: check_changes + if: ${{ github.event.pull_request.draft == false && needs.check_changes.outputs.has_non_docs_changes == 'true' }} env: CXXFLAGS: "-Werror" steps: @@ -127,7 +131,8 @@ jobs: test-petsc-gcc-debug: name: GCC [Debug] runs-on: ubuntu-latest - if: github.event.pull_request.draft == false + needs: check_changes + if: ${{ github.event.pull_request.draft == false && needs.check_changes.outputs.has_non_docs_changes == 'true' }} env: CXXFLAGS: "-Werror" steps: @@ -185,7 +190,8 @@ jobs: test-petsc-mpich-debug: name: GCC-12 + MPICH [Debug] runs-on: ubuntu-latest - if: github.event.pull_request.draft == false + needs: check_changes + if: ${{ github.event.pull_request.draft == false && needs.check_changes.outputs.has_non_docs_changes == 'true' }} env: CXXFLAGS: "-Werror" steps: