Richienb/node-polyfill-webpack-plugin

[Feature Req]: Can we include `fs` module polyfill to the plugin?

AkshitAggarwal opened this issue · 4 comments

This plugin is super helpful, so far most of the core modules are included as I can see. But I had to include the polyfill for fs module separately. Can we include that in the global plugin itself?

Which fs polyfill is used?

@Richienb there is no polyfill for fs in the browser, hence #45.

@afonsojramos What's the user experience with using 'fs': false?
Does it necessitate adding a note about it in the readme?

The point of this repo is to facilitate polyfills, and since there is no polyfill for fs, we might as well set it to false to set it as an empty implementation. Even some upgrade guides of webpack v4 to v5 suggest it.