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

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

Open
douglasdeodato opened this issue May 6, 2016 · 10 comments
Open

Comments

@douglasdeodato
Copy link

douglasdeodato commented May 6, 2016

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.
@jshthornton
Copy link

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.

@douglasdeodato
Copy link
Author

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.

@ahmednuaman
Copy link
Owner

PR?

@douglasdeodato
Copy link
Author

douglasdeodato commented Jul 5, 2016

that means please repeat? :) +1

@douglasdeodato
Copy link
Author

oh Pull Request :) , I will try.

@ahmednuaman
Copy link
Owner

👍

@nick11703
Copy link
Contributor

I think PR #138 will help with this.

@douglasdeodato
Copy link
Author

thank you @nick11703 @ahmednuaman ? :)

@douglasdeodato
Copy link
Author

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

@nick11703
Copy link
Contributor

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

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

No branches or pull requests

4 participants