-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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
feat(react-native): add convert-to-inferred generator for Expo and React Native #27326
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
☁️ Nx Cloud ReportCI is running/has finished running commands for commit 2b6f5a0. As they complete they will appear below. Click to see the status, the terminal output, and the build insights. 📂 See all runs for this CI Pipeline Execution ✅ Successfully ran 5 targets
Sent with 💌 from NxCloud. |
c521778
to
1d43fa4
Compare
packages/expo/src/generators/convert-to-inferred/lib/post-target-transformer.ts
Outdated
Show resolved
Hide resolved
packages/expo/src/generators/convert-to-inferred/lib/process-prebuild-options.ts
Outdated
Show resolved
Hide resolved
packages/expo/src/generators/convert-to-inferred/lib/process-install-options.ts
Outdated
Show resolved
Hide resolved
packages/expo/src/generators/convert-to-inferred/lib/process-build-options.ts
Outdated
Show resolved
Hide resolved
1d43fa4
to
a461c1b
Compare
a461c1b
to
089497a
Compare
ba8095a
to
a661171
Compare
a661171
to
0015cc4
Compare
f032959
to
d875fc7
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
1 comment, LGTM!
* Remove pod-install from dependsOn for all targets, it does pod-install when creating the app | ||
*/ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Has this been added?
workspaceRoot, | ||
} from '@nx/devkit'; | ||
|
||
import { runCliUpgrade } from '../../executors/upgrade/upgrade.impl'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this being used?
e141af0
to
894155d
Compare
894155d
to
8aa769d
Compare
8aa769d
to
1dd3c57
Compare
1dd3c57
to
2b6f5a0
Compare
This pull request has already been merged/closed. If you experience issues related to these changes, please open a new issue referencing this pull request. |
This PR adds
convert-to-inferred
generators to convert React Native an Expo apps using executors to use the corresponding inference plugins.Also:
@nx/react-native/plugin
so it is correctly set asupgradeTargetName
notupgradeTargetname