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

[@testing-library/react_v8.x.x] and [@testing-library/react_v9.x.x]: Fix typing regression #3704

Merged

Conversation

valscion
Copy link
Contributor

@valscion valscion commented Jan 17, 2020

Other notes:

Fixes #3520, that is, @testing-library/react types being broken ever since #3510 was merged in.

The first commit in this PR shows how the existing types are broken.

@valscion valscion changed the title Trigger tests of existing @testing-library/react types [@testing-library/react_v8.x.x] and [@testing-library/react_v9.x.x]: Fix typing regression Jan 17, 2020
@valscion
Copy link
Contributor Author

valscion commented Jan 17, 2020

Re-pushed my last commit as the CI got stuck last time: https://travis-ci.org/flow-typed/flow-typed/jobs/638417425

EDIT: Oh, that didn't trigger a re-run in CI. Does the good old classic close+reopen work?
EDIT2: Yup, that triggered a new CI run.

container?: HTMLElement,
baseElement?: HTMLElement,
hydrate?: boolean,
queries?: Q,
queries?: any,
Copy link
Contributor Author

@valscion valscion Jan 17, 2020

Choose a reason for hiding this comment

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

If we want to get rid of this any here, the code for that is over at #3705

I wanted to keep this PR easy to review while #3705 is a bit of a mouthful.

Copy link
Member

@pascalduez pascalduez left a comment

Choose a reason for hiding this comment

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

Thanks!

@pascalduez pascalduez merged commit 5556be3 into flow-typed:master Jan 18, 2020
@valscion valscion deleted the fix-testing-library-react-types branch January 18, 2020 15:10
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.

[@testing-library/react] Improve DX for render method
2 participants