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

Guard against this.options.babel.plugins #5986

Closed
wants to merge 3 commits into from

Conversation

dcyriller
Copy link
Contributor

@dcyriller dcyriller commented Apr 8, 2019

This would fix the emberaddons.com job.

I've not been able to precisely track down the root cause of the failure. My hypothesis is that a slight change in the dependencies resolution has occured (with ember-cli-babel?).

@runspired
Copy link
Contributor

@dcyriller seems to still happen once I made the packages hoist their dev-dependencies which I think reduces the chances that it's due to a weird resolution. This guard seems fine although we're still failing, so it's more likely an issue with babel resolution in general. I wonder if it is related to the failures in #5967

@dcyriller
Copy link
Contributor Author

Error in emberaddons job is:

Invalid Option: The plugins/built-ins 'transform-block-scoping,transform-typeof-symbol' passed to the 'exclude' option are not
    valid. Please check data/[plugin-features|built-in-features].js in babel-preset-env

This seems to indicate that the ember-data babel transpilation involves an old version of @babel/preset-env.

emberaddons.com consumes babel-preset-env@1.6.1 though ember-cli-babel@6.8.1. Two babel plugins used in ember-data babel build (specifically transform-block-scoping, transform-typeof-symbol) are not supported yet in this babel-preset-env version.

So, ember-data would be processed in emberaddons build with emberaddons' {ember-cli,}babel versions. But I would expect ember-data transpilation to be done with its own {ember-cli,}babel versions (as in ember-cli every addon controls its own transpilation 🤔).

Also, I can't reproduce this failure locally (same fresh clone with the same Node and friends versions).

I will close this PR, as it doesn't fix the issue. Maybe the results of my investigation will help someone debugging this!

@dcyriller
Copy link
Contributor Author

I've pushed a commit to use a fork of emberaddons. In this fork, ember-cli-babel is upgraded to latest. Let's see what happens.

@dcyriller
Copy link
Contributor Author

Interestingly, the build error is fixed but an other error pops up:

Error: Could not find module `@ember-data/model` imported from `ember-data/-private`

I'll close the PR as it doesn't fix the issue.

@dcyriller dcyriller closed this Apr 11, 2019
@dcyriller dcyriller deleted the fix-emberaddons-job branch April 11, 2019 16:08
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.

None yet

2 participants