Skip to content

fix(compiler-cli): do not error with prepocessing if component has no inline styles#41602

Closed
clydin wants to merge 1 commit intoangular:masterfrom
clydin:transform-resources
Closed

fix(compiler-cli): do not error with prepocessing if component has no inline styles#41602
clydin wants to merge 1 commit intoangular:masterfrom
clydin:transform-resources

Conversation

@clydin
Copy link
Copy Markdown
Member

@clydin clydin commented Apr 13, 2021

The asynchronous preprocessing check was not accounting for components that did not have any inline styles. In that case, the cache did not have an entry which then allowed the asynchronous check to run and fail the compilation. The caching during the asynchronous analysis phase now handles components without inline styles.

@clydin clydin added type: bug/fix action: review The PR is still awaiting reviews from at least one requested reviewer target: major This PR is targeted for the next major release labels Apr 13, 2021
@google-cla google-cla bot added the cla: yes label Apr 13, 2021
@pullapprove pullapprove bot requested a review from alxhub April 13, 2021 18:07
@clydin clydin added the area: compiler Issues related to `ngc`, Angular's template compiler label Apr 13, 2021
@ngbot ngbot bot added this to the Backlog milestone Apr 13, 2021
… inline styles

The asynchronous preprocessing check was not accounting for components that did not have any inline styles. In that case, the cache did not have an entry which then allowed the asynchronous check to run and fail the compilation. The caching during the asynchronous analysis phase now handles components without inline styles.
@clydin clydin force-pushed the transform-resources branch from 8f22f0c to fa4356c Compare April 13, 2021 18:14
@clydin clydin removed the action: review The PR is still awaiting reviews from at least one requested reviewer label Apr 13, 2021
@clydin clydin removed the request for review from alxhub April 13, 2021 21:28
@clydin clydin added the action: merge The PR is ready for merge by the caretaker label Apr 13, 2021
@AndrewKushnir AndrewKushnir added the action: presubmit The PR is in need of a google3 presubmit label Apr 13, 2021
@AndrewKushnir
Copy link
Copy Markdown
Contributor

Presubmit.

@AndrewKushnir AndrewKushnir removed the action: presubmit The PR is in need of a google3 presubmit label Apr 14, 2021
@clydin clydin added the target: rc This PR is targeted for the next release-candidate label Apr 14, 2021
@zarend zarend removed the target: major This PR is targeted for the next major release label Apr 14, 2021
zarend pushed a commit that referenced this pull request Apr 14, 2021
… inline styles (#41602)

The asynchronous preprocessing check was not accounting for components that did not have any inline styles. In that case, the cache did not have an entry which then allowed the asynchronous check to run and fail the compilation. The caching during the asynchronous analysis phase now handles components without inline styles.

PR Close #41602
@zarend zarend closed this in 1b43158 Apr 14, 2021
@clydin clydin deleted the transform-resources branch April 14, 2021 22:51
@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 May 15, 2021
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: compiler Issues related to `ngc`, Angular's template compiler cla: yes target: rc This PR is targeted for the next release-candidate type: bug/fix

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants