-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
feat: upgrade to Expo SDK 52 #2843
Conversation
@@ -26,6 +26,8 @@ module.exports = { | |||
}, | |||
], | |||
"@typescript-eslint/no-var-requires": 0, | |||
"@typescript-eslint/no-require-imports": 0, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Seems like the new eslint rules from Expo turned this on, so I kept it off for consistency.
@@ -26,6 +26,8 @@ module.exports = { | |||
}, | |||
], | |||
"@typescript-eslint/no-var-requires": 0, | |||
"@typescript-eslint/no-require-imports": 0, | |||
"@typescript-eslint/no-empty-object-type": 0, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Seems like the new eslint rules from Expo turned this on. I disabled it because we use this pattern quite a bit with some of the MST helpers.
First of all, thank you so much for this @frankcalise For anyone (just like me) wondering to get expo sdk 52 working: npx ignite-cli@latest new MyApp
yarn add expo@^52.0.0
npx expo install --fix
add this to your {
"expo": {
"newArchEnabled": true
}
} and then run npx expo prebuild --clean && npx expo run:android |
…creen styling and drawer fix (#2856 by @frankcalise)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
# [10.1.0](v10.0.5...v10.1.0) (2024-12-10) ### Features * upgrade to Expo SDK 52 ([#2843](#2843) by [@frankcalise](https://github.com/frankcalise)) ([fc2b357](fc2b357)), closes [#2856](#2856) [#2862](#2862)
🎉 This PR is included in version 10.1.0 🎉 The release is available on: Your semantic-release bot 📦🚀 |
Please verify the following:
yarn test
jest tests pass with new tests, if relevantyarn lint
eslint checks pass with new code, if relevantyarn format:check
prettier checks pass with new code, if relevantREADME.md
(or relevant documentation) has been updated with your changesDescribe your PR
CLI
--newArch
Boilerplate
npx expo install --fix
expo-splash-screen
configuration via config plugin (as recommended) over app.jsonsplash
keysreact-native-drawer-layout
patch now that v4 was releasedreact-native-mmkv
version for new architecture to be compatible with react-native 0.76.x