-
Notifications
You must be signed in to change notification settings - Fork 29
Add config sample with mini-css #18
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
Comments
It should be configured like that:
It works fine in our config and I wonder why it does not work for you folks. Would you be able to provide the full list of options for all 3 loaders? That might help finding the issue |
Now my config looks like this {
test: /\.styl$/,
use: [
{
loader: '@teamsupercell/typings-for-css-modules-loader',
options: {
formatter: 'none'
}
},
{
loader: MiniCssExtractPlugin.loader,
options: {
esModule: true,
},
},
{
loader: 'css-loader',
options: {
esModule: true,
localsConvention: 'camelCase',
modules: {
localIdentName: '[folder]-[local]__[hash:base64:5]'
}
}
}
'stylus-loader',
]
},
{
test: /\.css$/,
use: [
{
loader: '@teamsupercell/typings-for-css-modules-loader',
options: {
formatter: 'none'
}
},
{
loader: MiniCssExtractPlugin.loader,
options: {
esModule: true,
},
},
{
loader: 'css-loader',
options: {
esModule: true
}
}
]
}, |
Ah, thanks for that @inoyakaigor. I think it's because of |
@inoyakaigor I can't reproduce the error you're getting. Tried both - running |
@DanNSam hmm… it is strange. Anyway I will try to reproduce but a little bit later. Maybe this week. |
@DanNSam I did it. Ran under Node 12.16.1 |
I had moved to this package from original Jimdo/typings-for-css-modules-loader and got an error
My loaders config was
I spend a day to find a solution and the trick was in the order of loaders. typings-for-css-modules-loader should be last in order:
Please add in readme this solution.
The text was updated successfully, but these errors were encountered: