Skip to content

refactor(@angular-devkit/build-angular): remove unneeded option type casting from several builders#27252

Merged
clydin merged 2 commits intoangular:mainfrom
clydin:application/cleanup-2
Mar 12, 2024
Merged

refactor(@angular-devkit/build-angular): remove unneeded option type casting from several builders#27252
clydin merged 2 commits intoangular:mainfrom
clydin:application/cleanup-2

Conversation

@clydin
Copy link
Copy Markdown
Member

@clydin clydin commented Mar 11, 2024

Multiple cases where builder options were being cast to the JsonObject type have been removed. These casts are no longer needed and unnecessarily added complexity to the code.

…casting from several builders

Multiple cases where builder options were being cast to the `JsonObject` type
have been removed. These casts are no longer needed and unnecessarily added
complexity to the code.
@clydin clydin added the target: minor This PR is targeted for the next minor release label Mar 11, 2024
@clydin clydin requested a review from alan-agius4 March 11, 2024 17:35
@clydin clydin added the action: review The PR is still awaiting reviews from at least one requested reviewer label Mar 11, 2024
@alan-agius4 alan-agius4 added action: merge The PR is ready for merge by the caretaker and removed action: review The PR is still awaiting reviews from at least one requested reviewer labels Mar 12, 2024
@clydin clydin merged commit 01fa047 into angular:main Mar 12, 2024
@clydin clydin deleted the application/cleanup-2 branch March 12, 2024 13:46
@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 12, 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: 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