Skip to content

refactor(ivy): back to frog eyes ɵɵ - #30547

Closed
benlesh wants to merge 2 commits into
angular:8.0.xfrom
benlesh:FW-1337_patch_frog_eyes
Closed

benlesh wants to merge 2 commits into
angular:8.0.xfrom
benlesh:FW-1337_patch_frog_eyes

Conversation

@benlesh

@benlesh benlesh commented May 17, 2019

Copy link
Copy Markdown
Contributor

Patch version of #30546

@benlesh benlesh added state: WIP refactoring Issue that involves refactoring or code-cleanup PR target: patch-only risk: medium labels May 17, 2019
@benlesh
benlesh requested a review from IgorMinar as a code owner May 17, 2019 23:23
@benlesh
benlesh requested review from a team May 17, 2019 23:23
@ngbot ngbot Bot added this to the needsTriage milestone May 17, 2019
@benlesh
benlesh force-pushed the FW-1337_patch_frog_eyes branch from 2c29942 to 84977db Compare May 18, 2019 00:08

@kara kara left a comment

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.

LGTM

@benlesh benlesh added action: merge The PR is ready for merge by the caretaker and removed state: WIP labels May 18, 2019
@benlesh
benlesh force-pushed the FW-1337_patch_frog_eyes branch 3 times, most recently from fc7da37 to b16253c Compare May 21, 2019 16:33
@kara kara modified the milestones: needsTriage, version 8 May 21, 2019
@benlesh

benlesh commented May 21, 2019

Copy link
Copy Markdown
Contributor Author

This is blocked on #30593

Which is required to get the examples tests passing CI.

@kara

kara commented May 21, 2019

Copy link
Copy Markdown
Contributor

@benlesh I think that PR fixes a different CI issue

@benlesh

benlesh commented May 21, 2019

Copy link
Copy Markdown
Contributor Author

@kara Sorry, I should have added more context. I was working with @brandonroberts to figure out why I was having CI failures in aio examples on this PR, and he uncovered that it was linked to the other PR.

@benlesh
benlesh force-pushed the FW-1337_patch_frog_eyes branch from 70f5feb to 6a833ef Compare May 21, 2019 21:12
There appears to be a race condition with the package.json files during test runs
@jasonaden

Copy link
Copy Markdown
Contributor

Presubmit

jasonaden pushed a commit that referenced this pull request May 22, 2019
jasonaden pushed a commit that referenced this pull request May 22, 2019
There appears to be a race condition with the package.json files during test runs

PR Close #30547
@jasonaden

Copy link
Copy Markdown
Contributor

Merged

@jasonaden jasonaden closed this May 22, 2019
@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 15, 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 refactoring Issue that involves refactoring or code-cleanup risk: medium

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants