-
Notifications
You must be signed in to change notification settings - Fork 172
Open
Labels
bugSomething isn't workingSomething isn't working
Description
When a PR only changes files under scripts/** (e.g. scripts/downloadFrontend.js in PR #1479), the required status check build-and-test-e2e-windows / integration-windows-test stays in “Expected — Waiting for status to be reported” and blocks merging.
Why this happens
- The required check is produced by the reusable workflow call job
build-and-test-e2e-windowsin.github/workflows/debug_all.yml. .github/workflows/debug_all.ymlhason.pull_request.paths-ignore(andpush.paths-ignore) includingscripts/**.- So for script-only changes, the entire workflow is skipped, meaning the required check is never created/reported.
Impact
- If branch protection requires
build-and-test-e2e-windows / integration-windows-test, script-only PRs become unmergeable unless checks are manually bypassed.
Possible fixes
- Adjust
.github/workflows/debug_all.ymlpaths-ignoresoscripts/downloadFrontend.js(or allscripts/**) triggers the workflow. - Update branch protection required checks to not require this job when the workflow is path-filtered.
- Add a manual escape hatch: allow running
debug_all.ymlviaworkflow_dispatchand ensure branch protection accepts manually-triggered runs.
Example
┆Issue is synchronized with this Notion page by Unito
Metadata
Metadata
Assignees
Labels
bugSomething isn't workingSomething isn't working