Skip to content

Revert "feat(list reporter): add printWorkerIndex option to prefix output with worker index" - #43149

Merged
Dmitry Gozman (dgozman) merged 1 commit into
mainfrom
revert-42886-list-worker-index
Oct 6, 2026
Merged

Dmitry Gozman (dgozman) merged 1 commit into
mainfrom
revert-42886-list-worker-index

Conversation

@pavelfeldman

Copy link
Copy Markdown
Member

Reverts #42886

@dgozman
Dmitry Gozman (dgozman) merged commit f9864e5 into main Oct 6, 2026
42 of 43 checks passed
@dgozman
Dmitry Gozman (dgozman) deleted the revert-42886-list-worker-index branch October 6, 2026 15:49
@github-actions

github-actions Bot commented Oct 6, 2026

Copy link
Copy Markdown
Contributor

Test results for "tests 1"

1 failed
❌ [playwright-test] › playwright.spec.ts:303 › should respect headless in modifiers that run before tests @ubuntu-latest-node24

5 flaky ⚠️ [chromium-library] › library/chromium/css-coverage.spec.ts:112 › should report stylesheets across navigations `@chromium-ubuntu-22.04-node24`
⚠️ [chromium-library] › library/webmcp.spec.ts:216 › should scope tools to their frame `@chromium-ubuntu-22.04-node24`
⚠️ [chromium-library] › library/webmcp.spec.ts:238 › should call same-name tools in their own frame `@chromium-ubuntu-22.04-node22`
⚠️ [firefox-library] › library/browsercontext-cookies-third-party.spec.ts:257 › third party 'Partitioned;' cookies `@firefox-ubuntu-22.04-node20`
⚠️ [firefox-library] › library/browsercontext-cookies-third-party.spec.ts:470 › top level 'Partitioned;' cookie and same origin iframe `@firefox-ubuntu-22.04-node20`

52644 passed, 1270 skipped


Merge workflow run.

@github-actions

github-actions Bot commented Oct 6, 2026

Copy link
Copy Markdown
Contributor

🟢 Playwright bot here — I checked the CI failure, and it is a confirmed pre-existing flake

playwright.spec.ts:303 › should respect headless in modifiers that run before tests has failed on unrelated PRs and passed on rerun. This PR only reverts a list-reporter output option, so it does not reach browser launch or headless configuration.

Details

Pre-existing flake / infra

  • [playwright-test] › playwright.spec.ts:303 › should respect headless in modifiers that run before tests — the nested test launched headed Chromium without an available X server (Missing X server or $DISPLAY). The same test failed on unrelated #42096 and #41460, then cleared on #41460's rerun. The changes here are limited to the list reporter and its docs, types, and test, with no changes to the fixture or launch paths exercised by this test.

Triaged by the Playwright bot - agent run

@github-actions

github-actions Bot commented Oct 6, 2026

Copy link
Copy Markdown
Contributor

Test results for "MCP"

9035 passed, 1495 skipped


Merge workflow run.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants