-
Notifications
You must be signed in to change notification settings - Fork 4
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
Use webpack.common.js
instead of webpack.dev.js
for deployments
#247
Comments
Just wondering - does this mean we don't need both and can consolidate into just one webpack config file? (aka webpack.config.js) (Tangential idea: switch to vite) |
I think we do still need the @ switching to vite: hell yeah 😎 |
Actually yes, I think you're right. We can probably move that setting to the default webpack server, and it would just not do anything for all the built versions of w3cv. |
see #324 might resolve this...? |
Use Case
Currently, GH and S3 builds rely on
webpack.dev.js
(instead ofwebpack.common.js
). Switch them to usewebpack.common.js
instead to avoid unintended consequences if editing the dev server?Solution
Please describe your ideal solution
Alternatives
Please describe any alternatives you've considered, even if you've dismissed them
The text was updated successfully, but these errors were encountered: