Skip to content

fix(create-expo-nightly): Tighten workspace pattern#41410

Merged
kitten merged 1 commit intomainfrom
@kitten/fix/workspaces-traversal-create-expo-nightly
Dec 10, 2025
Merged

fix(create-expo-nightly): Tighten workspace pattern#41410
kitten merged 1 commit intomainfrom
@kitten/fix/workspaces-traversal-create-expo-nightly

Conversation

@kitten
Copy link
Copy Markdown
Member

@kitten kitten commented Dec 4, 2025

Why

See CI failure in: https://github.com/expo/expo/actions/runs/19916741962/job/57096975273
Caused by symlink added in: #41383

How

  • Update the workspaces pattern to only find Expo package folders by workspace specifier name (mirroring the path in Project.ts and root package.json:workspaces)
  • Ignore local-template in @expo/cli

Test Plan

  • CI should pass the mentioned run

Checklist

@kitten kitten requested review from Kudo, hassankhan and vonovak December 4, 2025 12:09
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Dec 4, 2025

Subscribed to pull request

File Patterns Mentions
packages/create-expo-nightly/** @Kudo

Generated by CodeMention

@kitten kitten changed the title fix/ci(create-expo-nightly): Tighten workspace pattern fix(create-expo-nightly): Tighten workspace pattern Dec 4, 2025
@expo-bot expo-bot added the bot: passed checks ExpoBot has nothing to complain about label Dec 4, 2025
Comment thread packages/create-expo-nightly/src/Packages.ts
@kitten kitten force-pushed the @kitten/fix/workspaces-traversal-create-expo-nightly branch from fbe88a9 to 6097b56 Compare December 8, 2025 20:26
@kitten kitten requested a review from Kudo December 8, 2025 20:26
@kitten kitten merged commit fdea731 into main Dec 10, 2025
12 of 14 checks passed
@kitten kitten deleted the @kitten/fix/workspaces-traversal-create-expo-nightly branch December 10, 2025 19:36
aleqsio pushed a commit that referenced this pull request Dec 22, 2025
# Why

See CI failure in:
https://github.com/expo/expo/actions/runs/19916741962/job/57096975273
Caused by symlink added in: #41383

# How

- ~~Update the workspaces pattern to only find Expo package folders by
workspace specifier name (mirroring the path in `Project.ts` and root
`package.json:workspaces`)~~
- Ignore `local-template` in `@expo/cli`

# Test Plan

- CI should pass the mentioned run

# Checklist

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

- [ ] 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)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bot: fingerprint compatible bot: passed checks ExpoBot has nothing to complain about

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants