Some small perf stuff: - Githubissues

ef4 / ember-browserify

ember-cli addon for easily loading CommonJS packages from npm via browserify.

MIT License

172 stars 28 forks source link

Some small perf stuff: #74

Closed stefanpenner closed 8 years ago

stefanpenner commented 8 years ago

use already provided listFiles instead of walking the disk again
don’t invalidate the output if nothing has changed

stefanpenner commented 8 years ago

On cache change, it still must read all input files to checksum them. This takes a good amount of time, especially when an app has thousands of files. I'm planning on tackling that next, this is up for an interim r?

ef4 commented 8 years ago

👍

stefanpenner commented 8 years ago

@ef4 FYI, i don't have merge/commit/release so i can't act on the 👍 yet.

stefanpenner commented 8 years ago

~~this may have an issue, looking into it now.~~ unrelated

stefanpenner commented 8 years ago

https://github.com/ef4/ember-browserify/pull/75 will actually supersede this