-
Notifications
You must be signed in to change notification settings - Fork 116
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
gulp-less print out nothing error in case extend a non-existing mixin #206
Comments
Does this happen with the less command line as well? |
@contra , for the lessc itself, it really dump out an warning when extending a non-existing class. |
@contra , i have run in the linux command line with lessc --lint, it really printed out the missed extended class as follows(.clearfix can not be found when extend it!): vagrant@homestead:~/nodewrapper$ lessc --lint /home/vagrant/Code/kidsit/resources/assets/less/style.less
extend ' .clearfix' has no matches
extend ' .clearfix' has no matches
extend ' .clearfix' has no matches |
This seems like an issue where less is not reporting errors back via the JS API. You should open an issue with less so they can add this in. |
ok, i have opened an issue on less repo. Hope they can have a investigation. But currently, they have not admit, it is less itself's issue. |
@contra , please see reply from less guys, it seems that the less compiler itself has exported the warnings to tooling. So, i think, if gulp-less accept some configuration parameter, this can work. |
@cnweibo Do you want to send a PR to add this? We don't use the less module directly, we use accord which doesn't give us access to this stuff. It will probably need to be added to accord then we can add a warning event from this plugin |
@contra , sorry, i do not know how to let this issue fix by accord. Thanks~! |
Hi, I have posted a thread in stackoverflow about gulp-less. I want to know how to deliver error information when an error happens during extend a non-existing mixing/class.
http://stackoverflow.com/questions/34107876/is-there-any-way-to-let-gulp-less-print-error-in-case-extend-a-non-existing-mixi
Can you have a look and give me clue. Following is copied from stackoverflow:
In my less build process, i use the gulp-less. In the build process, i use gulp-plumber to print out error information during that build. It works well for the less syntax error, reference to non-exist variables. Unfortunately, the gulp-less build task print out nothing in case we extend a non-existing mixin/class. Does anyone know about how to enable such kind of "error" or even warning during the build? Following is the gulpfile:
gulp.task('less',function(){
var pagelessentry = config.pagelessentry;
log(pagelessentry);
return gulp
.src(pagelessentry)
.pipe(plumber({
errorhandler: errorhandler
}))
.pipe(debug({
title: 'LESS'
}))
.pipe(sourcemaps.init())
.pipe(less())
.pipe(uncss({
html: ['index.html', 'http://homestead.app'],
ignore: [/header-down/,/header-up/]
}))
.pipe(sourcemaps.write(projectrootdir+'public/preparebuild/assets/css/',{includeContent: true}))
.pipe(gulp.dest(projectrootdir+'public/preparebuild/assets/css/'));
});
Following is the "wrong" less code where i want gulp to print out some "error or warning" information:
.footer{
background-color: @primary-background;
&:extend(.NONEXISTING all);
}
In above code, when we extend .NOEXISTING mixin, nothing printed out.
It is quite annoying thing.
Please help me out
Thanks~!
The text was updated successfully, but these errors were encountered: