fix: remove react and react-dom from dependencies

simplystuart commented 6 months ago

Description

Please replace this line with instructions on how to test your changes

For us to review and ship your PR efficiently, please perform the following steps:

[x] Open a bug/issue before writing your code 🧑‍💻. This ensures we can discuss the changes and get feedback from everyone that should be involved. If you`re fixing a typo or something that`s on fire 🔥 (e.g. incident related), you can skip this step.
[-] Read the contribution guidelines 📖. This ensures your code follows our style guide and passes our tests.
[-] Update or add tests (if any source code was changed or added) 🧪
[-] Update or add documentation (if features were changed or added) 📝
[-] Make sure the status checks below are successful ✅

puppy-eyes

netlify[bot] commented 6 months ago

Name	Link
Latest commit	f2eb735db3514ba1669c0f2426c2ed0176ca5113
Latest deploy log	https://app.netlify.com/sites/remix-edge-on/deploys/66293580ef04d900072cc9e4
Deploy Preview	https://deploy-preview-145--remix-edge-on.netlify.app
Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

netlify[bot] commented 6 months ago

Name	Link
Latest commit	f2eb735db3514ba1669c0f2426c2ed0176ca5113

simplystuart commented 6 months ago

FYI react and react-dom aren't needed as direct dependencies as @remix-run/react includes both.

ascorbic commented 6 months ago

Thanks, this makes sense