ckolderup/picaro — pull request #7 by hoverbird (2012-01-31 04:49:46+00:00)
Title: Break picaro.js into modules

Let me know what you think of this direction. Each file is a "module", and the first line of each file declares what other modules it depends on. Require.js uses those dependencies to figure out what order to load the scripts in, and when the app is ready to run... eventually it can also be used to bundle things up efficiently.

Anything not declared at the top of a file is effectively invisible to it- this makes the whole system a lot easier to reason about, and to test. The Inventory, for instance, has no knowledge of anything outside of itself, making it super easy to test.

I recommend ignoring the final commit as that's just indentation changes made now that everything is wrapped. I think github must provide a way to only look at the diff provided by the first 4 commits...?