[PD1-138] Standardize Continuous Integration workflow - #7
Merged
joshuanapoli merged 1 commit intoAug 20, 2026
Merged
Conversation
Copy link
Copy Markdown
Member
Sorry, something went wrong.
Copy link
Copy Markdown
Member
Author
|
The license-check failure is pre-existing and unrelated: Python License Check has been failing on main for the last several runs. This pull request only renames the CI workflow and narrows the push trigger. |
Sorry, something went wrong.
CharlesKleeven
approved these changes
Aug 20, 2026
CharlesKleeven
left a comment
Copy link
Copy Markdown
There was a problem hiding this comment.
Thank you
Sorry, something went wrong.
## 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) <noreply@anthropic.com>
joshuanapoli
force-pushed
the
jn/pd1-138/standardize-ci
branch
from
f6549e8 to
b6b579d
Compare
August 20, 2026 18:42
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
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:
What changed
No job, step, or matrix entry changed, so the checks that run are the same ones as before.