CLI exits with status 1 in npm commands about import-sort

[briced]

The exit code is the number of modified files, (https://github.com/renke/import-sort/blob/master/packages/import-sort-cli/src/index.ts)

Problem is, in Unix system, status code 0 is "Everything is ok", every other status is "There's an error"

#33 should to fix this, but it's a big breaking change, and I don't know if this will be merged.

[briced]

@herkulano
--status, as I understand it, returns the status code, without doing the rewriting (kind of dry-run), so it's coherent to make it return 0.
For --diff the work is done, so it's normal to return a status code "everything is ok".

If it fails, it will call bail() and return -1

[Frikki]

Here's how to suppress the status code when different than 0:

"fix:sort": "import-sort -o './' || true"

... or run with the silent flag:

$ npm run fix:sort --silent

[herkulano]

@briced for --diff and --status it makes senses to exit with 1 as it means they have found errors.

It's only when you're doing --write or --overwrite that it shouldn't exit with 1 because the files have been successfully changed, so they should be correct and there are no errors in them. The write command should only exit with 1 if there's an error writing the files.