Skip to content

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 a team and kmcfaul and removed request for a team May 5, 2025 18:43
@rebeccaalpert rebeccaalpert force-pushed the react19 branch 9 times, most recently from ee576b6 to 71cb6f0 Compare May 5, 2025 19:44
@karelhala karelhala requested a review from Hyperkid123 May 6, 2025 07:30
@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
@nicolethoen
Copy link
Contributor

@rebeccaalpert could you rename this PR and your first commit message so that this is a feat not a chore and will publish a new prerelease of the component-groups when this merges?

@rebeccaalpert rebeccaalpert changed the title chore(React19): Enable React19 feat(React19): Enable React19 May 15, 2025
@rebeccaalpert
Copy link
Member Author

All set!

"@patternfly/react-code-editor": "^6.0.0",
"@patternfly/patternfly": "^6.0.0",
"@types/react": "^18.2.33",
"@types/react-dom": "^18.3.1",
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These can be reverted

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should be all set!

@rebeccaalpert rebeccaalpert force-pushed the react19 branch 3 times, most recently from 404e6ba to 38445a3 Compare May 16, 2025 16:02
@rebeccaalpert rebeccaalpert merged commit 1ee5173 into patternfly:main May 19, 2025
6 checks passed
@github-actions
Copy link

🎉 This PR is included in version 6.3.0-prerelease.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

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.

component groups - react 19 support

5 participants