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

Clarify react-native dependency to be >=0.51.0, < 1.0.0 #481

Merged
merged 2 commits into from
Sep 26, 2018

Conversation

Empact
Copy link
Contributor

@Empact Empact commented Apr 20, 2018

For 1.0.0 and above, caret means "matches major version",
but for 0.x.0 and above, caret means "matches major and minor version".

It's clear from usage that the intention for this dependency is to
match the looser 1.0.0 caret usage. This change matches that, enabling
explicit compatibility with react-native releases from the past 5 months.

https://docs.npmjs.com/misc/semver#caret-ranges-123-025-004
https://www.npmjs.com/package/react-native

For 1.0.0 and above, caret means "matches major version",
but for 0.x.0 and above, caret means "matches major and minor version".

It's clear from usage that the intention for this dependency is to
match the looser 1.0.0 caret usage. This change matches that, enabling
explicit compatibility with react-native releases from the past 5 months.

https://docs.npmjs.com/misc/semver#caret-ranges-123-025-004
https://www.npmjs.com/package/react-native
@Empact
Copy link
Contributor Author

Empact commented Apr 20, 2018

Partially fixes #439

@sesam
Copy link

sesam commented Jul 22, 2018

ping @Empact and maintainers, any love for this? Leaving warnings around unattended is creating a bad smell, mmkay :)

@Empact
Copy link
Contributor Author

Empact commented Jul 23, 2018

Resolved merge conflicts

@sesam
Copy link

sesam commented Sep 26, 2018

@itinance @gensc004 Have you had time to look at this PR? This is a logical next step after #535.
If you don't want <1.0, then maybe ">=0.51.0 <0.58.0"

@itinance itinance merged commit 90211b5 into itinance:master Sep 26, 2018
@itinance
Copy link
Owner

Release is out. Thx for the reminder, @sesam !

@vicary vicary mentioned this pull request Apr 4, 2020
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