Rename AsyncMode -> ConcurrentMode#13732
Merged
Merged
Conversation
Run Prettier Fix up More renames
Details of bundled changes.Comparing: d0c0ec9...d02c8b8 react-test-renderer
react-is
scheduler
Generated by 🚫 dangerJS |
gaearon
reviewed
Sep 26, 2018
|
|
||
| export function isAsyncMode(object: any) { | ||
| return typeOf(object) === REACT_ASYNC_MODE_TYPE; | ||
| export function isConcurrentMode(object: any) { |
Collaborator
There was a problem hiding this comment.
Both this and symbol export rename would be a breaking change technically. Unfortunately we forgot the unstable prefix here. Let’s keep both and add a deprecation warning for isAsyncMode.
Contributor
Author
There was a problem hiding this comment.
I've made the change. Did I do it correctly?
gaearon
reviewed
Sep 26, 2018
| 'and will be removed in React 17+. Update your code to use ' + | ||
| 'ReactIs.isConcurrentMode() instead. It has the exact same API.', | ||
| ); | ||
| if (!hasWarnedAboutDeprecatedIsAsyncMode) { |
Collaborator
There was a problem hiding this comment.
This should be inside if (__DEV__)
gaearon
approved these changes
Sep 26, 2018
Contributor
|
Hey @trueadm. This change will require a corresponding change in react-devtools. Are you also going to own that? |
Contributor
Author
|
@bvaughn Just found the comment. Sure! |
Contributor
|
Is it the breaking change? |
Contributor
|
No. Both async and concurrent modes are "unstable_" (e.g. |
acdlite
pushed a commit
to plievone/react
that referenced
this pull request
Oct 5, 2018
* Rename AsyncMode -> ConcurrentMode
Merged
This was referenced Oct 24, 2018
This was referenced Oct 24, 2018
|
Are there any doc about the usage of this ConcurrentMode Mode? |
Collaborator
|
No — it's unstable. We only document stable features. |
This was referenced Oct 29, 2018
gaearon
pushed a commit
that referenced
this pull request
Oct 31, 2018
This was referenced Nov 1, 2018
This was referenced Dec 1, 2018
jetoneza
pushed a commit
to jetoneza/react
that referenced
this pull request
Jan 23, 2019
* Rename AsyncMode -> ConcurrentMode
jetoneza
pushed a commit
to jetoneza/react
that referenced
this pull request
Jan 23, 2019
…3959) - Partial revert of react#13732 - Fixes react#13958.
NMinhNguyen
referenced
this pull request
in enzymejs/react-shallow-renderer
Jan 29, 2020
* Rename AsyncMode -> ConcurrentMode
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
7 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This PR renames
AsyncModetoConcurrentMode. Specifically:Reactexportsunstable_ConcurrentModenow rather thanunstable_AsyncMode.ReactTestRendernow takesunstable_isConcurrentas an option rather thanunstable_isAsync.isAsynchave been renamed toisConcurrentto reflect the naming changes.REACT_ASYNC_MODE_TYPEhas been renamed toREACT_CONCURRENT_MODE_TYPE.react-isnow exportsisConcurrentMode, renamed fromisAsyncMode.