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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

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