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

Drop broken jsnext:main #1327

Closed
wants to merge 1 commit into from
Closed

Drop broken jsnext:main #1327

wants to merge 1 commit into from

Conversation

taion
Copy link
Contributor

@taion taion commented Jan 30, 2016

Closes #1042

Per #1042 (comment), it looks like there's not much benefit in adding a proper ES2015 module build, and the current jsnext:main pointing at untranspiled code doesn't accomplish anything other than breaking rollup users.

As I recall, there are remain other reasons for continuing to distribute src/ (Flow-related?), so I didn't drop that from files.

@gaearon
Copy link
Contributor

gaearon commented Jan 30, 2016

Shall we do something like https://github.com/rackt/react-redux/pull/256/files? I'm open to distributing a proper ES6 module build.

@taion
Copy link
Contributor Author

taion commented Jan 30, 2016

Ah – I didn't see that, and thought #1042 was lingering. Yes, that's a better way to go if you want to distribute an ES2015 build. Easier with Babel 5, too. I'll close this out for now, then.

@taion taion closed this Jan 30, 2016
@taion taion deleted the drop-jsnext-main branch January 30, 2016 19:09
@taion
Copy link
Contributor Author

taion commented Jan 30, 2016

BTW, this is all we have to do with Babel 5: remix-run/react-router#2530.

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

Successfully merging this pull request may close these issues.

2 participants