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

React Native 0.71.4 → 0.71.7 #5761

Merged
merged 6 commits into from
Apr 24, 2023
Merged

React Native 0.71.4 → 0.71.7 #5761

merged 6 commits into from
Apr 24, 2023

Conversation

kraenhansen
Copy link
Member

What, How & Why?

Upgrading React Native to 0.71.7 using https://react-native-community.github.io/upgrade-helper/?from=0.71.4&to=0.71.7 in the root package and the integration test app. I considered doing the upgrade in the example app, but since this is an instance of our templates, I'd rather allow for a follow-up template upgrading those and the example app.

@kraenhansen kraenhansen self-assigned this Apr 24, 2023
@cla-bot cla-bot bot added the cla: yes label Apr 24, 2023
@kraenhansen kraenhansen requested review from takameyer and elle-j April 24, 2023 14:23
"react-native": "0.71.4",
"react-native": "0.71.7",
Copy link
Contributor

Choose a reason for hiding this comment

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

We have some more occurrences of 0.71.4 and 0.71.5 in various package.json. Should we update those too perhaps?

Copy link
Member Author

Choose a reason for hiding this comment

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

As the description mentions:

I considered doing the upgrade in the example app, but since this is an instance of our templates, I'd rather allow for a follow-up template upgrading those and the example app.

Are there other occurrences outside of the examples and our templates you're thinking of?

Copy link
Contributor

Choose a reason for hiding this comment

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

Our realm and realm-react packages.

Copy link
Member Author

@kraenhansen kraenhansen Apr 24, 2023

Choose a reason for hiding this comment

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

Good catch 🙂 I'll push that directly to main 🙈

@kraenhansen kraenhansen merged commit 156435c into main Apr 24, 2023
@kraenhansen kraenhansen deleted the kh/rn-0.71.7 branch April 24, 2023 14:41
papafe added a commit that referenced this pull request Apr 25, 2023
* main:
  Update yaml to version 2.2.2 (#5763)
  Upgraded RN to 0.71.7 for "realm" and "@realm/react"
  React Native 0.71.4 → 0.71.7 (#5761)
  Split changelog and compability matrix. Update readme and support. (#5748)
  fix build scripts for ios and android
  Removed simple test/example used to bootstrap bindgen project
  Make bindgen-lib's .eslintrc be freestanding, and copy to js bindgen dir
  move js_spec.yml out of bindgen-lib
  Move js-specific TS files out of bindgen-lib
  change --template arg to be path to source file
  move packages/bindgen to packages/realm/bindgen/vendor/bindgen-lib
  rename realm_js_helpers.h to just realm_helpers.h
  Iterating Android tests (#5746)
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Mar 15, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants