You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
AFAICT react, react-dom and route-manifest are only needed if one is using react.
Thus, wouldn't it make more sense to mark these peerDependencies as optional?
We would only need to update README.md and the site instructions, apart from package.json. Not sure if it'll be worth adding a try/catch in the react code, if we update the instructions.
AFAICT react, react-dom and route-manifest are only needed if one is using react.
Thus, wouldn't it make more sense to mark these peerDependencies as optional?
We would only need to update README.md and the site instructions, apart from package.json. Not sure if it'll be worth adding a try/catch in the react code, if we update the instructions.
It'll be a breaking change, though.
/CC @addyosmani
The text was updated successfully, but these errors were encountered: