-
Notifications
You must be signed in to change notification settings - Fork 38
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
Wiser, safer vendor includes #78
Conversation
We discussed offline over slack. Please move the css tree to Rest LGTM. |
index.js
Outdated
content => `if (typeof FastBoot === 'undefined') { ${content} }` | ||
); | ||
|
||
return new MergeTrees(vendorTree, dropzoneJs); |
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.
new MergeTrees([vendorTree, dropzoneJs]);
index.js
Outdated
} | ||
); | ||
|
||
return new MergeTrees(styleTree, dropzoneCss); |
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.
Same here.
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.
Seems trying to merge with a null tree throws an error Expected Broccoli node, got undefined for inputNodes[0]
Hey, this looks good. Could you rebase with the latest changes on the dev branch? Thanks! |
Had to rebase with newer changes. Added these changes to most recent commit d9355e3 |
This fixes #76- @kratiahuja, maybe you could give your 2 cents?