From 825fcd6a3120491c85b7db13635dd7b68a516045 Mon Sep 17 00:00:00 2001 From: Joe Fusco Date: Wed, 19 Aug 2026 10:29:44 -0700 Subject: [PATCH] fix: make playwright.yml workflow_call-only like its CI siblings playwright.yml had its own pull_request/push triggers and concurrency block on top of being called as a job from ci.yml. Concurrency expressions in a called reusable workflow resolve in the caller's context, so its group collapsed to the exact same group as ci.yml's own top-level concurrency ("CI-refs/heads/main"), and GitHub cancelled the nested playwright job before it could run -- flipping ci.yml's conclusion to failure on nearly every push even though every visible job passed. Dropping the standalone triggers/concurrency, to match phpcs.yml/phpstan.yml/phpunit.yml/plugin-check.yml/typos.yml, removes the collision and stops Playwright from running twice per push. --- .github/workflows/playwright.yml | 13 ------------- 1 file changed, 13 deletions(-) diff --git a/.github/workflows/playwright.yml b/.github/workflows/playwright.yml index 4b8cd5c..73f37b4 100644 --- a/.github/workflows/playwright.yml +++ b/.github/workflows/playwright.yml @@ -2,19 +2,6 @@ name: Playwright on: workflow_call: - pull_request: - paths: - - '**.php' - - '**.ts' - - 'tests/e2e/**' - - 'playwright.config.ts' - - '.github/workflows/playwright.yml' - push: - branches: [main] - -concurrency: - group: ${{ github.workflow }}-${{ github.ref }} - cancel-in-progress: ${{ github.ref != 'refs/heads/main' }} permissions: contents: read