Skip to content

[feat] add expo plugin config options#172

Merged
Kudo merged 1 commit into
Kudo:mainfrom
kmartinezmedia:expo-plugin-options
Mar 27, 2023
Merged

[feat] add expo plugin config options#172
Kudo merged 1 commit into
Kudo:mainfrom
kmartinezmedia:expo-plugin-options

Conversation

@kmartinezmedia

Copy link
Copy Markdown
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?

@Kudo Kudo left a comment

Copy link
Copy Markdown
Owner

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