@redmunds do you have cycles to review this one? Feel free to reassign if not.

Glenn and I couldn't think of any good reason why we didn't make this a submodule originally, so it seems ok to merge assuming the code changes look good.

@TuckerWhitehouse there are still some whitespace diffs in brackets-concat.js -- can you try to back those out?

Also -- are the SHAs pointing to the same versions of Require, text, and i18n that we had before? Or is this upgrading us to the latest versions?

@peterflynn I reverted that change to brackets-concat.js (learned a lot about git and rebasing this evening...)
The SHA's are pointing to upgraded versions of each (whatever the latest was when I did the submodule add 9 days ago or so).

• RequireJS: v2.1.1 >> v2.1.5
• Text: v1.0.8 >> v2.0.5
• i18n: v1.0.0 >> v2.0.2

Edit: I added another commit, updating to the latest versions

• RequireJS: v2.1.5
• Text: v2.0.6

@TuckerWhitehouse can you update your pull request? There's a merge conflict.

@jasonsanjose I think I fixed it (Conflict in NOTICE for some odd reason?)

Looks like the latest errors from Travis are due to connectivity to GitHub and not any errors from running grunt.

Reviewing...

This looks good. @jasonsanjose Are you sure Travis is happy? :)

Actually, I was able to kick off a new build...I think this is a new Travis feature. Under the settings icon, "Restart Build". Travis is happy now.

Merging.