Skip to content

Bump to expo sdk 58 - #4612

Merged
m-bert merged 1 commit into
mainfrom
@mbert/bump-expo-58
Oct 19, 2026
Merged

m-bert merged 1 commit into
mainfrom
@mbert/bump-expo-58

Conversation

@m-bert

@m-bert m-bert commented Oct 19, 2026

Copy link
Copy Markdown
Collaborator

Description

This PR bumps expo-example to Expo SDK 58

Test plan

Run example apps

Copilot AI balanced review requested due to automatic review settings October 19, 2026 09:32

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

An organization owner can choose the repository organization as the billing source for reviews requested by members with a paid Copilot license. Learn about Copilot code review billing. Review the Copilot code review billing settings.

@m-bert
m-bert requested a review from coado October 19, 2026 09:32
@m-bert

m-bert commented Oct 19, 2026

Copy link
Copy Markdown
Collaborator Author

@coderabbitai where review

@coderabbitai

coderabbitai Bot commented Oct 19, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration
  • Configuration used: Repository UI
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 9e828f1a-3232-4e48-b921-09d90adbed7b
📥 Commits

Reviewing files that changed from the base of the PR and between edcdd44 and 375416f.

⛔ Files ignored due to path filters: 2
  • .yarn/patches/expo-modules-core-npm-57.0.13-a9a5cb7828.patch is excluded by !**/.yarn/**
  • yarn.lock is excluded by !**/yarn.lock, !**/*.lock
📒 Files selected for processing: 2
  • apps/expo-example/package.json
  • package.json

Included review availability: This review used your included allowance. Your plan provides up to 8 included reviews per hour; 7 remain after this review.

📝 Summary

Summary by CodeRabbit

  • Chores
    • Updated the example app to newer Expo, React, and React Native versions, keeping it aligned with the latest supported development platform.
    • Refreshed related type definitions and removed outdated package overrides.

The Expo example updates Expo, React Native, and related package versions. The root package removes Yarn resolutions for expo-modules-jsi and the patched expo-modules-core dependency.

Changes: Expo example dependency updates

Layer / File(s) Summary
Update Expo example dependencies
apps/expo-example/package.json, package.json
The Expo example updates Expo, React Native, and related package versions. The root package removes resolutions for expo-modules-jsi and expo-modules-core.

Priority: ⬇️ Low

Merge Risk: ⚪ Minimal · up to 37541

No actionable dependency mismatch was identified, so the PR appears mergeable after normal checks. The example’s runtime build was not verified.

Pre-merge checks: Passed 3
Check name Status Explanation
Title check Passed The title clearly identifies the main change: updating the Expo example to Expo SDK 58.
Linked Issues check Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check Passed Check skipped because no linked issues were found for this pull request.
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Warning

Some tools did not complete. Review the errors below.

🔧 ESLint

If the error stems from missing dependencies, add them to the package.json file. For unrecoverable errors (e.g., due to private dependencies), disable the tool in the CodeRabbit configuration.

package.json

Parsing error: ESLint was configured to run on <tsconfigRootDir>/package.json using parserOptions.project: /tsconfig.json
The extension for the file (.json) is non-standard. You should add parserOptions.extraFileExtensions to your config.

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@coderabbitai

coderabbitai Bot commented Oct 19, 2026

Copy link
Copy Markdown

Tip

For best results, initiate chat on the files or code changes.

🧩 Analysis chain

🏁 Script executed:

#!/bin/bash
gh pr view 4612 --repo software-mansion/react-native-gesture-handler --json reviews,comments,statusCheckRollup --jq '{reviews: [.reviews[] | {author: .author.login, state, submittedAt, body}], comments: [.comments[] | select(.author.login | test("coderabbit"; "i")) | {author: .author.login, createdAt, body}], checks: .statusCheckRollup}'

Length of output: 14795


@m-bert The review is complete. The result is in the CodeRabbit summary comment: “No actionable comments were generated in the recent review.” No separate review was posted.

The review covered both package.json files. Path filters excluded yarn.lock and the deleted Yarn patch. ESLint could not parse the root package.json; that error is listed in the same comment.

You are interacting with an AI system.

@m-bert
m-bert merged commit 8665cd1 into main Oct 19, 2026
11 checks passed
@m-bert
m-bert deleted the @mbert/bump-expo-58 branch October 19, 2026 14:13
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