-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
[frameworks] Add autodetection for RedwoodJS #4915
Conversation
Co-authored-by: Steven <steven@ceriously.com>
"value": "yarn rw db up --no-db-client --auto-approve && yarn rw build" | ||
}, | ||
"devCommand": { | ||
"value": "yarn rw dev" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Don't we need to add --port $PORT
or something to pass the port?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
They don't support that flag unfortunately, they use a redwood.toml file to set the port.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
My understanding is that this dev command won't be used, since the @vercel/redwood
Runtime is special-cased. Is that correct?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It uses config.devCommand
here:
vercel/packages/redwood/src/index.ts
Line 92 in 041e8cc
devCommand = 'yarn rw dev', |
So it could be overridden if the user changes their project settings.
@vercel/redwood
#4676