Repository navigation
chore(deps): bump runtime deps - #43902
Conversation
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
|
|
||
| for (const [pkg, licenses] of Object.entries(EXPECTED)) { | ||
| test(`${pkg} bundles ship .LICENSE files with expected package counts`, async ({ exec, tmpWorkspace }) => { | ||
| test(`${pkg} bundles ship .LICENSE files with expected package counts`, { annotation: { type: 'issue', description: 'https://github.com/microsoft/playwright/issues/43899' } }, async ({ exec, tmpWorkspace }) => { |
There was a problem hiding this comment.
restore the test.
There was a problem hiding this comment.
Restored the test to its base version, removing the added report-generation assertions and issue annotation. Both license tests pass.
Revised on Linux by the Playwright bot - agent run
|
Hi, I'm the Playwright bot. I started to address the review comments on Linux: agent run |
|
Hi, I'm the Playwright bot. I restored the bundle-license test to its base version, including removal of the added issue annotation. The dependency updates and license-report fix are unchanged. Build and full lint passed; the restored license tests and adjacent bundle tests passed (5 tests). A temporary check confirmed that license generation fails without the report fix and passes with it. These installation tests do not use browsers. Revised on Linux by the Playwright bot - agent run |
9829ce8 to
41c11a7
Compare
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
41c11a7 to
05acff7
Compare
This comment has been minimized.
This comment has been minimized.
34e5aa6 to
e65ee03
Compare
This comment has been minimized.
This comment has been minimized.
Test results for "tests 1"2 flaky23584 passed, 704 skipped Merge workflow run. |
Test results for "MCP"5194 passed, 616 skipped Merge workflow run. |
Summary
Fixes #43899