Skip to content

CI: path-filter PRs so ofxiOS does not run the full matrix - #8650

Draft
danoli3 wants to merge 1 commit into
openframeworks:masterfrom
danoli3:ci-path-filter
Draft

danoli3 wants to merge 1 commit into
openframeworks:masterfrom
danoli3:ci-path-filter

Conversation

@danoli3

@danoli3 danoli3 commented Nov 5, 2026

Copy link
Copy Markdown
Member

Path-filter PRs so a change like #8648 (three ofxiOS files) does not start VS, Android, msys2, RPi, emscripten, and two Linux GCC jobs.

Behaviour

Event Jobs
push to master / bleeding full matrix (unchanged)
PR touching addons/ofxiOS/**, scripts/ci/ios/**, scripts/ci/tvos/**, ios/tvOS templates build-ios-tvos only
PR touching scripts/ci/vs/**, scripts/vs/**, VS templates/projects build-vs only
PR touching macos/osx CI and templates build-macos only
PR touching core (libs/openFrameworks, other addons, .github/workflows, …) full matrix

Linux lib paths only runs when linux scripts/projects change.

Push of a feature branch to openframeworks/openFrameworks no longer runs OF (that was a second full matrix next to the PR when stacking #8648).

Merge gate

Job CI succeeds when every started job succeeded. Skipped jobs are OK. Point branch protection at CI rather than every matrix cell, or skipped VS/Android checks will look red on ofxiOS PRs.

Example

#8648 would run changes + build-ios-tvos (ios + tvos) + CI (~3–5 min) instead of ~27 jobs.

Master and bleeding still run every job. Pull requests use
dorny/paths-filter:

  addons/ofxiOS, ios/tvOS CI and templates → build-ios-tvos
  scripts/ci/vs and VS projects/templates → build-vs
  macos/osx CI and templates → build-macos
  anything else → full matrix

Push is limited to master and bleeding so stacking a branch onto
openframeworks no longer starts a second full OF run beside the PR.

Linux lib paths only runs on linux path changes. The CI job is the
merge gate: skipped jobs are OK, a failure is not.
@danoli3
danoli3 marked this pull request as draft November 5, 2026 06:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant