Skip to content
This repository was archived by the owner on Feb 26, 2024. It is now read-only.

fix(bundling): switch to using umd bundles#457

Merged
juliemr merged 1 commit intoangular:masterfrom
juliemr:namefix
Sep 19, 2016
Merged

fix(bundling): switch to using umd bundles#457
juliemr merged 1 commit intoangular:masterfrom
juliemr:namefix

Conversation

@juliemr
Copy link
Copy Markdown
Member

@juliemr juliemr commented Sep 19, 2016

This means that we will stop polluting the global namespace when
zone.js is included.

Fixes #456

@juliemr
Copy link
Copy Markdown
Member Author

juliemr commented Sep 19, 2016

note: I've included the updates from running npm run prepublish here because it makes the review more meaningful.

Let me know if you'd like me to revert the changes to stuff in dist/

Comment thread dist/async-test.js Outdated
@@ -1,3 +1,9 @@
(function (global, factory) {
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We should add copyright here:

/**
 * @license
 * Copyright Google Inc. All Rights Reserved.
 *
 * Use of this source code is governed by an MIT-style license that can be
 * found in the LICENSE file at https://angular.io/license
 */

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Also while it is nice to see final product, do revert dist folder. before checking in.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Added copyright.

ACK for reverting dist folder, will do before checking in.

Comment thread dist/async-test.js
(factory());
}(this, (function () { 'use strict';

(function () {
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All of the specs have their own closure which auto-detects global. We should remove these?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done

@juliemr
Copy link
Copy Markdown
Member Author

juliemr commented Sep 19, 2016

Responded to comments and updated the dist files - I'll squash and revert the dist files before merging if this looks good to you.

This means that we will stop polluting the global namespace when
zone.js is included.

Fixes angular#456
@juliemr juliemr merged commit 8dd06e5 into angular:master Sep 19, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants