shopify_python icon indicating copy to clipboard operation
shopify_python copied to clipboard

How should linter extensions handle exceptions?

Open cfournie opened this issue 8 years ago • 0 comments

Disabling a linter message doesn't disable the check for such a linter rule. If such a linter rule check throws an exception because it wasn't well tested, then the entire linter is unusable. How should we deal with this?

There are two main proposals:

  • Allow exceptions to be raised and be fixed in a new release (and recommend not upgrading to a faulty version); or
  • Surround everything within a try-catch and report exceptions as false-positives with a custom message

cfournie avatar Jul 17 '17 13:07 cfournie