From ca1b91c21c748e402efaba17348a0afb42f36d6c Mon Sep 17 00:00:00 2001 From: Joshua Napoli Date: Thu, 20 Aug 2026 08:06:06 -0400 Subject: [PATCH] [PD1-138] Standardize Continuous Integration workflow ## Why The Dependabot review-and-merge automation we run in `ui` reacts to CI through a `workflow_run` trigger, and `workflow_run` matches a workflow by its **display name**, not its filename. Rolling that automation out across the fleet therefore needs one agreed name in every Python and JavaScript repository. The standard, now documented in the `defining-github-workflows` skill in `meta-software`, is: - the workflow lives at `.github/workflows/ci.yml` and is named `CI`; - it triggers on `pull_request`, and on `push` restricted to the default branch, so a feature branch does not run CI twice and the automation can rely on the pull request run; - any `paths:` filter includes the dependency manifest and lock file, so a dependency-only pull request still runs CI; - it never skips `dependabot[bot]`, since a skipped run means no automated review. ## What changed - Renamed the workflow from `Continuous Integration` to `CI`. - Limited `push` runs to `main`; pull request runs already cover feature branches. No job, step, or matrix entry changed, so the checks that run are the same ones as before. Co-Authored-By: Claude Opus 5 (1M context) --- .github/workflows/ci.yml | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml index a2d0cc4..e94e161 100644 --- a/.github/workflows/ci.yml +++ b/.github/workflows/ci.yml @@ -1,4 +1,4 @@ -name: Continuous Integration +name: CI on: pull_request: @@ -9,6 +9,7 @@ on: - src/** - tests/** push: + branches: [main] paths: - .github/workflows/ci.yml - uv.lock