-
-
Notifications
You must be signed in to change notification settings - Fork 10.4k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
#12578 breaks command react-router dev --any-param
#12606
Comments
#12578 is indeed the issue, you can fixed it locally by patching the @react-router/dev package with the following changes. diff --git a/bin.js b/bin.js
index 88866010646f42058bd8daf29896fb6be55d4eb7..0373bf96f7c42f5460dc2e9fea394e9a7e947e06 100755
--- a/bin.js
+++ b/bin.js
@@ -5,7 +5,7 @@ let arg = require("arg");
// default `NODE_ENV` so React loads the proper version in it's CJS entry script.
// We have to do this before importing `run.ts` since that is what imports
// `react` (indirectly via `react-router`)
-let args = arg({}, { argv: process.argv.slice(2) });
+let args = arg({}, { argv: process.argv.slice(2), stopAtPositional: true });
if (args._[0] === "dev") {
process.env.NODE_ENV = process.env.NODE_ENV ?? "development";
} else { Adding |
P1. It's not just |
same, |
I've released |
I believe this issue should remain open. I'm still getting the following error on windows 11
extra info:
|
This specific issue with the |
I'm using React Router as a...
framework
Reproduction
"dev": "react-router dev --host",
pnpm run dev
System Info
Used Package Manager
pnpm
Expected Behavior
Just works
Actual Behavior
The text was updated successfully, but these errors were encountered: