Skip to content

Conversation

@crisbeto
Copy link
Member

Reduces the runtime of the table tests to about 1/3 of the one from master by not recompiling all the individual test cases for each test.

Reduces the runtime of the table tests to about 1/3 of the one from master by not recompiling all the individual test cases for each test.
@crisbeto crisbeto requested a review from andrewseguin as a code owner March 21, 2018 21:15
@googlebot googlebot added the cla: yes PR author has agreed to Google's Contributor License Agreement label Mar 21, 2018
@andrewseguin
Copy link
Contributor

Awesome - thanks for identifying this slowness and making a fix.

@andrewseguin andrewseguin added pr: lgtm action: merge The PR is ready for merge by the caretaker and removed pr: needs review labels Mar 21, 2018
@andrewseguin andrewseguin merged commit cc24766 into angular:master Mar 22, 2018
crisbeto added a commit to crisbeto/material2 that referenced this pull request Mar 22, 2018
Along the same lines as angular#10517. Refactors the menu unit tests to make them faster and (hopefully) more reliable.
@angular-automatic-lock-bot
Copy link

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 8, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

action: merge The PR is ready for merge by the caretaker cla: yes PR author has agreed to Google's Contributor License Agreement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants