-
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
[Bugfix] this.pickFiles did not exist on addon #1892
Conversation
@@ -212,7 +212,7 @@ Addon.prototype.jshinting = function() { | |||
}; | |||
|
|||
Addon.prototype.addonJsFiles = function(tree) { | |||
var files = pickFiles(tree, { | |||
var files = this.pickFiles(tree, { | |||
srcDir: '/', |
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.
May need to call this._requireBuildPackages() method here first.
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.
isn't it in the constructor?
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.
ah its not, should it be in the constructor instead?
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.
If it were in the constructor we would pay the initialize hit even when the addon does nothin build related.
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.
Also would not allow using ember-cli to build ember either (due to es6ify issue indicated in the original issue).
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.
where do we eagerly instantiate the add-on?
- i'll split the up as you recommended.
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.
eec13be
to
2fee486
Compare
2fee486
to
12c507d
Compare
[Bugfix] this.pickFiles did not exist on addon
No description provided.