Skip to content

Treat .css and .sass/.scss as side effectful - #5197

Merged
gaearon merged 1 commit into
masterfrom
side-css
Oct 1, 2018
Merged

Treat .css and .sass/.scss as side effectful#5197
gaearon merged 1 commit into
masterfrom
side-css

Conversation

@gaearon

@gaearongaearon commented Oct 1, 2018

Copy link
Copy Markdown
Contributor

Fixes#5140. Fixes#5188.

While unfortunate due to concerns in #5188 (comment), I don't feel comfortable with webpack's default behavior until webpack adds a warning or an error for this (webpack/webpack#6571 (comment)). When webpack does, we can reconsider and go with webpack's default behavior.

I checked that this change fixes the reproducing case in https://github.com/cdaringe/will-u-load-my-styl.

@gaearongaearon added this to the 2.0.0 milestone Oct 1, 2018
@gaearon
gaearon merged commit 4a96247 into masterOct 1, 2018
@Timer
Timer deleted the side-css branch October 1, 2018 14:08
zmitry pushed a commit to zmitry/create-react-app that referenced this pull request Oct 2, 2018
@locklockBot locked and limited conversation to collaborators Jan 19, 2019
Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Document CSS being removed or force CSS to be effectful react-scripts@next drops css import in production build

3 participants

@gaearon@simonjoom@facebook-github-bot