From 29aaa4436f0879fe32d19c11f3c8bce94d197d45 Mon Sep 17 00:00:00 2001 From: Sunyoung Yoo <17974113+YooSunYoung@users.noreply.github.com> Date: Fri, 6 Mar 2026 10:40:52 +0100 Subject: [PATCH 1/5] Add report job status step to CI workflow --- .github/workflows/ci.yml | 9 +++++++++ 1 file changed, 9 insertions(+) diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml index 0085ec85..5fe7dd18 100644 --- a/.github/workflows/ci.yml +++ b/.github/workflows/ci.yml @@ -87,3 +87,12 @@ jobs: environments: docs-${{ matrix.package }} - name: Build docs run: pixi run -e docs-${{ matrix.package }} docs ${{ matrix.package }} + + report: + name: Report Job Status + needs: [changes, formatting, test, docs] + if : success() + runs-on: ubuntu-24.04 + steps: + - name: Report Job Status + run: echo "All Tests Succeeded" From ad26b104397609b1c6a6208b38e2f63fc9e4d10a Mon Sep 17 00:00:00 2001 From: Sunyoung Yoo <17974113+YooSunYoung@users.noreply.github.com> Date: Fri, 6 Mar 2026 10:43:51 +0100 Subject: [PATCH 2/5] Comment out package check in CI workflow Commented out the condition checking for empty packages in test and docs jobs. --- .github/workflows/ci.yml | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml index 5fe7dd18..f221c45f 100644 --- a/.github/workflows/ci.yml +++ b/.github/workflows/ci.yml @@ -51,7 +51,7 @@ jobs: test: name: Test ${{ matrix.package }} needs: [changes, formatting] - if: needs.changes.outputs.packages != '[]' + # if: needs.changes.outputs.packages != '[]' strategy: fail-fast: false matrix: @@ -71,7 +71,7 @@ jobs: docs: name: Docs ${{ matrix.package }} needs: [changes, test] - if: needs.changes.outputs.packages != '[]' + # if: needs.changes.outputs.packages != '[]' strategy: fail-fast: false matrix: From 6001a04e10a413b55435b46992c77a6d1403282c Mon Sep 17 00:00:00 2001 From: Sunyoung Yoo <17974113+YooSunYoung@users.noreply.github.com> Date: Fri, 6 Mar 2026 10:47:01 +0100 Subject: [PATCH 3/5] Update CI workflow conditions for tests and docs --- .github/workflows/ci.yml | 9 ++++++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml index f221c45f..c53c9015 100644 --- a/.github/workflows/ci.yml +++ b/.github/workflows/ci.yml @@ -51,7 +51,7 @@ jobs: test: name: Test ${{ matrix.package }} needs: [changes, formatting] - # if: needs.changes.outputs.packages != '[]' + if: needs.changes.outputs.packages != '[]' strategy: fail-fast: false matrix: @@ -71,7 +71,7 @@ jobs: docs: name: Docs ${{ matrix.package }} needs: [changes, test] - # if: needs.changes.outputs.packages != '[]' + if: needs.changes.outputs.packages != '[]' strategy: fail-fast: false matrix: @@ -91,7 +91,10 @@ jobs: report: name: Report Job Status needs: [changes, formatting, test, docs] - if : success() + if: | + always() && + (needs.test.result == 'skipped' || needs.test.result == 'success') && + (needs.docs.result == 'skipped' || needs.docs.result == 'success') runs-on: ubuntu-24.04 steps: - name: Report Job Status From cc53c20d9de49912b4335a975ff668ca828083aa Mon Sep 17 00:00:00 2001 From: Sunyoung Yoo <17974113+YooSunYoung@users.noreply.github.com> Date: Fri, 6 Mar 2026 10:56:28 +0100 Subject: [PATCH 4/5] Refactor CI job status reporting logic --- .github/workflows/ci.yml | 20 +++++++++++++++----- 1 file changed, 15 insertions(+), 5 deletions(-) diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml index c53c9015..2458e19f 100644 --- a/.github/workflows/ci.yml +++ b/.github/workflows/ci.yml @@ -91,11 +91,21 @@ jobs: report: name: Report Job Status needs: [changes, formatting, test, docs] - if: | - always() && - (needs.test.result == 'skipped' || needs.test.result == 'success') && - (needs.docs.result == 'skipped' || needs.docs.result == 'success') + if: always() runs-on: ubuntu-24.04 steps: - - name: Report Job Status + - name: Report Job Success run: echo "All Tests Succeeded" + if: | + needs.formatting.result == 'success' && + (needs.test.result == 'skipped' || needs.test.result == 'success') && + (needs.docs.result == 'skipped' || needs.docs.result == 'success') + + - name: Report Job Failure + run: | + echo "Not All Tests Succeeded..." + exit 1 + if: | + needs.formatting.result != 'success' || + (needs.test.result != 'skipped' && needs.test.result == 'failure') || + (needs.docs.result != 'skipped' && needs.docs.result == 'failure') From 67f29bceeffb9ac7b301b2c33342759afa76cedd Mon Sep 17 00:00:00 2001 From: Sunyoung Yoo <17974113+YooSunYoung@users.noreply.github.com> Date: Fri, 6 Mar 2026 11:00:45 +0100 Subject: [PATCH 5/5] Remove white spaces --- .github/workflows/ci.yml | 1 - 1 file changed, 1 deletion(-) diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml index 2458e19f..3f754600 100644 --- a/.github/workflows/ci.yml +++ b/.github/workflows/ci.yml @@ -100,7 +100,6 @@ jobs: needs.formatting.result == 'success' && (needs.test.result == 'skipped' || needs.test.result == 'success') && (needs.docs.result == 'skipped' || needs.docs.result == 'success') - - name: Report Job Failure run: | echo "Not All Tests Succeeded..."