Skip to content

Conversation

@iamdanthedev
Copy link

Hi! I've been using the library for some time and got tired from importing it in a non-ts fashion, so here are type definitions, they seem to work. Although I've never wrote type defs for js libraries, some review would be appreciated if there is anyone able to do that

Thanks

Copy link
Contributor

@kylecesmat kylecesmat left a comment

Choose a reason for hiding this comment

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

Nice work! If you can handle the one comment and then ping me for re-review I can get this merged + tagged!

"object-assign": "^4.1.1",
"prop-types": "^15.5.10",
"react": "^16.2.0",
"react-dom": "^16.2.0",
Copy link
Contributor

Choose a reason for hiding this comment

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

Remove these from dependencies since they are already required in peerDependencies below with looser ranges :)

Choose a reason for hiding this comment

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

I'v done this in a new PR

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