chore: move react-native-get-random-values to peerDependencies

[ahmadi-akbar]

fix #3055

[lfdt-bot]

🎉 Snyk checks have passed. No issues have been found so far.

✅ security/snyk check is complete. No issues have been found. (View Details)

[ivaylonikolov7]

Hello @ahmadi-akbar . Thank you for contribution 🙌

Just a couple of questions on the PR. How would that fix the issue for you? As far as I understood this will still install the react-native-get-random-values unless there is some network issues? This will install on all platforms regardless of optional:true flag. My understanding of the optional:true flag is that it wouldn't break the install operation if it fails on specific platforms or isn't able to download the package.

[ahmadi-akbar]

please read this about peer dependencies
https://nodejs.org/en/blog/npm/peer-dependencies
https://docs.npmjs.com/cli/v9/configuring-npm/package-json#peerdependenciesmeta

[ivaylonikolov7]

Hm weird. Sorry, something happened with Github when I merged a dependabot PR, the pull bot force pushed on your PR and when I merged it said I merged your changes.

I read the peerDependenciesMeta field. I will check if it makes any breaking changes on our already existing users. I will recreate the PR so you don't waste time. But you can recreate it. Sorry about the bot force pushing.

[ivaylonikolov7]

Recreated your PR here - #3203 @ahmadi-akbar. We can continue the discussion there.