From 3ac23aa208d0f2bf54fec4c6d5e60a440a0490fb Mon Sep 17 00:00:00 2001 From: Philippe Canal Date: Wed, 19 Aug 2026 12:37:25 -0500 Subject: [PATCH] [ci] Match event_file skip ci conditional with build_* The discrepancy was spotted by MAI-Code-1.1-Flash: Skip logic is inconsistent between jobs The build jobs check both the PR title and the skip ci label, but the event_file job at root-ci.yml:692-708 only checks the title. That means a PR can be skipped in the build jobs but still produce an event artifact, which is inconsistent and makes debugging harder. --- .github/workflows/root-ci.yml | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/.github/workflows/root-ci.yml b/.github/workflows/root-ci.yml index a7820f201a3f4..97cee2d758717 100644 --- a/.github/workflows/root-ci.yml +++ b/.github/workflows/root-ci.yml @@ -641,7 +641,11 @@ jobs: # can be skipped if the tag [skip-ci] or [skip ci] is written in the title. if: | (github.repository_owner == 'root-project' && github.event_name != 'pull_request') || - (github.event_name == 'pull_request' && !(contains(github.event.pull_request.title, '[skip-ci]') || contains(github.event.pull_request.title, '[skip ci]'))) + (github.event_name == 'pull_request' && !( + contains(github.event.pull_request.title, '[skip-ci]') || + contains(github.event.pull_request.title, '[skip ci]') || + contains(github.event.pull_request.labels.*.name, 'skip ci') + )) name: "Upload Github event context for the test summary step" runs-on: ubuntu-latest