Skip to content

V8/static queries patch - #30641

Closed
alxhub wants to merge 5 commits into
angular:8.0.xfrom
alxhub:v8/static-queries-patch
Closed

alxhub wants to merge 5 commits into
angular:8.0.xfrom
alxhub:v8/static-queries-patch

Conversation

@alxhub

@alxhub alxhub commented May 23, 2019

Copy link
Copy Markdown
Member

Patch version of #30639, including Kara's cherry-picked fixes.

@alxhub
alxhub requested a review from IgorMinar as a code owner May 23, 2019 19:53
@alxhub
alxhub requested review from a team May 23, 2019 19:53
@alxhub alxhub added hotlist: release-blocker action: merge The PR is ready for merge by the caretaker labels May 23, 2019

@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

@kara kara added the merge: caretaker note Alert the caretaker performing the merge to check the PR for an out of normal action needed or note label May 23, 2019
@kara

kara commented May 23, 2019

Copy link
Copy Markdown
Contributor

merge-assistance: global approval

@alxhub
alxhub force-pushed the v8/static-queries-patch branch from 712d158 to fbf247d Compare May 23, 2019 20:37
@kara kara added the action: cleanup The PR is in need of cleanup, either due to needing a rebase or in response to comments from reviews label May 23, 2019
@alxhub
alxhub force-pushed the v8/static-queries-patch branch from fbf247d to f9fe21c Compare May 23, 2019 20:44
@alxhub alxhub removed the action: cleanup The PR is in need of cleanup, either due to needing a rebase or in response to comments from reviews label May 23, 2019
@alxhub
alxhub force-pushed the v8/static-queries-patch branch from f9fe21c to db618de Compare May 23, 2019 21:22
@alxhub
alxhub requested a review from a team May 23, 2019 21:22
This commit makes the static flag on @ViewChild and @ContentChild required.

BREAKING CHANGE:

In Angular version 8, it's required that all @ViewChild and @ContentChild
queries have a 'static' flag specifying whether the query is 'static' or
'dynamic'. The compiler previously sorted queries automatically, but in
8.0 developers are required to explicitly specify which behavior is wanted.
This is a temporary requirement as part of a migration; see
https://angular.io/guide/static-query-migration for more details.

@ViewChildren and @ContentChildren queries are always dynamic, and so are
unaffected.
@alxhub
alxhub force-pushed the v8/static-queries-patch branch from db618de to 8230ad7 Compare May 23, 2019 21:41
@alxhub
alxhub requested a review from a team May 23, 2019 21:41

@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

@mary-poppins

Copy link
Copy Markdown

You can preview 712d158 at https://pr30641-712d158.ngbuilds.io/.
You can preview fbf247d at https://pr30641-fbf247d.ngbuilds.io/.
You can preview f9fe21c at https://pr30641-f9fe21c.ngbuilds.io/.
You can preview db618de at https://pr30641-db618de.ngbuilds.io/.
You can preview 8230ad7 at https://pr30641-8230ad7.ngbuilds.io/.

jasonaden pushed a commit that referenced this pull request May 23, 2019
jasonaden pushed a commit that referenced this pull request May 23, 2019
jasonaden pushed a commit that referenced this pull request May 23, 2019
jasonaden pushed a commit that referenced this pull request May 23, 2019
This commit makes the static flag on @ViewChild and @ContentChild required.

BREAKING CHANGE:

In Angular version 8, it's required that all @ViewChild and @ContentChild
queries have a 'static' flag specifying whether the query is 'static' or
'dynamic'. The compiler previously sorted queries automatically, but in
8.0 developers are required to explicitly specify which behavior is wanted.
This is a temporary requirement as part of a migration; see
https://angular.io/guide/static-query-migration for more details.

@ViewChildren and @ContentChildren queries are always dynamic, and so are
unaffected.

PR Close #30641
@alxhub alxhub closed this May 23, 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 merge: caretaker note Alert the caretaker performing the merge to check the PR for an out of normal action needed or note

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants