Merge Chad's changes from https://github.com/broccolijs/broccoli-filter/pull/28

[zackthehuman]

This merge's Chad's changes from broccolijs/broccoli-filter#28 into this plugin.

[stefanpenner]

Awesome! Will review and merge if good tonight. This plus @kellyselden fix of hash for dep may let us get babel/jshint persistent out this weekend!

[zackthehuman]

That's awesome! Let me know if there is anything I can do.

[stefanpenner]

• Ember-cli's internal config replacer needs to stop invaliding on each rebuild
• broccoli-jshint broccoli-jscs both also need to stop always rebuilding

Something more related to this, is getting broccoli-babel-transpiler using this pr

[stefanpenner]

@zackthehuman window support has landed for hash-for-dep stefanpenner/hash-for-dep@335d69b

[zackthehuman]

@stefanpenner Shall I bump the version in package.json in this branch?

[stefanpenner]

Yup and remove the windows check.

[zackthehuman]

Bumping to 1.0.0 of hash-for-dep break the tests. On my mac this happens: Error: Cannot find module '/User.//package.json'.

I haven't tracked down the source of the issue yet, though.

[kellyselden]

@zackthehuman It looks like my change may have broke OSX. @stefanpenner We may need to add OSX testing in Travis to prevent this.

[stefanpenner]

merged the two git histories (as best as possible) and merged manually.

[stefanpenner]

looking into the OSX issue now

[stefanpenner]

@zackthehuman & @kellyselden I am unable to replicate that issue on OSX

@zackthehuman can you share reproduction steps?

[zackthehuman]

@stefanpenner & @kellyselden Maybe my OSX issue was a fluke. I just pulled the latest from master and that branch works fine. Previously the steps to reproduce were: npm install and then npm test. Anyway, looks good to me now.

[stefanpenner]

Anyway, looks good to me now.

lets assume a fluke, but keep in on our radar.

Just a headsup, im working to fix the last bugs with using this and the broccoli-babel-transpiler then i'll move on the jshint and jscs.