upgrade cg-busy 4.1.4 to npm

[ghernandez81]

[ghernandez81]

currently in npm it's 4.1.0

[gabehayes]

Also, there is a tag missing for v4.1.4, so you can't event set a branch/tag target in the interim.

[cgross]

Thats because there is no 4.1.4. The latest ver is 4.1.3.

[Valko54]

Please, can you upgrade angular-busy in NPM ? (npm install [email protected] doesn't exist)

[axelson]

+1

[benoror]

+1 Hi, I would also benefit from having it updated, let me give you my reason:

Support for Webpack + CommonJS Package Manager

Motivation

I'm in the process of refactoring an app to support modern JS tooling like NPM, Webpack & ES6.

Code Change

A small change is needed to support CommonJS:

/* commonjs package manager support (eg componentjs) */
if (typeof module !== "undefined" && typeof exports !== "undefined" && module.exports === exports){
  module.exports = 'cgBusy';
}

• Source: ui-router release
• Or: UMD (Universal Module Definition)

Working Examples

I have done other PRs for some of my dependencies:

• https://github.com/wix/angular-tree-control/pull/152
• https://github.com/gdi2290/angular-md5/issues/8
  • https://github.com/gdi2290/angular-md5/pull/9
• https://github.com/muratcorlu/angular-urlify/pull/1
• https://github.com/tarkanlar/Angular-Urlify/pull/1

Sources

• Using Angular 1.x With ES6 and Webpack by Jesus Rodriguez
• Webpack & Angular by Shawn McKay
• Angular and Webpack for Modular Applications by @kentcdodds
• NPM for Everything

[benoror]

BTW @gabehayes you can use latest master commit with the following format in package.json:

"angular-busy": "git+https://github.com/cgross/angular-busy.git"

[myflowpl]

+1 but i think there should be module.exports = 'cgBusy' instead of module.exports = 'angular-md5' then it's compatibile with CommonJS and Angular style (and Webpack)

[benoror]

You're right, fixed! Any chance we could se an npm release soon @cgross ?

[bikegriffith]

Would love to see this so I don't have to fork and maintain to use in my webpack-driven project