chore: remove unused npm-run-all dependency

[MichaelDeBoey]

This upgrades from npm-run-all to npm-run-all2. Primarily because the latter is actively maintained, has security updates and a reduced footprint in the dependency tree.

I'm sure people from the wider @e18e ecosystem cleanup (like @43081j) will be very happy to see these kind of changes as well

This package isn't used by any of the code, so it can be removed

[changeset-bot]

⚠️ No Changeset found

Latest commit: cc9c0cf

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

[outslept]

Hey, I found that the npm-run-all dependency might be unnecessary:

• Searching for "npm-run-all" returns 0 results in the codebase (except the actual package in package.json's dependency field)
• The only mentions of "run-s" and "run-p" are in an old changelog (v1.16.0), where they were already removed in a diff:

  {
    // ...
    "scripts": {
  -    "build": "run-s \"build:*\"",
  +    "build": "remix build",
  -    "build:css": "npm run generate:css -- --minify",
  -    "build:remix": "remix build",
  -    "dev": "run-p \"dev:*\"",
  +    "dev": "remix dev",
  -    "dev:css": "npm run generate:css -- --watch",
  -    "dev:remix": "remix dev",
  -    "generate:css": "npx tailwindcss -o ./app/tailwind.css",
      "start": "remix-serve build"
    }
    // ...
  }

Can we safely remove this dependency since it's not being used anywhere?

Source (L1921 & L1925):

react-router/packages/react-router-dev/CHANGELOG.md

Line 1921 in c06ec85

- "build": "run-s \"build:*\"",