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
Use broccoli-funnel instead of broccoli-static-compiler #3672
Conversation
awesome, by sure though that: |
tests failures look race related. I suspect we recently regressed on something related... in all likely-hood one of or deps has a race again... |
this appears to be a duplicate of: #3621 |
Ah, yes, now I'm getting the tests to cover my code. So things are OK then. Travis is green too. Re dup, I caught a few extra lines I think, so you should prefer my PR :) |
Ya. Done :) |
Use broccoli-funnel instead of broccoli-static-compiler
Gotta figure out those races. These brittle tests have been getting worse this week |
ah, I forgot add-ons depend on having access to |
@@ -364,7 +363,7 @@ Addon.prototype.treeForPublic = function(tree) { | |||
return tree; | |||
} | |||
|
|||
return this.pickFiles(tree, { |
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.
specifically this case.
broccoli-funnel has the
allowEmpty
option that we needed CustomStaticCompiler for.I didn't get any failing tests, so I'm not 100% sure that the code in
lib/models/addon.js
works properly, but it definitely looks correct. :)