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

peerDeps React 19 #242

Closed
wants to merge 2 commits into from
Closed

Conversation

slorber
Copy link
Contributor

@slorber slorber commented Dec 20, 2024

This is the forked v1.3 version adding support for React 19 in Docusaurus (see facebook/docusaurus#10763)

I didn't upgrade to v2 and used an old tag for the fork because it's not clear what the v2 breaking changes are (undocumented, no changelog)

@slorber
Copy link
Contributor Author

slorber commented Dec 24, 2024

mmm sorry, it was meant to be a PR on my own fork, not the upstream repo.

But IMHO you should still allow React 19 as peerDeps

As the maintainer of Docusaurus, I can see that this lib (v1.3.0) works fine under React 19.

However, I don't know if v2.x works. We didn't upgrade to v2 because it's not clear to me what the v2 breaking changes are (no changelog nor release notes 🤷 ), so I just published a fork of v1.3.0 that allows React 19 and it's working fine.

@slorber slorber closed this Dec 24, 2024
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.

2 participants