Skip to content
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

Monorepo #58

Closed
wants to merge 9 commits into from
Closed

Monorepo #58

wants to merge 9 commits into from

Conversation

shuesken
Copy link
Contributor

@shuesken shuesken commented Jan 7, 2025

No description provided.

@chmac
Copy link
Member

chmac commented Jan 10, 2025

This isn't just a monorepo PR oder? It seems to also break running the app via Expo Go. That means that the "normal" development process won't work. Now everybody needs to run a local build. I also don't see that documented anywhere in the readme.

I think we should be very careful before we merge. Maybe it makes sense to wait until after nostroots 7, or to make sure we have a bulletproof setup working that everybody can replicate.

When I try to build android locally, I get this error:

pkgx +eas eas build --platform android --local
The EAS build profile does not specify a Node.js version. Using the version specified in .nvmrc: 20.18.1
Failed to read the app config from the project using "npx expo config" command: spawn npx ENOENT.
Falling back to the version of "@expo/config" shipped with the EAS CLI.
We detected that you are building for the "store" distribution. Resolving the environment for environment variables used during the build t.
No environment variables with visibility "Plain text" and "Sensitive" found for the "production" environment on EAS servers.

✔ Incremented versionCode from 6 to 7.
✔ Using remote Android credentials (Expo server)
✔ Using Keystore from configuration: Build Credentials iJvIg2YPf5 (default)
✔ Compressed project files 1s (49.1 MB)
✔ Computed project fingerprint
spawn npm ENOENT
    Error: build command failed.

@shuesken
Copy link
Contributor Author

squashed commits and removed the build file, continuing conversation here: #59

@shuesken shuesken closed this Jan 14, 2025
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.

2 participants