Add 'prop-types' as a dependency (or remove it)

[templarian]

Add 'prop-types' as a dependency (or remove it)

[Templarian]

Interesting, marked this a bug. Found the docs, yeah will need to fix this:

https://github.com/facebook/prop-types#how-to-depend-on-this-package

[lafiosca]

I ran into this problem as well because I'm using Yarn PnP for my project. In that case, adding prop-types to your app's top-level dependencies is not sufficient because Yarn PnP is very strict about packages declaring their dependencies. Additionally, @mdi/react also depends on react without declaring it.

The workaround for me was to add this to the packageExtensions section of my root .yarnrc.yml:

  '@mdi/react@*':
    dependencies:
      prop-types: '^15.5.7'
      react: '*'

[willin]

then how to resolve this?

[some-user123]

Stumbled across this today as the fix is not published on npm. Any plans to release the fix?