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

Ensure ember-data/-private module is emitted properly for ember-cli… #5031

Merged
merged 1 commit into from
Jun 24, 2017

Conversation

stefanpenner
Copy link
Member

… < 2.12.

ember-cli@2.12 and higher have the addon itself responsible for transpiling
modules so that the output of the treeForAddon hook is expected to be
AMD.

Prior versions of ember-cli (i.e. < 2.12) do second pass (that is not possible
to disable) to transpile the output of treeForAddon again (to transpile
modules).

This commit changes things around leveraging ember-cli-babel@6's built in
ability to detect the various ember-cli version scenarios and either
transpile ember-data/-private down to AMD or not correctly.

index.js Outdated
let bowerDeps = this.project.bowerDependencies();
this.debugTree = BroccoliDebug.buildDebugCallback('ember-data');

var bowerDeps = this.project.bowerDependencies();
Copy link
Member

Choose a reason for hiding this comment

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

🔪 var

@stefanpenner stefanpenner force-pushed the forward-port branch 2 times, most recently from 736f7cc to d7a8215 Compare June 22, 2017 22:08
@runspired
Copy link
Contributor

🎉

… < 2.12.

ember-cli@2.12 and higher have the addon itself responsible for transpiling
modules so that the output of the `treeForAddon` hook is expected to be
AMD.

Prior versions of ember-cli (i.e. < 2.12) do second pass (that is not possible
to disable) to transpile the output of `treeForAddon` **again** (to transpile
modules).

This commit changes things around leveraging ember-cli-babel@6's built in
ability to detect the various ember-cli version scenarios and either
transpile `ember-data/-private` down to AMD or not correctly.
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

4 participants