Skip to content

chore(React19): Enable React19 #725

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

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

rebeccaalpert
Copy link
Member

@rebeccaalpert rebeccaalpert commented May 5, 2025

Did not run into the same circular dependency issues as I did in ChatBot, but changing the React dev dependency breaks the docs framework. It does build with every React version, but the docs framework stops working with 19. Leaving the dev dependency at 18 for now until we can drop the docs framework.

@patternfly-build
Copy link

patternfly-build commented May 5, 2025

@rebeccaalpert rebeccaalpert force-pushed the react19 branch 5 times, most recently from dc50285 to 66b217a Compare May 5, 2025 18:42
@rebeccaalpert rebeccaalpert linked an issue May 5, 2025 that may be closed by this pull request
2 tasks
@rebeccaalpert rebeccaalpert changed the title Draft: chore(React19): Enable React19 chore(React19): Enable React19 May 5, 2025
@rebeccaalpert rebeccaalpert requested review from kmcfaul and a team and removed request for a team May 5, 2025 18:43
@rebeccaalpert rebeccaalpert force-pushed the react19 branch 8 times, most recently from fd86f23 to ee576b6 Compare May 5, 2025 19:23
@Hyperkid123
Copy link
Collaborator

@karelhala the changes should be fine. The only real breaking changes were the removal of prop types which are not used here anyway. I might have some time tomorrow, but the fact the Cypress tests are passing is a good enough sign to me that the runtime is not broken.

@rebeccaalpert rebeccaalpert force-pushed the react19 branch 2 times, most recently from 529ef83 to 71cb6f0 Compare May 7, 2025 14:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

component groups - react 19 support
4 participants