Skip to content

refactor(@angular/build): remove unused target parameter from getFeatureSupport#33192

Merged
alan-agius4 merged 1 commit into
angular:mainfrom
alan-agius4:feature-supported
May 13, 2026
Merged

refactor(@angular/build): remove unused target parameter from getFeatureSupport#33192
alan-agius4 merged 1 commit into
angular:mainfrom
alan-agius4:feature-supported

Conversation

@alan-agius4
Copy link
Copy Markdown
Collaborator

The target parameter in the getFeatureSupport function is no longer used internally, so it has been removed to simplify the function signature and call sites.

@alan-agius4 alan-agius4 requested a review from clydin May 13, 2026 13:00
@alan-agius4 alan-agius4 added action: review The PR is still awaiting reviews from at least one requested reviewer target: rc This PR is targeted for the next release-candidate labels May 13, 2026
Copy link
Copy Markdown

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request removes the unused target parameter from the getFeatureSupport utility function and updates all call sites within the esbuild and Vite build tools. A review comment points out that the JSDoc description for getFeatureSupport still incorrectly states that the function generates a map based on a list of targets, which should be corrected to match the new implementation.

Comment thread packages/angular/build/src/tools/esbuild/utils.ts
…ureSupport

The `target` parameter in the `getFeatureSupport` function is no longer used internally, so it has been removed to simplify the function signature and call sites.
@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 May 13, 2026
@alan-agius4 alan-agius4 merged commit c14eeca into angular:main May 13, 2026
34 of 35 checks passed
@alan-agius4 alan-agius4 deleted the feature-supported branch May 13, 2026 14:47
@alan-agius4
Copy link
Copy Markdown
Collaborator Author

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

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/build target: rc This PR is targeted for the next release-candidate

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants