Skip to content

refactor(core): tree-shake PROPAGATION_STOPPED_SYMBOL#61004

Closed
arturovt wants to merge 1 commit intoangular:mainfrom
arturovt:refactor/core-tree-shake-propagationStopped
Closed

refactor(core): tree-shake PROPAGATION_STOPPED_SYMBOL#61004
arturovt wants to merge 1 commit intoangular:mainfrom
arturovt:refactor/core-tree-shake-propagationStopped

Conversation

@arturovt
Copy link
Copy Markdown
Contributor

Adds __PURE__ annotations to PROPAGATION_STOPPED_SYMBOL to enable tree-shaking, even if is is not referenced. This variable is not dropped when Angular is imported from a module that has sideEffects set to true.

image

Adds `__PURE__` annotations to `PROPAGATION_STOPPED_SYMBOL` to enable tree-shaking, even if is is not referenced. This variable is not dropped when Angular is imported from a module that has `sideEffects` set to `true`.
@pullapprove pullapprove bot requested review from alxhub and csmick April 25, 2025 22:59
@pullapprove pullapprove bot added the requires: TGP This PR requires a passing TGP before merging is allowed label Apr 25, 2025
@angular-robot angular-robot bot added the area: core Issues related to the framework runtime label Apr 25, 2025
@ngbot ngbot bot added this to the Backlog milestone Apr 25, 2025
Copy link
Copy Markdown
Member

@pkozlowski-opensource pkozlowski-opensource left a comment

Choose a reason for hiding this comment

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

LGTM

Reviewed-for: primitives

@pkozlowski-opensource pkozlowski-opensource removed the request for review from alxhub April 30, 2025 14:59
@pkozlowski-opensource pkozlowski-opensource added the target: patch This PR is targeted for the next patch release label Apr 30, 2025
@thePunderWoman thePunderWoman requested review from iteriani and removed request for csmick April 30, 2025 14:59
@iteriani
Copy link
Copy Markdown
Contributor

LGTM

Reviewed-for: primitives-shared

@pkozlowski-opensource pkozlowski-opensource added the action: merge The PR is ready for merge by the caretaker label Apr 30, 2025
AndrewKushnir pushed a commit that referenced this pull request Apr 30, 2025
Adds `__PURE__` annotations to `PROPAGATION_STOPPED_SYMBOL` to enable tree-shaking, even if is is not referenced. This variable is not dropped when Angular is imported from a module that has `sideEffects` set to `true`.

PR Close #61004
@AndrewKushnir
Copy link
Copy Markdown
Contributor

This PR was merged into the repository by commit 13fd1b3.

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

AndrewKushnir pushed a commit that referenced this pull request Apr 30, 2025
Adds `__PURE__` annotations to `PROPAGATION_STOPPED_SYMBOL` to enable tree-shaking, even if is is not referenced. This variable is not dropped when Angular is imported from a module that has `sideEffects` set to `true`.

PR Close #61004
@arturovt arturovt deleted the refactor/core-tree-shake-propagationStopped branch April 30, 2025 20:02
@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 May 31, 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: core Issues related to the framework runtime requires: TGP This PR requires a passing TGP before merging is allowed target: patch This PR is targeted for the next patch release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants