Browse Source

Take stderr into account when deciding on success

master
chimo 6 years ago
parent
commit
fe9fc81a0a
2 changed files with 3 additions and 3 deletions
  1. +2
    -2
      node/ScssDomain.js
  2. +1
    -1
      package.json

+ 2
- 2
node/ScssDomain.js View File

@ -20,11 +20,11 @@
// Call external scss-lint command
// Exit codes: https://github.com/brigade/scss-lint/blob/14ea8408dbdd867f33482825d6ccb80f841fbe19/lib/scss_lint/cli.rb#L11
exec(cmd, { maxBuffer: maxExecBuffer }, function (error, stdout/*, stderr*/) {
exec(cmd, { maxBuffer: maxExecBuffer }, function (error, stdout, stderr) {
var message;
// These error codes are okay
if (error === null || error.code === 1 || error.code === 2) {
if (!stderr && (error === null || error.code === 1 || error.code === 2)) {
callback(false, stdout);
} else {
switch(error.code) {


+ 1
- 1
package.json View File

@ -3,7 +3,7 @@
"title": "SCSS Lint",
"description": "Adds SCSS lint support to Brackets.",
"homepage": "https://github.com/chimo/brackets-scss-lint",
"version": "0.3.0",
"version": "0.3.0dev1",
"keywords": ["lint", "linting", "linter", "scss"],
"categories": "linting",
"author": "chimo <chimo@chromic.org>",


Loading…
Cancel
Save