Add silent flag to avoid default node error message on --fix

[kristianmandrup]

Not clear how to run xo --fix in silent mode to avoid the exit code 1 error displayed.
Please advice. Thanks :)

18:19 $ npm run fix

> nodelib-2017@1.0.0 fix /Users/kristianmandrup/repos/node-libs/nodelib-2017
> xo --fix --quiet


  src/a.test.js:2:8
  ✖  2:8  goofy is defined but never used.  no-unused-vars

  1 error

npm ERR! Darwin 16.1.0
npm ERR! argv "/usr/local/bin/node" "/usr/local/bin/npm" "run" "fix"
npm ERR! node v6.9.1
npm ERR! npm  v4.0.1
npm ERR! code ELIFECYCLE
npm ERR! nodelib-2017@1.0.0 fix: `xo --fix --quiet`
npm ERR! Exit status 1
npm ERR! 
npm ERR! Failed at the nodelib-2017@1.0.0 fix script 'xo --fix --quiet'.

[kristianmandrup]

Discovered a solution xo --fix -- -s :)

[kristianmandrup]

Would be nice if there was a way to display the errors, but still avoid the final node fail error.
When I run xo with --fix and "semicolon": false" my code still has semicolons.

[kristianmandrup]

My xo package.json settings:

  "xo": {
    "esnext": true,
    "parser": "babel-eslint",
    "semicolon": false,
    "reporter": "compact",
    "plugins": [
      "filenames"
    ],
    "rules": {
      "filenames/match-regex": [
        2,
        "^[a-z-\\.]+$",
        true
      ]
    }
  },

[sindresorhus]

Not clear how to run xo --fix in silent mode to avoid the exit code 1 error displayed.

Why do you want to avoid the exit code 1? --fix reports the remaining errors so you know what you need to manually fix.

When I run xo with --fix and "semicolon": false" my code still has semicolons.

That's weird. The semi rule has auto-fixing: http://eslint.org/docs/rules/semi

"filenames/match-regex": [

Btw, XO already includes a filename rule that does this:
https://github.com/sindresorhus/eslint-plugin-unicorn/blob/master/docs/rules/filename-case.md

[friday]

You can add || true after the xo command. It would have the same effect as the -s flag

  "scripts": {
    "fix": "xo --fix || true"
  }

[kristianmandrup]

Thanks a lot guys. I will try.

[fregante]

Closing as resolved. || true is the native way to do this and it's shorter than any flag ever could.

Eslint doesn't offer such a flag either https://eslint.org/docs/latest/user-guide/command-line-interface