Skip to content

refactor(core): Add warning when signal equality is false for object.is#52532

Closed
atscott wants to merge 1 commit intoangular:mainfrom
atscott:signalequal
Closed

refactor(core): Add warning when signal equality is false for object.is#52532
atscott wants to merge 1 commit intoangular:mainfrom
atscott:signalequal

Conversation

@atscott
Copy link
Copy Markdown
Contributor

@atscott atscott commented Nov 6, 2023

This commit adds a warning when a signal equality function returns false but Object.is returns true.

@atscott atscott added the target: rc This PR is targeted for the next release-candidate label Nov 6, 2023
This commit adds a warning when a signal equality function returns
`false` but `Object.is` returns `true`.
@atscott atscott added the action: merge The PR is ready for merge by the caretaker label Nov 6, 2023
@atscott atscott removed the request for review from alxhub November 6, 2023 22:26
@atscott
Copy link
Copy Markdown
Contributor Author

atscott commented Nov 6, 2023

This PR was merged into the repository by commit f887792.

@atscott atscott closed this in f887792 Nov 6, 2023
atscott added a commit that referenced this pull request Nov 6, 2023
…is (#52532)

This commit adds a warning when a signal equality function returns
`false` but `Object.is` returns `true`.

PR Close #52532
@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 Dec 7, 2023
ChellappanRajan pushed a commit to ChellappanRajan/angular that referenced this pull request Jan 23, 2024
…is (angular#52532)

This commit adds a warning when a signal equality function returns
`false` but `Object.is` returns `true`.

PR Close angular#52532
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: 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