Skip to content

refactor(ivy): delete the old global-analysis based ivy compiler - #27306

Closed
alxhub wants to merge 1 commit into
angular:masterfrom
alxhub:ngtsc-remove-global-ivy-compiler
Closed

alxhub wants to merge 1 commit into
angular:masterfrom
alxhub:ngtsc-remove-global-ivy-compiler

Conversation

@alxhub

@alxhub alxhub commented Nov 27, 2018

Copy link
Copy Markdown
Member

This commit deletes the old Ivy compiler (enableIvy: true) which was based
on ngc's global analysis of the application and all libraries. This commit
also switches the behavior of enableIvy to use 'true' to enable ngtsc.

@alxhub
alxhub requested a review from mhevery November 27, 2018 22:43
@mary-poppins

Copy link
Copy Markdown

You can preview 4eaf69f at https://pr27306-4eaf69f.ngbuilds.io/.

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.

Remove this comment

This commit deletes the old Ivy compiler (enableIvy: true) which was based
on ngc's global analysis of the application and all libraries. This commit
also switches the behavior of enableIvy to use 'true' to enable ngtsc.
@alxhub
alxhub force-pushed the ngtsc-remove-global-ivy-compiler branch from 4eaf69f to 0eece0e Compare November 28, 2018 20:02
@alxhub alxhub added the area: core Issues related to the framework runtime label Dec 11, 2018
@ngbot ngbot Bot added this to the needsTriage milestone Dec 11, 2018
@alxhub alxhub closed this Dec 14, 2018
@angular-automatic-lock-bot

Copy link
Copy Markdown

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.

@angular-automatic-lock-bot angular-automatic-lock-bot Bot locked and limited conversation to collaborators Sep 14, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

area: core Issues related to the framework runtime cla: yes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants