Added commonjs support

[emiluen]

No description provided.

[marceljuenemann]

I'm not really familiar with CommonJS, is that compatible with RequireJS?! What is the impact of this PR on users that don't use CommonJS? Are all users that use the main attribute of the package.json using CommonJS?

I think this PR looks good to me though, unlike #301 it doesn't add anything for users that don't use CommonJS.

[emiluen]

The index.js file will make the angular module into a CommonJS module, and exports the name of that angular module. This makes it possible to import in nodejs/webpack/es6 and so on. For users that don't use CommonJS, the angular-drag-and-drop-lists.js is unchanged, so it will not make a difference. Other modules use the same way to support CommonJS, for example: https://github.com/danialfarid/ng-file-upload/blob/master/index.js https://github.com/angular-ui/bootstrap/blob/master/index.js

…

On Tue, Jan 17, 2017 at 12:51 AM, Marcel Juenemann ***@***.*** > wrote: I'm not really familiar with CommonJS, is that compatible with RequireJS?! What is the impact of this PR on users that don't use CommonJS? Are all users that use the main attribute of the package.json using CommonJS? I think this PR looks good to me though, unlike #301 <#301> it doesn't add anything for users that don't use CommonJS. — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#350 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/ARep5Ldir_LF4SfwP755B7DSDmsWiFaJks5rTAJogaJpZM4KsZy8> .

[marceljuenemann]

Thanks! Can you resolve the merge conflict? Then I'll merge the PR

[graingert]

@marceljuenemann @emiluen I've rebased this for you in #388

[skyzito]

Hey guys , any trouble to accept this PR ?

[MatheusArleson]

any trouble with this PR?