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
Export package.json #908
Export package.json #908
Conversation
".": { | ||
"import": "./src/index.js", | ||
"require": "./dist/index.cjs" | ||
}, | ||
"./package.json": "./package.json" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Are we able to remove this then?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't understand the question, sorry
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Are we now able to remove "./package.json": "./package.json"
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That's one of the main things I'm adding in this PR. It's essential to achieving the aims of the PR, namely being able to import/require 'node-fetch/package.json' in nodejs 12.7.0+
Member of the Node.js modules team here. It is likely a very good idea to land this change. |
@jimmywarting Are you able to release a new beta containing this update? |
@wheresrhys See #912 |
Thank you |
@wheresrhys Released |
Fixes #907 (see the issue for context)