Skip to content

Conversation

alan-agius4
Copy link
Collaborator

With this change we re-emit assets referenced in component stylesheets which where uneffected by the change that re-triggered a re-compilation.

Since we cache the the result of processed component CSS, during a re-compilation postcss-cli-resources plugin will not run which causes assets to be to emit. With this change we now cache the asset and re-emit them on every change.

Closes #20882

@google-cla google-cla bot added the cla: yes label May 21, 2021
@alan-agius4 alan-agius4 requested a review from clydin May 21, 2021 11:40
@alan-agius4 alan-agius4 added the target: patch This PR is targeted for the next patch release label May 21, 2021
@alan-agius4 alan-agius4 added the action: merge The PR is ready for merge by the caretaker label May 21, 2021
With this change we re-emit assets referenced in component stylesheets which where uneffected by the change that re-triggered a re-compilation.

Since we cache the the result of processed component CSS, during a re-compilation `postcss-cli-resources` plugin will not run which causes assets to be to emit. With this change we now cache the asset and re-emit them on every change.

Closes #20882
@alan-agius4 alan-agius4 reopened this May 21, 2021
@clydin clydin merged commit 5855374 into angular:master May 22, 2021
@alan-agius4 alan-agius4 deleted the asset-cache branch May 22, 2021 04:48
@angular-automatic-lock-bot
Copy link

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 Jun 22, 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 target: patch This PR is targeted for the next patch release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Angular 12 : Loading svg using url(../../assets/images/app_logo.svg) is not working in Scss
2 participants