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

[feat] add expo plugin config options #172

Merged
merged 1 commit into from
Mar 27, 2023

Conversation

kmartinezmedia
Copy link
Contributor

I'd like to be able to configure if v8 is applied to iOS build. I would expect to be able to configure like this in my expo app.config.js.

plugins: [
  ['react-native-v8', { android: true, ios: false }] 
]

Are you open to contributions?

Copy link
Owner

@Kudo Kudo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

sorry being late to take this. the change is awesome. thanks for having this great fix 👏

@Kudo Kudo merged commit 9509f5e into Kudo:main Mar 27, 2023
leotm added a commit to MetaMask/metamask-mobile that referenced this pull request Apr 18, 2023
patch-package react-native-v8 creates 271 diffs after no changes, so this excludes those diffs

Fix pod install:

[!] CocoaPods could not find compatible versions for pod "RNV8":
  In Podfile:
    RNV8 (from `../node_modules/react-native-v8`)

Specs satisfying the `RNV8 (from `../node_modules/react-native-v8`)` dependency were found, but they required a higher minimum deployment target.

Better solution would be a PR like:
Kudo/react-native-v8#172
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