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

FIX Fix devtool error and remove comments in prod #54

Merged

Conversation

GuySartorelli
Copy link
Member

@GuySartorelli GuySartorelli commented Dec 12, 2022

Webpack complains and won't build in production mode because devtool is set to an empty string (which used to be acceptable). This has to be false now.

docs: https://webpack.js.org/configuration/devtool/

I've also re-removed comments, which were being included by default against my expectations.
Having no comments is the behaviour for the CMS 4 version, so this just fixes a regression where that wasn't happening.

NOTE

After this is merged, I need to create a new 2.0.0-alpha2 tag, re-release on npm, and run yarn install on all the related PRs so they're all using this new tag.

Parent issue

@GuySartorelli GuySartorelli changed the title FIX Devtool cannot be an empty string FIX Fix devtool error and remove comments in prod Dec 12, 2022
@emteknetnz emteknetnz merged commit 31d1dfe into silverstripe:2 Dec 12, 2022
@emteknetnz emteknetnz deleted the pulls/2/production-devtool-fix branch December 12, 2022 04:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants