Skip to content

Required check stuck as “Expected” when PR only touches scripts/** (workflow skipped by paths-ignore) #1483

@benceruleanlu

Description

@benceruleanlu

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-windows in .github/workflows/debug_all.yml.
  • .github/workflows/debug_all.yml has on.pull_request.paths-ignore (and push.paths-ignore) including scripts/**.
  • 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

  1. Adjust .github/workflows/debug_all.yml paths-ignore so scripts/downloadFrontend.js (or all scripts/**) triggers the workflow.
  2. Update branch protection required checks to not require this job when the workflow is path-filtered.
  3. Add a manual escape hatch: allow running debug_all.yml via workflow_dispatch and ensure branch protection accepts manually-triggered runs.

Example

┆Issue is synchronized with this Notion page by Unito

Metadata

Metadata

Assignees

No one assigned

    Labels

    bugSomething isn't working

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions