[lint] Upgrade to Prettier v3, typescript-eslint to v6 (#23544)Why --- Prettier 3 is out. Add support for it with this linter config. **Note for reviewer:** the first commit is the one with th
[lint] Upgrade to Prettier v3, typescript-eslint to v6 (#23544)Why --- Prettier 3 is out. Add support for it with this linter config. **Note for reviewer:** the first commit is the one with the actual changes. The rest of this PR are changes to get the linter passing (mostly autofix). How --- Update eslint-config-prettier and eslint-plugin-prettier. To address deprecation warnings, also update typescript-eslint/parser and typescript-eslint/eslint-plugin. Because of an update to typescript-eslint/parser, we need to suppress deprecation warnings (documented in a comment). Regenerated test snapshots. Due to the upgraded dependencies, typecasts and optional chaining are now auto-fixable by lint. This converts warnings into autofixes. Test Plan --- `yarn test` in the linter config. Run `expotools check --all --fix-lint --no-build --no-test --no-uniformity-check` to try this config on the whole repo. --------- Co-authored-by: Expo Bot <[email protected]>
show more ...
feat(expo)!: use Expo CLI when building for development in Xcode (#21397)# Why - Related #21396 - Use `npx expo start --dev-client` instead of `npx react-native start` when building a project
feat(expo)!: use Expo CLI when building for development in Xcode (#21397)# Why - Related #21396 - Use `npx expo start --dev-client` instead of `npx react-native start` when building a project from Xcode. This ensures we use the correct version of Metro regardless of where the project is started. - It's unclear if building from Android Studio is supposed to trigger the bundler in development but I couldn't get it to do so, so this PR is iOS-only. <!-- Please describe the motivation for this PR, and link to relevant GitHub issues, forums posts, or feature requests. --> # How - Copy over a bunch of the default scripts from `react-native/scripts` into the same locations in `expo/scripts` to make it easy to maintain this change and simple to migrate. I decided to omit the following as it appeared to be unused (sets RCT_METRO_PORT): ``` source "$THIS_DIR/.packager.env" ``` I also dropped: - Support for `$RCT_PACKAGER_LOGS_DIR`. - The `Process terminated. Press <enter> to close the window` log that is presented after closing the dev server. - Support for a custom React Native CLI config (this is not supported in Expo CLI). <!-- How did you build this feature or fix this bug and why? --> # Test Plan - TBD - Building for development from Xcode causes a terminal window running `npx expo start` will open instead of the community version. <!-- Please describe how you tested this change and how a reviewer could reproduce your test, especially if this PR does not include automated tests! If possible, please also provide terminal output and/or screenshots demonstrating your test/reproduction. --> # Checklist <!-- Please check the appropriate items below if they apply to your diff. This is required for changes to Expo modules. --> - [ ] Documentation is up to date to reflect these changes (eg: https://docs.expo.dev and README.md). - [ ] Conforms with the [Documentation Writing Style Guide](https://github.com/expo/expo/blob/main/guides/Expo%20Documentation%20Writing%20Style%20Guide.md) - [ ] This diff will work correctly for `expo prebuild` & EAS Build (eg: updated a module plugin). --------- Co-authored-by: Expo Bot <[email protected]> Co-authored-by: Kudo Chien <[email protected]>
Re-export config-plugins from expo package and update docs (#18855)
[packages] update problematic doc comments, regenerate files (#18128)
[eslint-config] remove `arrowParens` overwrite, reformat files (#14232)
[notifications] Check if icon file exists before removing (#13539)* Check if file exists before deleting * add test * changelog * Update packages/expo-notifications/CHANGELOG.md Co-auth
[notifications] Check if icon file exists before removing (#13539)* Check if file exists before deleting * add test * changelog * Update packages/expo-notifications/CHANGELOG.md Co-authored-by: Expo CI <[email protected]> Co-authored-by: Expo CI <[email protected]>
Bump config-plugins to v3.0.0 (#13390)* Bump config-plugins to v3.0.0 * Update withNotificationsiOS-test.ts * Update withNotificationsAndroid-test.ts
[notifications] feat: add custom notification sound files plugin (#12782)* add config plugin for notification sounds * add sound test android * add ios sound test * lint plugin * docs
[notifications] feat: add custom notification sound files plugin (#12782)* add config plugin for notification sounds * add sound test android * add ios sound test * lint plugin * docs * update readme * changelog * move functions to be tested into it. statements * feedback on NotificationsPluginProps * remove fs-extra, move some methods from async to sync * fix argument order * address iOS feedback * update docs and readme * lint * add plugin to ncl * add test to NCL * account for breaking change in @expo/config-plugins * set isBuildFile: true * re yarn after rebase * lint
Bump config plugins, config, metro-config, xdl, webpack-config (#13061)* Bump config plugins and config * extract statics
Remove deep imports from plugins (#13029)* Remove deep imports from plugins * changelog
[expo-facebook][expo-notifications] Fix XML module import path
created expo-notifications plugin (#11633)* created expo-notifications plugin * Update CHANGELOG.md * Update app.plugin.js * Update CHANGELOG.md * fix lint * Update package.json *
created expo-notifications plugin (#11633)* created expo-notifications plugin * Update CHANGELOG.md * Update app.plugin.js * Update CHANGELOG.md * fix lint * Update package.json * Update yarn.lock