Skip to content

Conversation

chrisbobbe
Copy link
Contributor

Part of the RN v0.62 -> v0.63 changes to the template app [1],
corresponding to facebook/react-native@82e823933.

That RN commit upgrades a few other deps; we don't take those
changes, for the following reasons:

  • We're already past ^7.6.2 of @babel/core.

  • We don't currently use @react-native-community/eslint-config
    (though we're considering it, in Add and enable @react-native-community/eslint-config. #4119).

  • We don't currently use babel-jest.

  • We're already past ^25.1.0 of jest.

  • We're already past ^0.58.0 of metro-react-native-babel-preset.

Run our usual yarn yarn-deduplicate && yarn, as prompted by
tools/test deps.

[1] https://react-native-community.github.io/upgrade-helper/?from=0.62.2&to=0.63.4

@chrisbobbe chrisbobbe requested a review from gnprice December 1, 2020 20:33
@chrisbobbe
Copy link
Contributor Author

Linking the RN v0.63 upgrade issue, #4245.

@chrisbobbe chrisbobbe mentioned this pull request Jan 4, 2021
Part of the RN v0.62 -> v0.63 changes to the template app [1],
corresponding to facebook/react-native@82e823933.

That RN commit upgrades a few other deps; we don't take those
changes, for the following reasons:

- We're already past ^7.6.2 of @babel/core.

- We don't currently use @react-native-community/eslint-config
  (though we're considering it, in zulip#4119).

- We don't currently use babel-jest.

- We're already past ^25.1.0 of jest.

- We're already past ^0.58.0 of metro-react-native-babel-preset.

Run our usual `yarn yarn-deduplicate && yarn`, as prompted by
`tools/test deps`.

[1] https://react-native-community.github.io/upgrade-helper/?from=0.62.2&to=0.63.4
@gnprice gnprice force-pushed the pr-upgrade-babel-runtime branch from de252fb to b1cd21e Compare January 4, 2021 22:29
@gnprice
Copy link
Member

gnprice commented Jan 4, 2021

Looks good, merged -- thanks!

@gnprice gnprice merged commit b1cd21e into zulip:master Jan 4, 2021
@chrisbobbe chrisbobbe deleted the pr-upgrade-babel-runtime branch November 5, 2021 00:00
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.

2 participants