Merge pull request #4 from meengit/review-fixes

Review fixes

Author: meengit

linter.py

@@ -11,14 +11,17 @@
 #
 
 """This module exports the Codeclimate plugin class."""
+import logging
 import os
 from SublimeLinter.lint import Linter, PermanentError, util
 
 
+logger = logging.getLogger('SublimeLinter.plugin.codeclimate')
+
+
 class Codeclimate(Linter):
     """Provides an interface to codeclimate."""
     defaults = {
-        'excludes': ['!${folder::}*'],
         'selector': (
             'source.css, '
             'source.go, '
@@ -35,8 +38,8 @@ class Codeclimate(Linter):
         )
     }
 
-    regex = r'^== ((?P<filename>.*)(?= \(\d+ issue\) ==))( \(\d+ issue\) ==\n(?P<line>\d+))(?:-\d+)?:\s(?P<message>.+)$'
-    multiline = True
+    regex = r'^(?P<line>\d+)(?:-\d+)?:\s(?P<message>.+)$'
+    multiline = False
     line_col_base = (1, 1)
     tempfile_suffix = '-'
     error_stream = util.STREAM_BOTH
@@ -47,13 +50,14 @@ def cmd(self):
         abs_path = self.filename
         working_dir = self.get_working_dir()
 
-        msg = 'The file \'%s\' is not part of the working dir (%s). ' \
-              'Please see the Linter\'s README.md to get further ' \
-              'instructions.' % (abs_path, working_dir)
+        msg = 'The file \'%s\' is not part of any open folder in ' \
+              'SublimeText. Please see the Linter\'s README.md to ' \
+              'get further instructions.' % (abs_path)
 
         if not (abs_path.startswith(os.path.abspath(working_dir) + os.sep)):
+            logger.warning(msg)
             self.notify_unassign()
-            raise PermanentError(msg)
+            raise PermanentError('File not part of an open folder in SublimeText.')
 
         file = os.path.relpath(abs_path, working_dir)
         return ['codeclimate', 'analyze', '${args}', file, '${xoo}']