Add an error attribute to not fail and only show warnings

Author: codingjoe

.relint.yml

@@ -4,3 +4,4 @@
   filename:
     - "*.py"
     - "*.js"
+  error: false

README.rst

@@ -25,6 +25,7 @@ You can write your own regular rules in a YAML file, like so:
       filename:
         - "*.py"
         - "*.js"
+      error: false
 
 The ``name`` attribute is the name of your linter, the ``pattern`` can be
 any regular expression. The linter does lint entire files, therefore your
@@ -33,6 +34,9 @@ expressions can match multiple lines and include newlines.
 You can narrow down the file types your linter should be working with, by
 providing the optional ``filename`` attribute. The default is ``*``.
 
+The optional `error` attribute allows you to only show a warning but not exit
+with a bad (non-zero) exit code. The default is `true`.
+
 The following command will lint all files in the current directory:
 
 .. code-block:: bash

relint.py

@@ -28,7 +28,7 @@ def parse_args():
     return parser.parse_args()
 
 
-Test = namedtuple('Test', ('name', 'pattern', 'hint', 'filename'))
+Test = namedtuple('Test', ('name', 'pattern', 'hint', 'filename', 'error'))
 
 
 def load_config(path):
@@ -42,6 +42,7 @@ def load_config(path):
                 pattern=re.compile(test['pattern']),
                 hint=test.get('hint'),
                 filename=filename,
+                error=test.get('error', True)
             )
 
 
@@ -79,7 +80,7 @@ def main():
     exit_code = 0
 
     for filename, test, match in matches:
-        exit_code = 1
+        exit_code = test.error if exit_code == 0 else exit_code
         if filename != _filename:
             _filename = filename
             lines = match.string.splitlines()