Simplify webpack alias

[jasonaden]

Using __dirname directly in the path-mapping.js file should be more reliable and should always be accurate. This simplifies setting up the aliases for Webpack to map to the esm distribution.

Output in the path-mapping.js file will now look like this:

"use strict"

var path = require('path');

module.exports = function() {
  return {
    "rxjs/AsyncSubject": path.resolve(__dirname, "AsyncSubject.js"),
    "rxjs/add/observable/interval": path.resolve(__dirname, "add/observable/interval.js"),
    "rxjs/util/tryCatch": path.resolve(__dirname, "util/tryCatch.js"),
    // ...
};
}

[rxjs-bot]

	Messages
📖

CJS: 2276.4KB, global: 746.2KB (gzipped: 120.0KB), min: 145.7KB (gzipped: 31.2KB)

Generated by 🚫 dangerJS

[coveralls]

Coverage remained the same at 97.399% when pulling b2ee797 on jasonaden:simplify_webpack_alias into a0b7362 on ReactiveX:master.

[kwonoj]

Change itself looks fine. I'd personally would capture __dirname once

function() {
  var dir = path.resolve(__dirname);
  return {
    "rxjs/AsyncSubject": path.join(dir, "AsyncSubject.js"),
}

like, but not sure how much gain it'll be honestly. probably not worth it?

[jasonaden]

@kwonoj Sounds good. I made that change.

[coveralls]

Coverage remained the same at 97.399% when pulling 63c1521 on jasonaden:simplify_webpack_alias into a0b7362 on ReactiveX:master.

[lock]

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.