Merge pull request #5 from richardxia/add_coffeelint_checker

Add coffeelint checker

codequality/checkers.py

@@ -1,6 +1,8 @@
 from subprocess import PIPE
 from subprocess import Popen
+import csv
 import re
+import StringIO
 
 
 checkers = {}
@@ -167,3 +169,30 @@ def get_version(cls):
             return ''
         else:
             return out.splitlines()[0].strip()
+
+
+@register(filetypes=('coffee',))
+class CoffeeLintChecker(Checker):
+    """
+    Checker integration with the coffeelint tool.
+    """
+    tool = 'coffeelint'
+
+    def check(self, paths):
+        if not paths:
+            return ()
+
+        cmd_pieces = [self.tool, '--csv']  # Use CSV output
+        cmd_pieces.extend(paths)
+        process = Popen(cmd_pieces, stdout=PIPE, stderr=PIPE)
+        out, err = process.communicate()
+        output_rows = csv.DictReader(StringIO.StringIO(out))
+        return [
+            {
+                'filename': row['path'],
+                'lineno': int(row['lineNumber']),
+                'colno': '',
+                'msg': row['message'],
+            }
+            for row in output_rows
+        ]

codequality/main.py

@@ -141,7 +141,7 @@ def _relevant_checkers(self, path):
 
         TODO: currently this is based off the file extension.  We would like to
         honor magic bits as well, so that python binaries, shell scripts, etc
-        but we're not guarunteed that `path` currently exists on the filesystem
+        but we're not guaranteed that `path` currently exists on the filesystem
         -- e.g. when version control for historical revs is used.
         """
         _, ext = os.path.splitext(path)
@@ -153,10 +153,10 @@ def _resolve_paths(self, *paths):
         Resolve paths into a set of filenames (no directories) to check.
 
         External tools will handle directories as arguments differently, so for
-        consistancy we just want to pass them filenames.
+        consistency we just want to pass them filenames.
 
         This method will recursively walk all directories and filter out
-        any paths that mach self.options.ignores.
+        any paths that match self.options.ignores.
         """
         result = set()
         for path in paths: