rollup / rollup-plugin-commonjs

This module has moved and is now available at @rollup/plugin-commonjs / https://github.com/rollup/plugins/blob/master/packages/commonjs
MIT License
502 stars 126 forks source link

Update README.md with note on symlinks #405

Closed bterlson closed 5 years ago

bterlson commented 5 years ago

This addresses the confusion reported in #402

IMO it would be nice to make this easier on people by supporting includeModule/excludeModule, but in the meantime we should document symlinks a bit better.

bterlson commented 5 years ago

Thanks for the review @shellscape, I've taken your suggestions.

bterlson commented 5 years ago

@shellscape can you double check the latest commit? I think the docs should prefer the module name form for namedExports since it's easier and more reliable.