ATM syntax errors are shown as Tokenization error but still reported as warning.
This should be changed to split warnings and errors and always return an exit code > 0 if there are errors. The --exit-code option could/should then be renamed to e.g. --fail-on-warning to make stricter checks available as opt-in.
The benefit of this would be that real errors let the command fail and warnings can be ignored which is useful in CI environments.
ATM syntax errors are shown as
Tokenization errorbut still reported as warning.This should be changed to split warnings and errors and always return an exit code > 0 if there are errors. The
--exit-codeoption could/should then be renamed to e.g.--fail-on-warningto make stricter checks available as opt-in.The benefit of this would be that real errors let the command fail and warnings can be ignored which is useful in CI environments.