ci(build): PR-only path filter for web-static & qt-webengine (slice 1) - #897
Open
frstrtr wants to merge 1 commit into
Open
ci(build): PR-only path filter for web-static & qt-webengine (slice 1)#897frstrtr wants to merge 1 commit into
frstrtr wants to merge 1 commit into
Conversation
On pull_request, skip the web-static verify and Qt WebEngine leak-gate jobs when their inputs did not change (web-static/** for web; ui/**, src/**, and the build system for qt). On push, tag and workflow_dispatch the filter is bypassed and every job runs, so the master rollup and release builds remain authoritative. None of the gated jobs are branch-protection required checks and nothing needs: them, so a skip reports as Skipped (not a false-green on a required gate) with no downstream cascade. Required coin/consensus gates are left unfiltered; filtering those safely needs an always-run aggregator plus a branch-protection re-point (operator action), tracked separately.
frstrtr
force-pushed
the
ci-steward/ci-path-filter
branch
from
July 28, 2026 04:09
b6920e2 to
e03ea0d
Compare
frstrtr
marked this pull request as ready for review
July 28, 2026 04:09
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
1 participant
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.
ci(build): PR-only path filter for web-static & qt-webengine
Adds a
changesjob (dorny/paths-filter) that lets the two costliestnon-required jobs skip on a PR when their inputs did not change.
"Web-static verify" is our top recurring gate-drag (wedged 4h44m on #930,
forced two manual run cancels, left #880 with a CANCELLED required check).
(a) The filter CANNOT skip a job whose inputs changed
Explicit trigger paths (a PR touching ANY of these RUNS the job):
web-static runs when the PR touches:
web-static/**(the entire web source + visual harness — the job runsnpm run visualand uploads fromweb-static/sharechain-explorer/tests/visual/out/, all under this prefix).github/workflows/build.yml(self-referential: any change to this workflow re-runs it)qt-webengine runs when the PR touches:
ui/**(the job configures/buildsui/c2pool-qt)src/**,CMakeLists.txt,conanfile*,cmake/**(over-inclusive on the C++/build surface — errs toward running).github/workflows/build.ymlThe filter is bypassed entirely on
push, tag, andworkflow_dispatch(
|| github.event_name != 'pull_request'), so the master rollup andrelease builds always run the full set and stay authoritative.
(b) No per-coin source-presence guard relaxed (#47 protection intact)
Only
web-staticandqt-webenginegainneeds: changes+ anif:.No per-coin build/smoke job and none of the #47 source-presence guards
are touched. Diff is a single file (
.github/workflows/build.yml, +35).Safety
None of the gated jobs are branch-protection required checks and nothing
needs:them — a skip reports as Skipped, never a false-green on arequired gate, with no downstream cascade. Required coin/consensus gates
are deliberately left unfiltered (that needs an always-run aggregator +
branch-protection re-point, tracked separately).