Skip to content

Detect esm builtins protocol change on node 12.20.0 - #1332

Merged
cspotcode merged 8 commits into
mainfrom
ab/detect-esm-builtins-protocol-change-on-node-12_20_0
May 23, 2021
Merged

Detect esm builtins protocol change on node 12.20.0#1332
cspotcode merged 8 commits into
mainfrom
ab/detect-esm-builtins-protocol-change-on-node-12_20_0

Conversation

@cspotcode

@cspotcodecspotcode commented May 23, 2021

Copy link
Copy Markdown
Collaborator

Fixes#1331

Adds node 12.16 to the test matrix
Run ESM tests on node 12.16 and up; previously was skipping them on <13
Updates tests to pass --experimental-modules where demanded by old versions of node
Add native module test to ./tests/esm-node-resolver to match ./tests/esm

@codecov

codecovBot commented May 23, 2021

Copy link
Copy Markdown

Codecov Report

Merging #1332 (8745dd8) into main (6b8323e) will increase coverage by 0.10%.
The diff coverage is n/a.

Impacted FilesCoverage Δ
src/esm.ts85.10% <0.00%> (+2.12%)⬆️

@cspotcodecspotcode added this to the 10.0.0 milestone May 23, 2021
@cspotcode
cspotcodeforce-pushed the ab/detect-esm-builtins-protocol-change-on-node-12_20_0 branch from 11ab0bf to 9ab0fb6CompareMay 23, 2021 00:47
@cspotcode
cspotcode merged commit 373b2be into mainMay 23, 2021
@cspotcode
cspotcode deleted the ab/detect-esm-builtins-protocol-change-on-node-12_20_0 branch October 11, 2021 03:46
Sign up for freeto 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.

Resolution of node builtins in ESM loader fails on node >=12.20.0, <13

1 participant

@cspotcode