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

test: Run with relevant React stable types #1352

Merged
merged 1 commit into from
Mar 17, 2025

Conversation

eps1lon
Copy link
Member

@eps1lon eps1lon commented Aug 28, 2024

This has no effect right now since latest and 18.x point at the same version.
Once React 19 is out, we'll be running tests in both 18 and 19.
The types should match that dimension.

Ideally, we'd be able to test RC types but npm claims npm install @types/react@npm:types-react@rc is an invalid comparator.
Putting that in the package.json works but I'm too lazy to write a dedicated script right now.

Copy link

codesandbox-ci bot commented Aug 28, 2024

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

Latest deployment of this branch, based on commit d955710:

Sandbox Source
react-testing-library-examples Configuration

@eps1lon eps1lon force-pushed the type-test-matrix branch 2 times, most recently from 7a954f1 to 88abd93 Compare August 28, 2024 15:53
This has no effect right now.
Once React 19 is out, we're running tests in both 18 and 19.
The types should match that dimension.

Ideally, we'd be able to test RC types but npm claims `npm install @types/react@npm:types-react@rc` is an invalid comparator.
Putting that in the package.json works but I'm too lazy to write a dedicated script right now.
@eps1lon eps1lon enabled auto-merge (squash) March 17, 2025 21:23
@eps1lon eps1lon disabled auto-merge March 17, 2025 21:24
@eps1lon eps1lon merged commit 65bc994 into testing-library:main Mar 17, 2025
10 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.

3 participants