Skip to content
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

Remove jsnext:main support from repo #83

Closed
shellscape opened this issue Dec 8, 2019 · 1 comment · Fixed by #152
Closed

Remove jsnext:main support from repo #83

shellscape opened this issue Dec 8, 2019 · 1 comment · Fixed by #152

Comments

@shellscape
Copy link
Collaborator

  • Rollup Plugin Name: node-resolve, all others
  • Rollup Plugin Version: n / a

Expected Behavior / Situation

n / a

Actual Behavior / Situation

After @TrySound mentioned that jsnext:main is deprecated, I did some digging and asking around. Rollup was really the only bundler that embraced it fully. As mentioned by Rich Harris here, jsnext:main has been superseded by module, and there are numerous sources that consider it deprecated as a result (for example).

Basically it never made it out of discussion, and Rollup supported it out of necessity at the time it was implemented.

Modification Proposal

So all that goes to say is that we have sufficient evidence to remove support from @rollup/node-resolve and to remove that field from any plugins in the repo.

@shellscape
Copy link
Collaborator Author

#85 took care of most of the work. Going to leave this open as several modules have to get a version bump and get published.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant