Skip to content

Conversation

@clydin
Copy link
Member

@clydin clydin commented Dec 15, 2025

This commit refactors the usage of the Spinner class in @angular-devkit/build-angular builders by removing explicit colors.redBright calls when invoking spinner.fail().

The Spinner.fail() method already applies red bright coloring internally. This change centralizes the styling logic within the Spinner class and prevents redundant coloring or unnecessary dependency usage at the call site.

…oring

This commit refactors the usage of the `Spinner` class in `@angular-devkit/build-angular` builders by removing explicit `colors.redBright` calls when invoking `spinner.fail()`.

The `Spinner.fail()` method already applies red bright coloring internally. This change centralizes the styling logic within the `Spinner` class and prevents redundant coloring or unnecessary dependency usage at the call site.
@clydin clydin added the target: minor This PR is targeted for the next minor release label Dec 15, 2025
@clydin clydin added the action: merge The PR is ready for merge by the caretaker label Dec 15, 2025
@clydin clydin merged commit 1b12967 into angular:main Dec 15, 2025
37 checks passed
@clydin
Copy link
Member Author

clydin commented Dec 15, 2025

This PR was merged into the repository. The changes were merged into the following branches:

@clydin clydin deleted the ng-build/remove-extra-color branch December 15, 2025 23:18
@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 Jan 15, 2026
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: @angular-devkit/build-angular target: minor This PR is targeted for the next minor release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants