schorfES / grunt-lintspaces

A Grunt task for checking spaces in files.
https://npmjs.com/package/grunt-lintspaces
MIT License
31 stars 9 forks source link

problem with spaces #18

Closed LynxDelli closed 10 years ago

LynxDelli commented 10 years ago

I use options in a grunt task:

less: { src: [ 'css/less/*/' ], options: { indentation: 'spaces', spaces: 2 } }

If I have 3 spaces it validate that indentation spaces are wrong, If I have 4 it pass and it shouldn't

schorfES commented 10 years ago

Hi, the lintspaces validator does not interpret your code. The validation of a line's indentation is only based on multiples of your spaces option.

Yesterday I released a new version of lintspaces and some minutes ago I also released a new version of grunt-lintspaces. Those new versions contain a guess indentation option. Maybe this option helps...

LynxDelli commented 10 years ago

Yes it's much better. I'm using it for less/css validation a sample where new guess is failing

Wrong that pass is: div { height: 0; }

Correct is: div { height: 0; }

Anyway thanks a lot, it is much better than it used to be :)

On Fri, Jul 11, 2014 at 9:57 PM, Norman Rusch notifications@github.com wrote:

Closed #18 https://github.com/schorfES/grunt-lintspaces/issues/18.

— Reply to this email directly or view it on GitHub https://github.com/schorfES/grunt-lintspaces/issues/18#event-140772337.

schorfES commented 10 years ago

Okay! But as I have written above lintspaces does not interpret your code. In this case additionally use a css validator.