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

NormalModuleReplacementPlugin has no effect with less-loader@4 #184

Closed
snadn opened this issue Mar 29, 2017 · 6 comments
Closed

NormalModuleReplacementPlugin has no effect with less-loader@4 #184

snadn opened this issue Mar 29, 2017 · 6 comments

Comments

@snadn
Copy link

snadn commented Mar 29, 2017

there is 3 files, a.less b.less c.less
a import b, and i use NormalModuleReplacementPlugin to replace b to c

when i use webpack@1 and less-loader@2, b is replaced by c.

but, upgrade to webpack@2 and less-loader@4, nothing happend

@jhnns
Copy link
Member

jhnns commented Mar 30, 2017

Hi, thanks for the bug report. It would be a big help if you could provide a minimal test repository that demonstrates the problem so I can check that out. Try to remove as much stuff as possible since it helps tracking down the problem.

@snadn
Copy link
Author

snadn commented Mar 31, 2017

@snadn
Copy link
Author

snadn commented May 11, 2017

Hey, what's the status on this?

@Biktop
Copy link

Biktop commented Jun 5, 2017

I confirm that NormalModuleReplacementPlugin doesn't work correctly in less-loader@4 but it works in v3.0.0.

@zombieJ
Copy link

zombieJ commented May 10, 2019

Currently Ant Design provides theme less file. User can use modifyVars to replace.
But we wish there have a better way to replace hole theme less instead of variable config.

Seems NormalModuleReplacementPlugin is a good solution. It should re-consider of this PR: #216

cc @michael-ciniawsky

@cap-Bernardito
Copy link
Member

NormalModuleReplacementPlugin is not intended for your purposes, and is no longer supported by less-loader. Use it instead of aliases.

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

Successfully merging a pull request may close this issue.

6 participants