Skip to content
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

Open
ShrimpCryptid opened this issue May 21, 2024 · 4 comments
Open

Comments

@ShrimpCryptid
Copy link
Contributor

Use Case

Currently, GH and S3 builds rely on webpack.dev.js (instead of webpack.common.js). Switch them to use webpack.common.js instead to avoid unintended consequences if editing the dev server?

    "start": "webpack serve --config webpack.dev.js  --env env=dev",
    "gh-build-nightly": "webpack --config webpack.dev.js --env env=production basename=/website-3d-cell-viewer",
    "gh-build-release": "webpack --config webpack.dev.js --env env=production basename=/website-3d-cell-viewer-release",
    "s3-build": "webpack --config webpack.dev.js --env env=production",

Solution

Please describe your ideal solution

Alternatives

Please describe any alternatives you've considered, even if you've dismissed them

@toloudis
Copy link
Contributor

toloudis commented May 23, 2024

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)

@ShrimpCryptid
Copy link
Contributor Author

ShrimpCryptid commented May 23, 2024

I think we do still need the webpack.dev.js config to enable route management when running the local dev server? But I think that's the only case where we'd need it.

@ switching to vite: hell yeah 😎

@ShrimpCryptid
Copy link
Contributor Author

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.

@toloudis
Copy link
Contributor

see #324 might resolve this...?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants