Skip to content

feat: add React 19 peer dependency support#25

Merged
RostiMelk merged 1 commit intomainfrom
feat/react-19-support
Feb 9, 2026
Merged

feat: add React 19 peer dependency support#25
RostiMelk merged 1 commit intomainfrom
feat/react-19-support

Conversation

@RostiMelk
Copy link
Member

Widen peer dependencies to support React 18 and 19. Upgrade dev dependencies to React 19.

Closes #8

- Widen peerDependencies to accept React ^18 || ^19
- Upgrade devDependencies to React 19 for development/testing
- Fix test script to set NODE_ENV=development for React act() support

Closes #8
@vercel
Copy link

vercel bot commented Feb 9, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
react-logo-soup Ready Ready Preview, Comment Feb 9, 2026 7:59pm

Request Review

@github-actions
Copy link

github-actions bot commented Feb 9, 2026

Benchmark Comparison: main vs feat/react-19-support

Threshold: 5%+ change, >100us absolute delta, and statistically significant (p<0.05).

Benchmark main feat/react-19-support Change p-value Verdict
content detection (1 logo) 34.45 us 35.73 us +3.7% 0.505 unchanged
render pass (20 logos) 1.02 us 1.12 us +10.3% <0.001 *** unchanged
mount 20 logos (no detection) 3.30 us 3.41 us +3.0% 0.770 unchanged
mount 20 logos (defaults) 824.65 us 874.81 us +6.1% <0.001 *** unchanged

No regressions detected.

Feature cost breakdown

How expensive are individual features? Measured on this run's HEAD commit.

Feature On Off Cost Sig
densityAware: true vs false 33.34 us 32.93 us ~same 0.467
alignBy: visual-center-y vs bounds 1.05 us 81 ns 13× slower <0.001 ***
cropToContent: true vs false 2.09 ms 44 ns 2.09 ms <0.001 ***
layout update: full mount vs cached 867.59 us 2.09 us 416× slower <0.001 ***

Full benchmark output in the CI job logs.

@RostiMelk RostiMelk marked this pull request as ready for review February 9, 2026 20:02
@RostiMelk RostiMelk merged commit 8a04e2e into main Feb 9, 2026
4 checks passed
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.

React peer dependency

1 participant