jrburke/r.js — pull request #51 by jwhitley (2011-12-06 01:48:34+00:00)
Title: Optional optimize callback

The second parameter to `requirejs.optimize`, `callback`, isn't optional but it appears that it should be. This patch makes the single-param usage of `requirejs.optimize` in [this gist](https://gist.github.com/1419063) work correctly.

**NOTE:** The [test included here](https://github.com/jwhitley/r.js/commit/538d540bc11bd18da95b4ecbf2957335e2d59a6c) is broken. Without the subsequent commit, it throws an exception, but the test harness doesn't report the error. This may be due to the callback being omitted in the new test; I'm not familiar with the test setup here.

This reminds me that I haven't filed a Dojo Foundation CLA yet. I'll get on that ASAP, whether or not it's needed here.