Skip to content
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

[test] Run React 18 integration tests with new createRoot API #26672

Merged
merged 46 commits into from
Jul 5, 2021

Conversation

eps1lon
Copy link
Member

@eps1lon eps1lon commented Jun 9, 2021

Passing with React 18 in 399f5af: https://app.circleci.com/pipelines/github/mui-org/material-ui/48718/workflows/89980172-722c-433a-8ce1-8a14202ca20f

It's mostly looking fine. There are some TODO items where we either can't run with createRoot or have to disable StrictMode (due to Strict Effects).

I'll work on the remaining TODO items incrementally. We should start continously testing integration though by merging this PR.

@eps1lon eps1lon added the test label Jun 9, 2021
@mui-pr-bot
Copy link

mui-pr-bot commented Jun 9, 2021

No bundle size changes (experimental)

Generated by 🚫 dangerJS against 89e4735

@eps1lon eps1lon force-pushed the test/concurrent-react branch from c7bdf1e to 4daf16f Compare June 9, 2021 18:23
@github-actions github-actions bot added PR: out-of-date The pull request has merge conflicts and can't be merged and removed PR: out-of-date The pull request has merge conflicts and can't be merged labels Jun 11, 2021
@eps1lon eps1lon force-pushed the test/concurrent-react branch 3 times, most recently from e4cc31e to eec31aa Compare June 11, 2021 09:42
@eps1lon eps1lon force-pushed the test/concurrent-react branch from eec31aa to 7a69c35 Compare June 13, 2021 16:22
@eps1lon eps1lon force-pushed the test/concurrent-react branch from 7a69c35 to 6c5e504 Compare June 13, 2021 16:38
@siriwatknp

This comment has been minimized.

@eps1lon

This comment has been minimized.

@siriwatknp

This comment has been minimized.

@eps1lon

This comment has been minimized.

@eps1lon eps1lon force-pushed the test/concurrent-react branch from 691c693 to b6034d9 Compare June 29, 2021 10:46
@eps1lon eps1lon force-pushed the test/concurrent-react branch from b70a8bf to a8a746d Compare June 30, 2021 21:29
@eps1lon eps1lon changed the title [test] Run react@next integration tests with new createRoot API [test] Run React 18 integration tests with new createRoot API Jul 1, 2021
@eps1lon eps1lon added this to the React 18 milestone Jul 1, 2021
@eps1lon eps1lon force-pushed the test/concurrent-react branch from c394a46 to 399f5af Compare July 1, 2021 15:09
@eps1lon eps1lon marked this pull request as ready for review July 1, 2021 16:25
@eps1lon eps1lon merged commit 699b5e9 into mui:next Jul 5, 2021
@eps1lon eps1lon deleted the test/concurrent-react branch July 5, 2021 16:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants