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

chore: set react version to the one in RN 0.59 #5499

Merged
merged 2 commits into from
Oct 24, 2019

Conversation

SimenB
Copy link
Contributor

@SimenB SimenB commented Oct 24, 2019

Checklist:

  • If this PR is a new feature, please reference an issue where a consensus about the design was reached (not necessary for small changes)
  • Make sure all of the significant new logic is covered by tests

#5451 (comment)

This ensures it'll work without errors for Expo. Current RN (0.61) is on React 16.9.

Btw, why is it in optionalDependencies and not peerDependencies ? You might get 2 copies of react then, which causes super weird errors

@hwillson
Copy link
Member

Thanks @SimenB! Now that npm/cli#224 has landed, we're going to switch to a combination of peerDependencies + peerDependenciesMeta. I'll add those changes here, so thanks for kickstarting this!

SimenB and others added 2 commits October 24, 2019 14:27
NPM just launched optional peer dependency support via
`peerDependenciesMeta` (npm/cli#224), so
we can now leverage this to make `react` a trully optional dep,
instead of using NPM's poorly named `optionalDependencies`
(since `optionalDependencies` are really just normal
dependencies). This means people using `@apollo/client` who
aren't interested in using React won't be annoyed by missing
`react` peer dependency warnings from npm.
@hwillson hwillson merged commit d12253e into apollographql:release-3.0 Oct 24, 2019
@SimenB SimenB deleted the patch-2 branch October 25, 2019 06:52
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Feb 1, 2023
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.

2 participants