-
Notifications
You must be signed in to change notification settings - Fork 55
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
[typescript] Result of hoc withRouter should not require props provided by it #223
Conversation
looks good, can you add a test exercising the type in types/text.jsx |
It was already exercised, not sure why it was not erroring because it did downstream in my app. |
@jquense Should we merge as-is, then, or do you want to look at the test? |
I'll take another look at the script. |
Whomever added dtslint ignored the existing test files. |
oops, that's unfortunate. can you fix that? |
Ok, original typescript tests added and fixed up based on the dtslint rules. Should be good to go. |
@jquense Can you take another look? If it's good, I'll cut a release. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, sorry missed these tests originally
released in v0.3.17 |
No description provided.