search

rollup / rollup-plugin-node-resolve

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

Warn about using deprecated jsnext:main package.json fields #158

Closed TrySound closed 5 years ago

TrySound commented 6 years ago

Ref #78 We may keep this deprecation for a while and then start to deprecate jsnext option.

guybedford commented 6 years ago

Is the idea to get everyone to use module rather? I'm just wondering if we should try and wait until it is clear what Node will be doing here to avoid churn?

TrySound commented 6 years ago

module is widely used right now. I just would like to prevent using even older stuff which everyone should omit.

guybedford commented 6 years ago

I just think deprecations should always have clear and stable upgrade paths. And until the NodeJS modules story is set this is not a stable upgrade path. So I would rather leave something working and stale than constantly asking people to change something and build resentment over time.

lukastaegert commented 5 years ago

I believe this is now obsolete after introduction of the mainfields option (that basically deprecated jsnext as well).