Skip to content

fix(compiler-cli): fix and re-enble expression lowering - #18570

Merged
vicb merged 1 commit into
angular:masterfrom
chuckjaz:fix-expression-lowering
Aug 8, 2017
Merged

vicb merged 1 commit into
angular:masterfrom
chuckjaz:fix-expression-lowering

Conversation

@chuckjaz

@chuckjaz chuckjaz commented Aug 7, 2017

Copy link
Copy Markdown
Contributor

Fixes issue uncovered by #18388 and re-enables expression
lowering disabled by #18513.

PR Type

What kind of change does this PR introduce?

[x] Bugfix

What is the current behavior?

Expression lowering was disabled because of a bug uncovered by #18388.

What is the new behavior?

Expression lowering is re-enabled and the issue uncovered by #18388 has been resolved.

Does this PR introduce a breaking change?

[ ] Yes
[x] No

@mary-poppins

Copy link
Copy Markdown

You can preview 3ad3963 at https://pr18570-3ad3963.ngbuilds.io/.

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.

typo visit

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.

"+ tests" -> re-enable the tests

@vicb vicb 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.

No additional spec ?

Fixes issue uncovered by angular#18388 and re-enables expression
lowering disabled by angular#18513.
@chuckjaz
chuckjaz force-pushed the fix-expression-lowering branch from 3ad3963 to 3bd00f9 Compare August 8, 2017 17:22
@mary-poppins

Copy link
Copy Markdown

You can preview 3bd00f9 at https://pr18570-3bd00f9.ngbuilds.io/.

@vicb vicb added area: core Issues related to the framework runtime action: merge The PR is ready for merge by the caretaker target: major This PR is targeted for the next major release labels Aug 8, 2017
@vicb
vicb merged commit 6f2038c into angular:master Aug 8, 2017
asnowwolf pushed a commit to asnowwolf/angular that referenced this pull request Aug 11, 2017
Fixes issue uncovered by angular#18388 and re-enables expression
lowering disabled by angular#18513.
juleskremer pushed a commit to juleskremer/angular that referenced this pull request Aug 26, 2017
Fixes issue uncovered by angular#18388 and re-enables expression
lowering disabled by angular#18513.
juleskremer pushed a commit to juleskremer/angular that referenced this pull request Aug 28, 2017
Fixes issue uncovered by angular#18388 and re-enables expression
lowering disabled by angular#18513.
@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 12, 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 area: core Issues related to the framework runtime cla: yes target: major This PR is targeted for the next major release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants