From 58c339af7aedd36589a2399ccc0a36fcf4e61f8d Mon Sep 17 00:00:00 2001 From: Tiffany Phan Date: Mon, 15 Jun 2026 15:47:45 -0700 Subject: [PATCH] ci: trigger eval-skills on every PR so required checks always report MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Branch protection requires "Unit tests", "Aggregate scores", and "Evaluate gate" — all of which come from this workflow. With the `paths:` filter, PRs that don't touch `skills/**`, `evals/**`, or this file never trigger the workflow, so the three required checks stay forever in "Expected — Waiting for status to be reported" and block merge. (#74 hit this.) Remove the trigger-level `paths:` filter so the workflow always runs on PRs. The existing diff job and per-job `if:` conditions already short-circuit the real work when no skills changed, and GitHub treats skipped required checks as passing. --- .github/workflows/eval-skills.yml | 11 +++++++---- 1 file changed, 7 insertions(+), 4 deletions(-) diff --git a/.github/workflows/eval-skills.yml b/.github/workflows/eval-skills.yml index e8d3a5e..b8c3dd1 100644 --- a/.github/workflows/eval-skills.yml +++ b/.github/workflows/eval-skills.yml @@ -13,11 +13,14 @@ name: Skill Evals # touching one skill costs roughly one suite's worth of API tokens. on: + # No `paths:` filter on pull_request — branch protection requires + # "Unit tests", "Aggregate scores", and "Evaluate gate" on every PR, and a + # path-filtered workflow leaves those checks stuck in "Expected — Waiting + # for status to be reported" for PRs that don't touch skills/ or evals/, + # blocking merge. The workflow fires on every PR; the `diff` job and the + # job-level `if:` conditions ensure that unchanged-skills PRs short-circuit + # to skipped (which counts as success for required checks). pull_request: - paths: - - "skills/**" - - "evals/**" - - ".github/workflows/eval-skills.yml" schedule: # 09:17 UTC daily - off the hour to avoid lining up with API rate limits. - cron: "17 9 * * *"