From 5893156b541de57b24530f37e2512d306d3b1987 Mon Sep 17 00:00:00 2001 From: Sunyoung Yoo <17974113+YooSunYoung@users.noreply.github.com> Date: Thu, 26 Mar 2026 14:02:19 +0100 Subject: [PATCH] Refactor CI failure condition logic I found out that skipped and success and failure are not the only conditions... --- .github/workflows/ci.yml | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml index 18934354..ea215149 100644 --- a/.github/workflows/ci.yml +++ b/.github/workflows/ci.yml @@ -128,6 +128,6 @@ jobs: 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') + !( needs.formatting.result == 'success' && + (needs.test.result == 'skipped' || needs.test.result == 'success') && + (needs.docs.result == 'skipped' || needs.docs.result == 'success') )