-
Notifications
You must be signed in to change notification settings - Fork 31
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
[node-modules-polyfill] Replace unmaintained rollup-plugin-node-polyfills
dependency
#18
Comments
MichaelDeBoey
changed the title
[node-modules-polyfill] Replace
[node-modules-polyfill] Replace unmaintained Jun 29, 2022
rollup-plugin-node-polyfills
dependencyrollup-plugin-node-polyfills
dependency
Any update on this ? |
@giuseppeg Is this fixed in @FredKSchott's @vafanassieff #19 implements the fix |
This was referenced Jan 28, 2023
6 tasks
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Same reasoning as in ionic-team/rollup-plugin-node-polyfills#32.
Since that repo is never touched in the last 3 years, it's probably a good idea to either look at the community forks (see ionic-team/rollup-plugin-node-polyfills#28) or create our own fork.
I would prefer to use @FredKSchott fork as he's the creator of
snowpack
& that's the one used by thesnowpack
library too, so it probably has the most up to date version of therollup-plugin-node-polyfills
lib.Edit: @FredKSchott's fork is also recommended by the @rollup team: https://rollupjs.org/guide/en/#warning-treating-module-as-external-dependency
The text was updated successfully, but these errors were encountered: