grunt-scss-lint icon indicating copy to clipboard operation
grunt-scss-lint copied to clipboard

Warning: Task "scsslint:allFiles" failed. Use --force to continue.

Open douglasdeodato opened this issue 9 years ago • 10 comments
trafficstars

I am getting this warning above this is my gruntfile.js the task is working perfect no problem at all.

`    scsslint: {
        allFiles: [
          'scss/_base.scss',
        ],
        options: {
          config: 'scss/.scss-lint.yml',
          reporterOutput: 'scss/scss-lint-report.xml',
          colorizeOutput: true,
          compact:false
        }
      },`

error:


Running scss-lint on allFiles
scss/core/_base.scss:1 [W] Comment: Use `//` comments everywhere
scss/core/_base.scss:8 [W] TrailingWhitespace: Line contains trailing whitespace
scss/core/_base.scss:13 [W] PropertySortOrder: Properties should be ordered background-color, font, height, min-height, width
scss/core/_base.scss:16 [W] NameFormat: Name of variable `darkWhite` should be written in all lowercase letters with hyphens instead of underscores
scss/core/_base.scss:23 [W] PropertySortOrder: Properties should be ordered list-style, margin, margin-bottom, padding
scss/core/_base.scss:40 [W] NameFormat: Name of mixin `remSize` should be written in all lowercase letters with hyphens instead of underscores
Results have been written to: scss/scss-lint-report.xml
Warning: Task "scsslint:allFiles" failed. Use --force to continue.

Aborted due to warnings.

scss-lint-report.xml is created in my folder showing the erros, how can i remove this message : Warning: Task "scsslint:allFiles" failed. Use --force to continue.

when I fix my file everything runs ok.

Running scss-lint on allFiles

>> 1 file is lint free
Results have been written to: scss/scss-lint-report.xml

Done, without errors.

douglasdeodato avatar May 06 '16 13:05 douglasdeodato

I agree on this one. The grunt runner should take the severity from scss-lint and determine whether it was successful (albeit with warnings) or unsuccessful due to errors.

jshthornton avatar Jun 08 '16 04:06 jshthornton

make it a bit confuse,special for people with no knowledge.

instead: Warning: Task "scsslint:allFiles" failed. Use --force to continue.

Aborted due to warnings.

why not: scsslint Task finished, please check the errors above to have your file with a correct syntax.

or something like that no? make more sense then task failed --force to continue.

usually task failed --force to continue appears when the gruntfile.js has a syntax error .

anyway my opnion.

douglasdeodato avatar Jun 10 '16 07:06 douglasdeodato

PR?

ahmednuaman avatar Jul 05 '16 15:07 ahmednuaman

that means please repeat? :) +1

douglasdeodato avatar Jul 05 '16 16:07 douglasdeodato

oh Pull Request :) , I will try.

douglasdeodato avatar Jul 05 '16 20:07 douglasdeodato

👍

ahmednuaman avatar Jul 06 '16 18:07 ahmednuaman

I think PR #138 will help with this.

nick11703 avatar Jul 12 '16 15:07 nick11703

thank you @nick11703 @ahmednuaman ? :)

douglasdeodato avatar Jul 12 '16 15:07 douglasdeodato

@nick11703 and @ahmednuaman any news about this pull request? thank you.

douglasdeodato avatar Aug 29 '16 10:08 douglasdeodato

We need the project owner @ahmednuaman to approve and merge PR #138

nick11703 avatar Aug 29 '16 13:08 nick11703