Skip to content

[pull] main from expo:main#4

Merged
pull[bot] merged 3 commits into
code:mainfrom
expo:main
Jul 15, 2025
Merged

[pull] main from expo:main#4
pull[bot] merged 3 commits into
code:mainfrom
expo:main

Conversation

@pull

@pull pull Bot commented Jul 15, 2025

Copy link
Copy Markdown

See Commits and Changes for more details.


Created by pull[bot] (v2.0.0-alpha.3)

Can you help keep this open source service alive? 💖 Please sponsor : )

Kudo and others added 3 commits July 15, 2025 12:00
# Why

fix running error on sdk 53 projects

# How

1. fix patch-project running error 
  ```
AssertionError [ERR_ASSERTION]: The return value should be string when
recursive is true
  ```

it's a regression from #35036. the `ensureDirectoryAsync` expects to
return path

2. on sdk53, **project.pbxproj** has some more dynamic generated
properties. we have to normalize more. this pr adds `@bacons/xcode` to
simplify the normalization code.

3. also fix e2e test
# Why

fix patch apply when there were space differences

# How

add `--ignore-whitespace` to git apply
# Why

On web standalone `Modal` was using default react-navigation
implementation. This PR adds usage of new web modal from `expo-router`

# How

1. Use different modal on web

# Test Plan

Manual testing

# Checklist

<!--
Please check the appropriate items below if they apply to your diff.
-->

- [x] I added a `changelog.md` entry and rebuilt the package sources
according to [this short
guide](https://github.com/expo/expo/blob/main/CONTRIBUTING.md#-before-submitting)
- [ ] This diff will work correctly for `npx expo prebuild` & EAS Build
(eg: updated a module plugin).
- [ ] Conforms with the [Documentation Writing Style
Guide](https://github.com/expo/expo/blob/main/guides/Expo%20Documentation%20Writing%20Style%20Guide.md)
@pull pull Bot locked and limited conversation to collaborators Jul 15, 2025
@pull pull Bot added the ⤵️ pull label Jul 15, 2025
@pull pull Bot merged commit 7f8096e into code:main Jul 15, 2025
20 of 26 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants