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

Make react and preact peerDependencies both optional #140

Merged
merged 1 commit into from
Mar 1, 2019

Conversation

developit
Copy link
Owner

This avoids misleading npm warnings. Fixes #139.
/cc @Timer @NathanielHill

@NathanielHill
Copy link

Awesome thanks. I believe this effects everyone using Next, so a big win to remove these errors 👍

@developit developit merged commit c7f69af into master Mar 1, 2019
@jamesreggio
Copy link

@developit — do you mind cutting a release? I see you landed a handful of PRs about a week ago, so it might be a good time to bundle them up.

@developit
Copy link
Owner Author

developit commented Mar 10, 2019

@jamesreggio yup, sorry about that - release is out now, I took some time to add support for preact X and fixed a couple issues in the react integration.

@developit developit deleted the optional-peer-deps branch March 10, 2019 20:42
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