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

latest 0.33.0 release breaks apps bootstrapped with requirejs/systemjs #848

Closed
3cp opened this issue Mar 19, 2018 · 4 comments
Closed

latest 0.33.0 release breaks apps bootstrapped with requirejs/systemjs #848

3cp opened this issue Mar 19, 2018 · 4 comments

Comments

@3cp
Copy link
Member

3cp commented Mar 19, 2018

I'm submitting a bug report

  • Library Version:
    0.33.0

  • Loader/bundler:
    SystemJS | RequireJS

Current behavior:

  • What is the expected behavior?
  • What is the motivation / use case for changing the behavior?

@EisenbergEffect latest 0.33.0 release breaks apps bootstrapped with requirejs/systemjs

The gulp-babel version was upgraded from 7.0.1 to 8.0.0-beta.2 in a routine version pump.

"gulp-babel": "^8.0.0-beta.2",

But 8.0.0-beta.2 requires additional peer dependency on @babel/core.

Even after manually install @babel/core, the app would not build properly, gulp complains

gulp-notify: [Error running Gulp] Error: .code is only allowed in root programmatic options

Revert back to 7.0.1 solves the issue. This issue doesn't affect webpack since it doesn't use gulp-babel.

@EisenbergEffect
Copy link
Contributor

Released fix! Apologies!

@tjad
Copy link

tjad commented Jan 3, 2019

I've run into a similar issue since updating my dependencies. What is the actual meaning / cause of this error?

@tjad
Copy link

tjad commented Jan 3, 2019

I generated a fresh project and swapped out my .babelrc for the new .babelrc.js which is not 100% yet, but looks much healthier!

@3cp
Copy link
Member Author

3cp commented Jan 3, 2019

It could be you have babel6 in your existing app. When you upgrade cli on an app you haven’t touched for quite a while, there are lots of other deps have to be upgraded too.

It is easier to create a fresh app skeleton then copy over your existing source code.

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

No branches or pull requests

3 participants