chore(testapp): Use newer angular beta #3121

sjelin · 2016-04-12T11:15:59Z

Need this for #3024

cnishina · 2016-04-13T17:48:42Z

This looks fine except this definitely makes the flaky test #3000 fail 100% of the time. This could might need some attention.

cnishina · 2016-04-13T20:49:55Z

Should probably get #3127 merged in first to pass this test. Other than that, LGTM

sjelin · 2016-04-15T13:56:07Z

Blocked on angular/angular#8089. Assigning Julie since that bug is much more her domain.

juliemr · 2016-04-25T20:16:35Z

We decided this is not blocked, since we should be running setTimeout outside of the Angular zone.

sjelin · 2016-04-26T00:22:39Z

setInterval is now being run outside of NgZone

sjelin · 2016-04-27T18:44:16Z

Craig assigning back to you since you're on call

sjelin assigned cnishina Apr 12, 2016

sjelin added the pr: needs review label Apr 12, 2016

googlebot added the cla: yes label Apr 12, 2016

cnishina added pr: LGTM and removed pr: needs review labels Apr 13, 2016

sjelin added the status: blocked label Apr 15, 2016

sjelin assigned juliemr and unassigned cnishina Apr 15, 2016

sjelin mentioned this pull request Apr 15, 2016

chore(tests): test for ngUpgrade support #3024

Merged

cnishina removed the pr: LGTM label Apr 15, 2016

juliemr removed the status: blocked label Apr 25, 2016

chore(testapp): Use newer angular beta

85b0c4e

sjelin force-pushed the testapp-ng branch from 25b462c to 85b0c4e Compare April 26, 2016 00:21

sjelin added the pr: needs review label Apr 26, 2016

sjelin mentioned this pull request Apr 26, 2016

enhancement(ng2 setInterval): Better error messages for using setInterval inside NgZone #3168

Open

sjelin assigned cnishina and unassigned juliemr Apr 26, 2016

cnishina added pr: LGTM and removed pr: needs review labels Apr 28, 2016

sjelin merged commit 3083097 into angular:master Apr 28, 2016

Provide feedback