Remove old rnpm for react-native 0.60

[carlosroberto555]

I have folow this migration guide for fix the rn60 warning
I don't have time to test this change yet, but I accept any feedbacks
#359

[brodycj]

Looks nice, I will try to test it later today.

I would favor that we keep the existing rnpm entry for now, so that people can continue using React Native pre-0.60 for a while.

[carlosroberto555]

I found this part of migration guide and fixed the react-native.config.js

[andpor]

Please reformat using 2 spaces for tabs.

[carlosroberto555]

How we can do android link with react-native link?
Anybody has the answer?

[carlosroberto555]

Looks nice, I will try to test it later today.

I would favor that we keep the existing rnpm entry for now, so that people can continue using React Native pre-0.60 for a while.

@brodybits I had read on migration guide, and see this info:

"rnpm" is deprecated and support for it will be removed in next major version of the CLI.

It's not because react-native version, but the cli version, since react-native ^0.59 they are independent.
New versions of react-native-cli show this warning and version 2.0.0 is not compatible.
But we can create a support branch for old versions of react-native that uses rnpm yet.
You can see more info here.

[carlosroberto555]

@andpor any more changes request?

[crobinson42]

+1 can we help move this along in any way?

[carlosroberto555]

I'll create a another npm package for solve this problem.
This PR has more a month, and not had activity.

[crobinson42]

I don’t think creating another npm package solves any problems. That just spreads the community out more. This package has weekly downloads and many contributors in just think it would be helpful if the owner or maintainer was a little more active or delegated permissions for keeping PRs like this moving along for the library.

…

Sent from my iPhone

On Aug 21, 2019, at 9:40 AM, Carlos Roberto ***@***.***> wrote: I'll create a another npm package for solve this problem. This PR has more a month, and not had activity. — You are receiving this because you commented. Reply to this email directly, view it on GitHub, or mute the thread.

[carlosroberto555]

Yes not solve, it's only a idea
This workaround install from my fork directly.

yarn add carlosroberto555/react-native-sqlite-storage

I used on my personal project, and autolink works like a charm.

[andpor]

The PR is merged and released under 4.0.0 - sorry for delay, quite busy lately. 4.0.0 has been published to npm.

[mikehardy]

The burden on library maintainers for react-native 0.60 (both in their paying project work, and whatever libraries they maintain) is pretty serious, and it's vacation season - glad to see this merged but everyone be as helpful as you can in the libraries you use, pretty much like submitting the PR here (❤️) and being patient. Cheers