remove react-scripts

[johnpmitsch]

Dependabot has an alert for nth-check, which is required by react-scripts

yarn list nth-check
yarn list v1.22.18
warning Filtering by arguments is deprecated. Please use the pattern option instead.
├─ nth-check@2.1.1
└─ react-scripts@5.0.1
   └─ nth-check@1.0.2

Unfortunately, react-scripts doesn't have a later production version (see open issue complaining about this)

I just removed it since it seems like we only use it in the ts declaration file that is included by default for create-react-app. There's an issue of people complaining about this too 😆 I don't think we need it for any of our usage and could workaround anything that we would need the types for, so I removed it for now

[johnpmitsch]

nth-check is still there, but it's upgraded to a sufficent version:

The earliest fixed version is 2.0.1.

yarn list nth-check
yarn list v1.22.18
warning Filtering by arguments is deprecated. Please use the pattern option instead.
└─ nth-check@2.1.1
✨  Done in 0.27s.
yarn why nth-check
yarn why v1.22.18
[1/4] 🤔  Why do we have the module "nth-check"...?
[2/4] 🚚  Initialising dependency graph...
[3/4] 🔍  Finding dependency...
[4/4] 🚡  Calculating file sizes...
=> Found "nth-check@2.1.1"
info Reasons this module exists
   - "@svgr#webpack#@svgr#plugin-svgo#svgo#css-select" depends on it
   - Hoisted from "@svgr#webpack#@svgr#plugin-svgo#svgo#css-select#nth-check"
info Disk size without dependencies: "120KB"
info Disk size with unique dependencies: "132KB"
info Disk size with transitive dependencies: "132KB"
info Number of shared dependencies: 1
✨  Done in 0.27s.

[coveralls]

coverage: 100.0%. remained the same when pulling 78f41e8 on nft_hooks_dependabot into ab54825 on main.