Skip to content

Update React peerDependency range #2 #91

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

Merged
merged 1 commit into from
May 9, 2025
Merged

Conversation

shiueo
Copy link
Contributor

@shiueo shiueo commented Apr 23, 2025

This PR updates the react peerDependency to allow React 19 ("react": ">=15.3.2 <20").

This is a follow-up to #39, and it also addresses #86, where support for newer versions of React is requested.

Updating this range allows projects using React 19 to install react-katex without peer dependency warnings

Test Results

--------------------|---------|----------|---------|---------|-------------------
File                | % Stmts | % Branch | % Funcs | % Lines | Uncovered Line #s 
--------------------|---------|----------|---------|---------|-------------------
All files           |    98.8 |      100 |     100 |   98.68 |                   
 src                |      96 |      100 |     100 |   95.65 |                   
  index.jsx         |      96 |      100 |     100 |   95.65 | 53                
 tests              |     100 |      100 |     100 |     100 |                   
  sharedExamples.js |     100 |      100 |     100 |     100 |                   
--------------------|---------|----------|---------|---------|-------------------

Test Suites: 2 passed, 2 total
Tests:       24 passed, 24 total
Snapshots:   0 total
Time:        1.828 s
Ran all test suites.

You can view the full test log for this PR here. (Tested via Github CI.)

I have been using React 19 with react-katex for several months without encountering any issues!

@thiagoloureiro
Copy link

@talyssonoc help us :)

@talyssonoc talyssonoc merged commit be1b098 into talyssonoc:master May 9, 2025
talyssonoc added a commit that referenced this pull request May 9, 2025
@thiagoloureiro
Copy link

thanks Talysson! you plan to deploy to npm?

@talyssonoc
Copy link
Owner

Yes! I will first release a version just with this change, and then I plan to do a revamp on the dependencies of the project :)

@thiagoloureiro
Copy link

sounds fantastic! will keep an eye here, obrigado!!

@talyssonoc
Copy link
Owner

Just published :)

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.

3 participants