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

Adds babel-preset-env #531

Merged
merged 2 commits into from
Sep 6, 2017
Merged

Adds babel-preset-env #531

merged 2 commits into from
Sep 6, 2017

Conversation

delambo
Copy link
Member

@delambo delambo commented Aug 28, 2017

Replaces babel-preset-latest with babel-preset-env in babel-preset-kyt-core.

Because the lerna hoisting doesn't work for all dependencies, I added some hoisting exceptions to the lerna.json configuration and a script that installs the starter kyt dependencies but keeps the lerna symlinks. That way, starter kyt commands like kyt dev can work against the local repository. One drawback to the hoisting that I haven't figured out is that kyt test does not work against the starter kyts, but I'm not terribly concerned about that since the starter kyt unit tests are for show.

@delambo delambo force-pushed the adds-babel-preset-env branch 3 times, most recently from d5dd2b5 to 908cd3e Compare August 28, 2017 01:55
tweaks

adds hot-loader

npm version

adds kyt eslint

fix test
@delambo delambo force-pushed the adds-babel-preset-env branch from 908cd3e to 6e688af Compare August 28, 2017 02:18
Copy link
Contributor

@tizmagik tizmagik left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Noyce! Excited to finally get this in! Thank you! 🙏

@tizmagik
Copy link
Contributor

As a follow-on PR (or maybe as part of this one) we should also set BABEL_ENV to server or client for the 2 builds. Server can be set to "node": "current" and the client to "browsers": ["last 2 versions", "safari >= 7"] (or whatever as a good default).

@delambo delambo merged commit c3cc155 into master Sep 6, 2017
@delambo delambo deleted the adds-babel-preset-env branch September 6, 2017 21:59
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.

3 participants