-
Notifications
You must be signed in to change notification settings - Fork 501
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
fix: normalize #build
windows path for dev and prerender presets
#70
Conversation
'nitro:rollup:before' (nitro) { | ||
if (isWindows) { | ||
// Windows dynamic imports should be file:// url | ||
nitro.options.alias['#build'] = pathToFileURL(nitro.options.buildDir).href |
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.
Shared situation with both is externals: { trace: false }
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.
Hm. Perhaps we could just test for that instead in rollup config, or are you happy with the preset option?
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.
Improved in 1ec2ee4. Extending alias was too late here.
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.
LGTM. Seems a more clear workaround at least.
#build
windows path for dev and prerender presets
π Linked issue
resolves nuxt/framework#4232
β Type of change
π Description
The fix applied to dev preset also needs to be applied to prerender preset. It seemed easiest way was to extract from options and add to preset itself. There may be a better way to do this.
π Checklist