adammcarth / react-native-segmented-picker

Selection picker wheel with multi-column support and optional native dependencies.
MIT License
96 stars 31 forks source link

Ignored b/c invalid config #28

Open Chrisissorry opened 4 years ago

Chrisissorry commented 4 years ago
yarn run v1.22.5
$ react-native run-ios
warn Package react-native-segmented-picker has been ignored because it contains invalid configuration. Reason: Option dependency.platforms.ios must be a object, instead got object

Using RN 0.63.2

adammcarth commented 4 years ago

Thanks for the report @Chrisissorry :)

Oh yeah, I'm aware of this one. It's a strange error message that I believe first popped up after installing react-native-swift. Are you using the native addon for iOS?

You can safely ignore the warning message for now, but I'll do my best to figure out how to fix it ASAP - because it's definitely annoying seeing it every time you do yarn install etc.

Chrisissorry commented 4 years ago

Yes I am using the native addon, but I saw this warning also when using the non-native version.

Actually I opened this issue because it would not render, but that could've been an issue on my side. I am pretty new to RN 😌

erodrig commented 4 years ago

It seems this is failing given the config file has null on it

/**

module.exports = { dependency: { platforms: { android: null, ios: null, }, }, };

adammcarth commented 3 years ago

Oh, good find @erodrig! I always assumed that error was being generated because of an invalid Podfile, but you're right - it looks like the react-native.config.js file is the culprit here.

I'm really not sure what to do though. The official React Native CLI documentation explicitly says to set it as null to disable Auto Linking?

So I'm actually thinking this is a bug with the react-native-swift package that we might need to raise with them. null is supposed to be a valid value, so they shouldn't warn us against using it.

adammcarth commented 3 years ago

^ Whoops. Turns out it's actually an issue with the React Native CLI. I've raised a PR.