Skip to content

perf(@angular-devkit/build-angular): avoid transforming empty component stylesheets#27293

Merged
alan-agius4 merged 1 commit intoangular:mainfrom
clydin:application/avoid-empty-component-styles
Mar 15, 2024
Merged

perf(@angular-devkit/build-angular): avoid transforming empty component stylesheets#27293
alan-agius4 merged 1 commit intoangular:mainfrom
clydin:application/avoid-empty-component-styles

Conversation

@clydin
Copy link
Copy Markdown
Member

@clydin clydin commented Mar 14, 2024

A stylesheet for a component that is empty or contains only whitespace will no longer be transformed and bundled during a build. Transforming the stylesheet was unnecessary work as an empty string would be returned as a result. While this may not be common in applications, it is a possibility. Both file and inline component stylesheets are affected by this change.

…nt stylesheets

A stylesheet for a component that is empty or contains only whitespace will
no longer be transformed and bundled during a build. Transforming the stylesheet
was unnecessary work as an empty string would be returned as a result. While
this may not be common in applications, it is a possibility. Both file and
inline component stylesheets are affected by this change.
@clydin clydin added target: patch This PR is targeted for the next patch release action: merge The PR is ready for merge by the caretaker labels Mar 14, 2024
@alan-agius4 alan-agius4 merged commit 99104cd into angular:main Mar 15, 2024
@clydin clydin deleted the application/avoid-empty-component-styles branch March 15, 2024 11:33
@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 Apr 15, 2024
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.

2 participants