Skip to content

refactor(@angular/cli): use command default for unspecified target builders#31558

Merged
alan-agius4 merged 1 commit intoangular:mainfrom
clydin:cli/default-target-builder
Oct 23, 2025
Merged

refactor(@angular/cli): use command default for unspecified target builders#31558
alan-agius4 merged 1 commit intoangular:mainfrom
clydin:cli/default-target-builder

Conversation

@clydin
Copy link
Member

@clydin clydin commented Oct 22, 2025

If a target definition from a project's configuration does not contain a builder field and the CLI command provides a default, the default will now be used for the command execution. This is in addition to the existing case where the command will use a synthetic target definition for commands that provide default builders.

…ilders

If a target definition from a project's configuration does not contain a
`builder` field and the CLI command provides a default, the default will
now be used for the command execution. This is in addition to the existing
case where the command will use a synthetic target definition for commands
that provide default builders.
@clydin clydin added the target: major This PR is targeted for the next major release label Oct 22, 2025
@clydin clydin requested a review from alan-agius4 October 22, 2025 14:34
@clydin clydin added the action: review The PR is still awaiting reviews from at least one requested reviewer label Oct 22, 2025
Copy link
Contributor

@hybrist hybrist left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is there a test case worth adding for this?

@alan-agius4 alan-agius4 added target: rc This PR is targeted for the next release-candidate action: merge The PR is ready for merge by the caretaker and removed target: major This PR is targeted for the next major release action: review The PR is still awaiting reviews from at least one requested reviewer labels Oct 23, 2025
@alan-agius4 alan-agius4 merged commit 744ae4e into angular:main Oct 23, 2025
36 checks passed
@alan-agius4
Copy link
Collaborator

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

@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 Nov 23, 2025
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/cli target: rc This PR is targeted for the next release-candidate

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants