Skip to content

[pull] master from facebook:master#410

Merged
pull[bot] merged 3 commits intoJCofman:masterfrom
facebook:master
Mar 20, 2021
Merged

[pull] master from facebook:master#410
pull[bot] merged 3 commits intoJCofman:masterfrom
facebook:master

Conversation

@pull
Copy link
Copy Markdown

@pull pull bot commented Mar 20, 2021

See Commits and Changes for more details.


Created by pull[bot]

Can you help keep this open source service alive? 💖 Please sponsor : )

acdlite added 3 commits March 19, 2021 15:28
We don't need this anymore. It only existed so we could cancel the
callback later. But canceling isn't necessary, was only an
"optimization" for something that almost never happens in practice.
We added this unstable feature a few years ago, as a way to opt out of
context updates, but it didn't prove useful in practice.

We have other proposals for how to address the same problem, like
context selectors.

Since it was prefixed with `unstable_`, we should be able to remove it
without consequence. The hook API already warned if you used it.

Even if someone is using it somewhere, it's meant to be an optimization
only, so if they are using the API properly, it should not have any
semantic impact.
@pull pull bot added the ⤵️ pull label Mar 20, 2021
@pull pull bot merged commit 6d3ecb7 into JCofman:master Mar 20, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant