-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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
chore(templates): integrate v2_dev #6650
Conversation
|
bb0e8a5
to
512467d
Compare
this is a workaround while we figure out why arc not able to opt out of their built-in reloads
30dd4c5
to
2faa8c2
Compare
@@ -1,10 +1,12 @@ | |||
/** @type {import('@remix-run/dev').AppConfig} */ | |||
module.exports = { | |||
devServerBroadcastDelay: 1000, | |||
export default { | |||
ignoredRouteFiles: ["**/.*"], | |||
server: "./server.ts", | |||
serverConditions: ["worker"], |
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.
Same comment I left on the last one—IMHO this should be ["workerd", "worker", "browser"]
@pcattori Should we get these changes in the stacks as well? |
Supercedes #6229
Deno template is not changed as their is an existing bug with that template. Will fix and update Deno template as a follow-on.