New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
perf(core): allow checkNoChanges
mode to be tree-shaken in production
#45913
Conversation
This commit guards all logic that exists for the `checkNoChanges` mode with `ngDevMode` checks such that the logic can be tree-shaken.
5f282b9
to
83f75f3
Compare
This saves 65514-65165=349 bytes in the |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@JoostK thanks for the improvement! 👍 I've added a few minor comments.
bf4232a
to
ad3103a
Compare
This PR was merged into the repository by commit 75b3d0f. |
@JoostK there was a merge conflict with the 13.3.x branch, so this PR was merged into the 14.0.x and main branches only. Could you please create a new PR and target the 13.3.x branch if the change should be included there as well? Thank you. |
This issue has been automatically locked due to inactivity. Read more about our automatic conversation locking policy. This action has been performed automatically by a bot. |
This commit guards all logic that exists for the
checkNoChanges
modewith
ngDevMode
checks such that the logic can be tree-shaken.