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

babel-preset-latest => babel-preset-env #359

Closed
wants to merge 15 commits into from
Closed

Conversation

tizmagik
Copy link
Contributor

@tizmagik tizmagik commented Jan 7, 2017

Closes #312

Still todo for this PR:

  • should use "node":"current" babel target setting for server build (docs)
  • add note about browser targets for userland babel configs

cc @jaredmcdonald

@c0b
Copy link

c0b commented Jan 25, 2017

coming from #356 I'm not sure how to use this PR if I merge it locally; as the docs mentioned I have created a .babelrc but it seems kyt is not honoring it

@tizmagik
Copy link
Contributor Author

tizmagik commented Jan 25, 2017

Ah, sorry for the confusion. The latest kyt on npm (0.3.0) does not honor a local .babelrc -- that will happen in 0.4.0 (which is what master reflects currently). For now, you'll need to manually override babel settings in modifyWebpackConfig() in kyt.config.js

If you're saying you've tried 0.4.0 (master branch) locally and it's not honoring your local .babelrc then that might be a bug...

* master: (26 commits)
  Expand version range of yarn for Travis (#400)
  update migration guide to 0.4.0 (#398)
  Update CHANGELOG.md
  adds 040 final version (#392)
  0.4.0 docs (#338)
  adds yarn version check to bootstrap (#391)
  Making kyt a dependency rather than a devDependency (#387)
  adds more functional tests (#388)
  rc5 (#384)
  fixes stylelintrc glob so that it respects file extensions (#383)
  prep work for rc3 (#381)
  Interactive setup (#378)
  Update webpack.base.js (#365)
  Adds monorepo bootstrap (#377)
  fixes linter peer dependencies (#380)
  Fix typo (#379)
  kyt-cli version bump (#376)
  updates the kyt-cli version (#370)
  fixes cli starter kyt path bug (#369)
  Adds prep for 0.4.0 release candidate (#368)
  ...
@delambo
Copy link
Member

delambo commented Mar 10, 2017

@tizmagik - Would it be safe to merge this as-is and followup with the other tasks?

@tizmagik
Copy link
Contributor Author

Yea maybe, let me see what's going on with the tests

delambo added 2 commits March 27, 2017 20:50
adds starter kyt beta versions
delambo and others added 5 commits March 28, 2017 21:07
* master:
  updates starter kyts to kyt 0.5.1 (#452)
# Conflicts:
#	bootstrap.js
#	packages/babel-preset-kyt-core/yarn.lock
#	packages/babel-preset-kyt-react/yarn.lock
#	packages/kyt-cli/yarn.lock
#	packages/kyt-core/package.json
#	packages/kyt-core/yarn.lock
#	packages/kyt-starter-static/package.json
#	packages/kyt-starter-static/starter-src/package.json
#	packages/kyt-starter-static/starter-src/yarn.lock
#	packages/kyt-starter-universal/package.json
#	packages/kyt-starter-universal/starter-src/package.json
#	packages/kyt-starter-universal/starter-src/yarn.lock
#	yarn.lock
@tizmagik
Copy link
Contributor Author

Closing in favor of #531

@tizmagik tizmagik closed this Aug 28, 2017
@staylor staylor deleted the babel-preset-env branch November 17, 2020 03:09
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.

4 participants