Skip to content

test: skip yarn-berry example runs on Node 20 - #185

Merged
Pavel Feldman (pavelfeldman) merged 1 commit into
microsoft:mainfrom
pavelfeldman:skip-yarn-berry-node20
Oct 5, 2026
Merged

Pavel Feldman (pavelfeldman) merged 1 commit into
microsoft:mainfrom
pavelfeldman:skip-yarn-berry-node20

Conversation

@pavelfeldman

Copy link
Copy Markdown
Member

The build (20.x, ubuntu-latest) and build (20.x, macos-latest) jobs fail on main because, on Node 20, yarn playwright test cannot load Playwright's ESM loader from the Yarn PnP zip cache (ERR_MODULE_NOT_FOUND .../playwright/lib/transform/esmLoader.js). Skip the two "run examples" tests for yarn-berry on Node 20; everything else still runs there.

Comment thread tests/integration.spec.ts
});

test('should generate be able to run TS examples successfully', { lock: 'apt-get' }, async ({ run, dir, exec, packageManager }) => {
test.skip(packageManager === 'yarn-berry' && process.version.startsWith('v20.'), 'Playwright ESM loader cannot be loaded from the Yarn PnP cache on Node 20');

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Node 20 is EOL'ed, why would we support create-playwright for it, let's just update the bots.

@pavelfeldman
Pavel Feldman (pavelfeldman) merged commit 6831799 into microsoft:main Oct 5, 2026
10 of 13 checks passed
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