This PR, when merged will leave us with publishing only one UMD bundle. This one UMD bundle should replace our current SFX bundle, as discussed in #5665 (a separate PR with removal will follow).

This PR also fixes #5593 - @rkirov could you please check if the exports and their layouts are what you need for your work?

As requested by @IgorMinar we don't leave bundle entry point file in our modules/angular2 tree so people won't be able to import from it etc.

@IgorMinar this PR differs slightly from the proposal described in #5665 as it doesn't implement:

reexport via the ng namespace the most commonly needed stuff used for defining components, directives, pipes and services via chained DSL. This should be just a handful of symbols and we should document that they are being reexported in ES5 bundles for convenience.

It might not be necessary as chaining looks good IMO, here is example of a simple app in ES5 with a bundle from this PR:

 var MyComponent = ng.core.Component({
      selector: 'my-app',
      template: '' +
      '<h1>Hello, {{name}}!</h1>' +
      ' Say hello to: <input [value]="name" (input)="name = $event.target.value">' +
      ''
    }).Class({constructor: function() {
      this.name = 'World';
    }});

    ng.platform.browser.bootstrap(MyComponent);

@vsavkin I can see that as part of 4ea5b6e you've added angular2/router/router_link_dsl.js to UMD bundles. I'm not sure why it is needed so didn't include it in this PR.

It is also related to #5700 where we've got some exports in angular2/angular2 that are "suspicious" and didn't make it to bundles from this PR.

Could you help me with those 2 items?

angular2/router/router_link_dsl.js

It has to be included

the rest looks good. can you send a follow up PR that adds markdown file into the repo with a table describing what is contained in what bundle.

thanks!

we'll later move the table to the dev guide but let's just publish it under https://github.com/angular/angular/tree/master/modules/angular2/docs for now

+1 to not reexporting Component and friends for now. let's see if ng.core.Component is sufficient.

Lgtm

Merging PR #5697 on behalf of @jelbourn to branch presubmit-jelbourn-pr-5697.

Merging PR #5697 on behalf of @jelbourn to branch presubmit-jelbourn-pr-5697.

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

_{This action has been performed automatically by a bot.}