Missing peer dependency in react-scripts

[Andarist]

You have peerDependenciesMeta there without actual peerDependencies - I believe this doesn't work as expected, peerDependenciesMeta requires peerDependencies to be there as it's merely additional information, not a replacement.

[ianschmitz]

Hmm it would seem you're correct: https://github.com/yarnpkg/yarn/pull/6671/files.

Adding under peerDependencies will start throwing warnings for folks that don't have npm@6.11.0 or later.

We may want to do this as part of our v4 release. I'll mark as such and see what the other maintainers think.

[Andarist]

From what I understand CRA+TS would be broken for strict package managers like pnpm or yarn in pnp mode without this peer dependency specified. I was facing similar problem lately for the other package and we decided to just go with specifying peer dep and marking it as optional. We can always recommend using new version of npm and on older versions it will merely cause a warning - things will still be very much functional, so... 🤷‍♂

[ianschmitz]

Good point. Oct 21 appears to have been the first Node release that included npm >= 6.11. I'm okay with the warning to support the other package managers.