Skip to content

Move dep for react-dom #592

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

Merged
merged 1 commit into from
May 31, 2018

Conversation

jooj123
Copy link
Contributor

@jooj123 jooj123 commented May 30, 2018

react-dom should be a peerDep and not a dependancy

Consumers should be deciding if they want to include this as part of their bundle not the library

Copy link
Member

@itsmichaeldiego itsmichaeldiego left a comment

Choose a reason for hiding this comment

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

Great work dude! This is getting merged and will be released in 1.0.4

@itsmichaeldiego itsmichaeldiego merged commit fc062a0 into google-map-react:master May 31, 2018
@jooj123 jooj123 deleted the patch/fix-deps branch June 1, 2018 07:53
@jooj123
Copy link
Contributor Author

jooj123 commented Jun 1, 2018

thanks @itsmichaeldiego 👍

@jooj123
Copy link
Contributor Author

jooj123 commented Jun 8, 2018

@itsmichaeldiego any chance we can get this one released?

@prasadhewage
Copy link

@itsmichaeldiego hey, can we have this change soon.

@itsmichaeldiego
Copy link
Member

@jooj123 @prasadhewage Sorry guys, I am in holidays in Greece and I haven't been that active.

I released this on v1.0.4

Cheers!

@lock
Copy link

lock bot commented Dec 1, 2019

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@lock lock bot locked as resolved and limited conversation to collaborators Dec 1, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants