chore(bundle): determine if compiler barrel can be omitted #5815

Activity

1. jeffbcross

   added

   area: build & ciRelated the build and CI infrastructure of the projectRelated the build and CI infrastructure of the project

   type: chore

   P1: urgent

   on Dec 10, 2015
2. jeffbcross

   self-assigned this

   on Dec 10, 2015
3. jeffbcross

   added this to the beta.0 milestone on Dec 10, 2015
4. jeffbcross commented on Dec 10, 2015

   

   jeffbcross

   on Dec 10, 2015

   ContributorAuthor

   More actions

   CC @tbosch @IgorMinar @vsavkin, save me some time if you know that one of these must be included in our public API.
5. jeffbcross commented on Dec 10, 2015

   

   jeffbcross

   on Dec 10, 2015

   ContributorAuthor

   More actions

   I should also check https://github.com/angular/universal
6. jeffbcross commented on Dec 10, 2015

   

   jeffbcross

   on Dec 10, 2015

   ContributorAuthor

   More actions

   URLResolver is used in these places outside of tests and compiler src:

   • A couple of dart transformers
   • button_spec.ts and other material tests to provide a JS-&-Dart-friendly resolver for internal tests
   • worker.ts where it's just re-exported
   • playground/src/material/button/index.ts and other material examples where DemoUrlResolver is used instead.
   • Not present in angular/universal
   • There are a bunch of places internally that are overriding URLResolver for non-test code.

   Perhaps URLResolver can be moved to angular2/core?
7. jeffbcross commented on Dec 10, 2015

   

   jeffbcross

   on Dec 10, 2015

   ContributorAuthor

   More actions

   AppRootUrl is used in these places outside of tests and compiler src:

   • worker_app_common.ts and worker_render_common
   • worker.ts where it's just re-exported
   • Not present in angular/universal
   • Not present internally
8. jeffbcross commented on Dec 10, 2015

   

   jeffbcross

   on Dec 10, 2015

   ContributorAuthor

   More actions

   I just talked IRL with @tbosch about all of these dependencies, and they could all be removed from the JS bundle. AppRootURL can be deleted, which I'll also do as part of this issue.
9. pkozlowski-opensource

   closed this as completedin 20c6eebon Dec 11, 2015
10. tbosch

    added a commit that references this issue on Dec 11, 2015

    fix(angular2): remove `angular2.ts` module

    ...

    This issue will close once commit 200dc00 is merged into the 'main' branch.

    200dc00
11. jeffbcross

    added 5 commits that reference this issue on Dec 12, 2015

    fix(compiler): remove AppRootUrl

    ...

    736871e

    fix(web_workers): remove unnecessary setup module and AppRootUrl

    ...

    d367a8a

    fix(compiler): remove AppRootUrl

    ...

    ed2c25e

    fix(web_workers): remove unnecessary setup module and AppRootUrl

    ...

    a885f37
12. angular-automatic-lock-bot commented on Sep 7, 2019

    

    angular-automatic-lock-botBot

    on Sep 7, 2019

    More actions

    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.}
13. angular-automatic-lock-bot

    locked and limited conversation to collaborators on Sep 7, 2019