Skip to content

Conversation

alan-agius4
Copy link
Collaborator

The configuration now ensures that when a package is listed for exclusion, all paths within that package including sub-paths like @foo/bar/baz are marked as external and not prebundled by the development server.

For example, specifying @foo/bar in the exclude list will prevent the development server from bundling any files from the @foo/bar package, including its sub-paths such as @foo/bar/baz.

This aligns with esbuild external option behaviour https://esbuild.github.io/api/#external

Closes #29170

@alan-agius4 alan-agius4 force-pushed the external-multiple-entry branch 3 times, most recently from 94496f2 to 5d04b0c Compare March 4, 2025 13:22
@alan-agius4 alan-agius4 marked this pull request as draft March 4, 2025 14:08
@alan-agius4 alan-agius4 force-pushed the external-multiple-entry branch from 5d04b0c to 359686f Compare March 5, 2025 12:25
@alan-agius4 alan-agius4 requested a review from clydin March 5, 2025 12:51
@alan-agius4 alan-agius4 added the action: review The PR is still awaiting reviews from at least one requested reviewer label Mar 5, 2025
@alan-agius4 alan-agius4 marked this pull request as ready for review March 5, 2025 12:51
@alan-agius4 alan-agius4 force-pushed the external-multiple-entry branch from 359686f to e4c4195 Compare March 5, 2025 15:37
…dling

The configuration now ensures that when a package is listed for exclusion, all paths within that package including sub-paths like `@foo/bar/baz`  are marked as external and not prebundled by the development server.

For example, specifying `@foo/bar` in the exclude list will prevent the development server from bundling any files from the `@foo/bar` package, including its sub-paths such as `@foo/bar/baz`.

This aligns with esbuild external option behaviour https://esbuild.github.io/api/#external

Closes angular#29170
@alan-agius4 alan-agius4 added target: patch This PR is targeted for the next patch release 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 6, 2025
@alan-agius4 alan-agius4 force-pushed the external-multiple-entry branch from e4c4195 to 5daa0e7 Compare March 6, 2025 17:54
@alan-agius4 alan-agius4 merged commit f0dd60b into angular:main Mar 6, 2025
30 checks passed
@alan-agius4 alan-agius4 deleted the external-multiple-entry branch March 6, 2025 18:12
@alan-agius4
Copy link
Collaborator Author

The changes were merged into the following branches: main, 19.2.x

@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 Apr 6, 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/build target: patch This PR is targeted for the next patch release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Exclude all entrypoints of a library from prebundling
2 participants