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

fix(react-native): default template fails when envs are not set #28931

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

maciekstosio
Copy link

Current Behavior

When envs are not set metro fails in .babelrc.js on process.env.NX_TASK_TARGET_TARGET.includes('storybook')

Expected Behavior

When envs are not set don't fail on the if and just precede.

@maciekstosio maciekstosio requested a review from a team as a code owner November 14, 2024 10:04
Copy link

vercel bot commented Nov 14, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Skipped Deployment
Name Status Preview Updated (UTC)
nx-dev ⬜️ Ignored (Inspect) Visit Preview Nov 14, 2024 10:08am

maciekstosio added a commit to software-mansion/radon-ide that referenced this pull request Nov 19, 2024
Fixes #683

This PR adds nx monorepo setup handling by: 
- adding recognizing nx monorepo root 
- installing dependencies in the workspace root 
- adding nx-monorepo test-app 

**Remarks**:
- I created PR (nrwl/nx#28931) to nx so their
template doesn't fail in Radon IDE, we don't set their envs, my
understanding is that those are used internally by their CLI tool, and I
don't think want to implement that
- ~~iOS fails on `bundle install && bundle exec pod install` with
`EACCES: Permission denied` I'm investigating if it's related to IDE or
something else (i.e. my setup), manual `pod install` solves the issue~~
seems like problem with my setup (those files were owned by root,
reinstalling ruby using rbenv solved the problem)

### How Has This Been Tested: 
**Scenario:** Open iOS and Android, check if it builds.
**Tested on:** 
- workspace-example
- expo-52
- nx-monorepo
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