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.

Sign up for GitHub

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

Jump to bottom

ESM packages (public-ip) need to be imported. Use 'import' to reference the package instead. #60

Closed
VishwaGauravIn opened this issue Feb 1, 2022 · 2 comments
Closed

ESM packages (public-ip) need to be imported. Use 'import' to reference the package instead. #60

VishwaGauravIn opened this issue Feb 1, 2022 · 2 comments

Comments

@VishwaGauravIn
Copy link

image
@sindresorhus
Copy link
Owner

https://gist.github.com/sindresorhus/a39789f98801d908bbc7ff3ecc99d99c

@noamsan
Copy link

noamsan commented Feb 20, 2022 •
edited

I am also experiencing this issue. I realize there is no call to require() in the library.

This issue is also referenced in this SO question, which actually references your normalize-url package @sindresorhus. Is it mayberelated to one of public-ip's dependencies that uses require()?

The issue also appears in this SO question, in which the node-fetch dependency is supposedly the issue.

Repository owner locked as resolved and limited conversation to collaborators Feb 21, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@sindresorhus @noamsan @VishwaGauravIn