future-architect/cheetah-grid

Move react and react-dom to peerDependencies from dependencies

Closed this issue ยท 4 comments

Recently we've decided to use React version of cheetah-grid as we previously used a vanilla version, it works perfectly for us ๐Ÿ‘

We are using react 18, and noticed in pnpm-lock.yaml appeared react-17.

image

I've looked at package.json of cheetah-grid-react and noticied that react is specified as dependency there

I think we should specify react as peerDependency instead.
Moreover, we should allow react version >= 17.

@shibukawa Is it possible to change dependencies?

@ota-meshi do you want me to send a pull request?

Yeah! Pull requests are welcome. Thank you.

@ota-meshi I've created pull request #420