The Wayback Machine - https://web.archive.org/web/20260517041320/https://github.com/angular/angular-cli/pull/14907
Skip to content

test: add initial ivy large test support#14907

Merged
vikerman merged 1 commit into
angular:masterfrom
clydin:ivy-test-large
Jul 2, 2019
Merged

test: add initial ivy large test support#14907
vikerman merged 1 commit into
angular:masterfrom
clydin:ivy-test-large

Conversation

@clydin
Copy link
Copy Markdown
Member

@clydin clydin commented Jun 26, 2019

No description provided.

@clydin clydin force-pushed the ivy-test-large branch 3 times, most recently from 060cafd to 883e92b Compare June 27, 2019 00:47
@clydin clydin added the target: major This PR is targeted for the next major release label Jun 27, 2019
Comment thread .circleci/config.yml Outdated
@clydin clydin force-pushed the ivy-test-large branch 9 times, most recently from 535e06f to 09d0271 Compare June 28, 2019 18:20
@clydin clydin requested a review from filipesilva June 28, 2019 20:17
@clydin clydin marked this pull request as ready for review June 28, 2019 20:18
@clydin clydin force-pushed the ivy-test-large branch 3 times, most recently from 78e0806 to 85144a3 Compare June 29, 2019 00:32
Copy link
Copy Markdown
Contributor

@filipesilva filipesilva left a comment

Choose a reason for hiding this comment

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

LGTM aside from the AOT rebuild.

I think we should have tracking issues for each DISABLED_FOR_IVY test though, so that in the future when we look at these tests we have a clear link to know if it should be working now. Could you add a comment with links on each of those please?

Comment thread packages/angular_devkit/build_angular/test/browser/rebuild_spec_large.ts Outdated
Copy link
Copy Markdown
Contributor

@filipesilva filipesilva left a comment

Choose a reason for hiding this comment

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

LGTM, we need to follow up on the known Ivy failures.

@vikerman vikerman merged commit 99fa7bd into angular:master Jul 2, 2019
@clydin clydin deleted the ivy-test-large branch July 2, 2019 17:05
@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 13, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

target: major This PR is targeted for the next major release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants

HTTPS · web.archive.org
← Home