-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Incorporate addons into EmberApp. #1025
Conversation
👍 |
woow! 📈 |
💰 💰 💰 💰 💰 💰 💰 💰 💰 💰 💰 💰 💰 💰 💰 💰 💰 💰 💰 💰 💰 💰 |
|
||
EmberApp.prototype.addonTreesFor = function(step) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
step
is unclear, maybe type
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Updated.
Incorporate the `Project` models, recent addon support into `EmberApp`. Uses the following hooks/functions that are called on each addon (if the method exists): * `included` -- Called at the end of the `EmberApp` constructor. * `treesFor` -- Called at various points in the building up of the final dist tree. Receives the name of the step that is being processed. The return value is expected to be a Broccoli tree. The following steps are available with this PR: * `app` * `styles` * `vendor` ---- For a sample addon please take a look at: [ember-cli-pretender](https://github.com/rjackson/ember-cli-pretender/blob/master/index.js) `ember-cli-pretender` can be used (with these changes) via the following steps: ```bash npm install -g ember-cli ember new some-awesome-app cd some-awesome-app npm install --save-dev ember-cli-pretender ember server ``` No modifications are needed to the `Brocfile.js` at all.
<3 this is a great start. |
Incorporate addons into EmberApp.
👍 |
Yay! (y) |
this is just great! 😆 |
Forgive my ignorance, but how do I import Pretender into a test file? When I instantiate a I have tried a variety of |
This is a JSHint warning. You either need to add Pretender to he predef section in your tests/.jshintrc, or add a file specific comment to whitelist it. |
Thanks so much for the quick response. I'll modify the .jshintrc. |
Incorporate the
Project
models, recent addon support intoEmberApp
.Uses the following hooks/functions that are called on each addon (if the method exists):
included
-- Called at the end of theEmberApp
constructor.treesFor
-- Called at various points in the building up of the final dist tree. Receives the name of the step that is being processed. The return value is expected to be a Broccoli tree. The following steps are available with this PR:app
styles
vendor
For a sample addon please take a look at: ember-cli-pretender
ember-cli-pretender
can be used (with these changes) via the following steps:npm install -g ember-cli ember new some-awesome-app cd some-awesome-app npm install --save-dev ember-cli-pretender ember server
No modifications are needed to the
Brocfile.js
at all.