Skip to content

"rnpm" deprecation warnings encourage library to not be backwards compatible #346

@thymikee

Description

@thymikee

Description

If a package defines both "rnpm" key in package.json and a react-native.config.js file, we still produce warnings:

config =
readLegacyDependencyConfigFromDisk(root) ||
readDependencyConfigFromDisk(root);

I don't think this is what we want, because if a package wants to get rid of the warning, they'll need to remove "rnpm" entry and in turn bump major version, becoming only compatible with RN 0.60+.

We need to swap the order of loading deps and handle it accordingly.

Metadata

Metadata

Assignees

Labels

bugSomething isn't working

Type

No type

Projects

No projects

Relationships

None yet

Development

No branches or pull requests

Issue actions