Skip to content

Use "node:fs" instead of "fs" #11

Closed
boenrobot wants to merge 1 commit intosindresorhus:mainfrom
boenrobot:runtimeCompat
Closed

Use "node:fs" instead of "fs" #11
boenrobot wants to merge 1 commit intosindresorhus:mainfrom
boenrobot:runtimeCompat

Conversation

@boenrobot
Copy link
Copy Markdown

@boenrobot boenrobot commented Jul 26, 2024

And bump the minimum Nodejs version to the versions that support this feature.

Dev dependencies are also bumped to the newest ones that can run on said minimal nodejs versions and do not require further changes to the code or tests.

Also added Windows to the CI tests and updated actions/setup-node to v4. Excluded Node 14 from MacOS, as those binaries are not available anymore.


This improves compatibility with runtimes other than nodejs (workerd in particular is known to only provide node compatibility layer for "node:" prefixed modules) at the cost of leaving behind older nodejs versions.

@boenrobot boenrobot force-pushed the runtimeCompat branch 3 times, most recently from d5df099 to ce9a5a2 Compare July 26, 2024 10:23
… feature.

Dev dependencies are also bumped to the newest ones that can run on said minimal nodejs versions and do not require further changes to the code or tests.

Also added Windows to the CI tests and updated `actions/setup-node` to `v4`. Excluded Node 14 from MacOS, as those binaries are not available anymore.
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.

1 participant