refactor(commonjs): Added support for commonjs

[aaronroberson]

Added ui.router to module exports so that after installing via npm, the package can be required in the angular module dependency list such as:

angular.module('myApp', [require('angular-ui-router')]);

This is helpful for Browserify and Webpack.

[nateabele]

Patch looks fine. Please fix the PR per https://radify.github.io/PR.js/#/angular-ui/ui-router/pulls/1960 -- thanks.

[aaronroberson]

I've update the PR to reflect the requirements but it is not updating in radify. Am I missing something?

[aaronroberson]

@nateabele: I forgot to tag you in my response.

[christopherthielen]

@aaronroberson it's the subject of the commit, not the PR

[aaronroberson]

How do I change that?

On Tue, May 19, 2015, 6:26 PM Chris Thielen [email protected]
wrote:

@aaronroberson https://github.com/aaronroberson it's the subject of the
commit, not the PR

—
Reply to this email directly or view it on GitHub
#1960 (comment)
.

[nateabele]

You need to fix the commit message itself. I think git commit --amend -m "refactor(package): Added support for commonjs"; git push -f.

[aaronroberson]

Worked like a charm, thank you! I wasn't aware of the --amend flag until
now :D

On Wed, May 20, 2015 at 8:44 AM Nate Abele [email protected] wrote:

You need to fix the commit message itself. I think git commit --amend -m
"refactor(package): Added support for commonjs"; git push -f.

—
Reply to this email directly or view it on GitHub
#1960 (comment)
.

[nateabele]

@aaronroberson Awesome. Glad I could help with your new thing learned today. 😄