Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

v3+: Can we use "node:<buildinName>" as import specifier in v3+? #4595

Closed
frank-dspeed opened this issue Jul 31, 2022 · 7 comments · Fixed by #4549
Closed

v3+: Can we use "node:<buildinName>" as import specifier in v3+? #4595

frank-dspeed opened this issue Jul 31, 2022 · 7 comments · Fixed by #4549

Comments

@frank-dspeed
Copy link
Contributor

frank-dspeed commented Jul 31, 2022

Expected Behavior / Situation

i type something like

cat my.js | grep "node:"

and get all nodejs buildin modules that are required by the module

Actual Behavior / Situation

i write the command and i get nothing

Modification Proposal

import { readFile } from "fs/promises";
// should be for example
import { readFile } from "node:fs/promises";

Target Branch

@lukastaegert
Copy link
Member

I put it into #4596

@lukastaegert
Copy link
Member

This issue has been resolved via #4596 as part of rollup@3.0.0-4. Note that this is a pre-release, so to test it, you need to install Rollup via npm install rollup@3.0.0-4 or npm install rollup@beta. It will likely become part of a regular release later.

@rollup-bot
Copy link
Collaborator

This issue has been resolved via #4596 as part of rollup@3.0.0-5. Note that this is a pre-release, so to test it, you need to install Rollup via npm install rollup@3.0.0-5 or npm install rollup@beta. It will likely become part of a regular release later.

@rollup-bot
Copy link
Collaborator

This issue has been resolved via #4596 as part of rollup@3.0.0-6. Note that this is a pre-release, so to test it, you need to install Rollup via npm install rollup@3.0.0-6 or npm install rollup@beta. It will likely become part of a regular release later.

@rollup-bot
Copy link
Collaborator

This issue has been resolved via #4596 as part of rollup@3.0.0-7. Note that this is a pre-release, so to test it, you need to install Rollup via npm install rollup@3.0.0-7 or npm install rollup@beta. It will likely become part of a regular release later.

@rollup-bot
Copy link
Collaborator

This issue has been resolved via #4596 as part of rollup@3.0.0-8. Note that this is a pre-release, so to test it, you need to install Rollup via npm install rollup@3.0.0-8 or npm install rollup@beta. It will likely become part of a regular release later.

@rollup-bot
Copy link
Collaborator

This issue has been resolved via #4596 as part of rollup@3.0.0. You can test it via npm install rollup.

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

Successfully merging a pull request may close this issue.

3 participants