rusackas opened a new pull request, #40780:
URL: https://github.com/apache/superset/pull/40780
### SUMMARY
Follow-up to #40772. **Docs-only PRs currently can't merge** — they hang on
`cypress-matrix (0, chrome)`, `cypress-matrix (1, chrome)`, and
`playwright-tests (chromium)` showing *"Expected — Waiting for status to be
reported"* (e.g. #38108).
**Root cause** (same as #40772): `cypress-matrix` and `playwright-tests` are
matrix jobs gated on change detection (`python || frontend`). On a PR that
touches neither group — a docs-only PR — they're skipped at the **job
level**,
which happens *before* matrix expansion, so the per-combination contexts are
never produced. Branch protection requires them → it waits forever.
The frontend Dependabot case (#40772) didn't expose this for the e2e jobs
because a frontend change sets `frontend=true`, so cypress/playwright run.
Only docs-only / neither-group PRs hit it — and the docs "build" people
expect
(the Netlify Deploy Preview) does run and pass; it just isn't a required GHA
check, so the orphaned cypress/playwright contexts are what actually gate.
**Fix:** add always-running `cypress-matrix-required` /
`playwright-tests-required`
anchors that pass when the underlying job is `success` **or** `skipped`, and
require those instead of the matrix-expanded names. A single
`cypress-matrix-required` replaces both shard contexts. The matrix jobs stay
fully skipped on unrelated PRs.
`.asf.yaml`:
- `cypress-matrix (0, chrome)` + `cypress-matrix (1, chrome)` →
`cypress-matrix-required`
- `playwright-tests (chromium)` → `playwright-tests-required`
With this, every required matrix job (`unit-tests`, `test-postgres` via
#40772;
`cypress-matrix`, `playwright-tests` here) has a stable anchor, so PRs of any
shape — python, frontend, or docs-only — can satisfy branch protection.
### TESTING INSTRUCTIONS
- Docs-only PR: cypress/playwright skip; the anchors run and pass →
mergeable.
- Code PR: the matrix jobs run; the anchors pass after them (and fail if any
shard genuinely fails).
### ADDITIONAL INFORMATION
- [ ] Has associated issue:
- [ ] Changes UI
- [ ] Includes DB Migration
- [ ] Introduces new feature or API
- [ ] Removes existing feature or API
🤖 Generated with [Claude Code](https://claude.com/claude-code)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
