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

build: enable Ivy for 8.x #647

Merged
merged 1 commit into from
Oct 10, 2019
Merged

build: enable Ivy for 8.x #647

merged 1 commit into from
Oct 10, 2019

Conversation

Splaktar
Copy link
Contributor

@Splaktar Splaktar commented Oct 10, 2019

PR Checklist

Please check that your PR fulfills the following requirements:

  • The commit message follows our guidelines
  • Tests for the changes have been added or weren't appropriate

What is the current behavior?

Ivy isn't enabled for 8.x.

Issue Number: N/A

What is the new behavior?

Ivy is enabled for 8.x.

Other information

Current material.angular.io
zipped production bundle - 853 KB
unzipped bundle resources - 4.3 MB

With Ivy enabled
zipped production bundle - 645 KB ✅
unzipped bundle resources - 2.4 MB ✅

Made possible by getting angular/components#17282 and angular/components#17228 merged back to 8.2.3.

@Splaktar Splaktar self-assigned this Oct 10, 2019
@Splaktar Splaktar requested review from jelbourn and mmalerba October 10, 2019 21:51
@Splaktar Splaktar changed the title build: enable Ivy build: enable Ivy for 8.x Oct 10, 2019
Copy link
Member

@jelbourn jelbourn left a comment

Choose a reason for hiding this comment

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

LGTM

@jelbourn jelbourn merged commit ac284f8 into 8.x Oct 10, 2019
@Splaktar Splaktar deleted the enable-ivy branch October 10, 2019 22:12
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants