Match leading drive specifier in message regexes

Author: jawshooah

lib/overcommit/hook/pre_commit/css_lint.rb

@@ -4,7 +4,7 @@ module Overcommit::Hook::PreCommit
   # @see https://github.com/CSSLint/csslint
   class CssLint < Base
     MESSAGE_REGEX = /
-      ^(?<file>[^:]+):\s
+      ^(?<file>(?:\w:)?[^:]+):\s
       (?:line\s(?<line>\d+)[^EW]+)?
       (?<type>Error|Warning)
     /x

lib/overcommit/hook/pre_commit/es_lint.rb

@@ -12,7 +12,7 @@ def run
       #   path/to/file.js: line 1, col 0, Error - Error message (ruleName)
       extract_messages(
         output.split("\n").grep(/Warning|Error/),
-        /^(?<file>[^:]+):[^\d]+(?<line>\d+).*?(?<type>Error|Warning)/,
+        /^(?<file>(?:\w:)?[^:]+):[^\d]+(?<line>\d+).*?(?<type>Error|Warning)/,
         lambda { |type| type.downcase.to_sym }
       )
     end

lib/overcommit/hook/pre_commit/go_lint.rb

@@ -13,7 +13,7 @@ def run
       #   path/to/file.go:1:1: Error message
       extract_messages(
         output.split("\n"),
-        /^(?<file>[^:]+):(?<line>\d+)/
+        /^(?<file>(?:\w:)?[^:]+):(?<line>\d+)/
       )
     end
   end

lib/overcommit/hook/pre_commit/go_vet.rb

@@ -15,7 +15,7 @@ def run
       #   path/to/file.go:7: Error message
       extract_messages(
         result.stderr.split("\n"),
-        /^(?<file>[^:]+):(?<line>\d+)/
+        /^(?<file>(?:\w:)?[^:]+):(?<line>\d+)/
       )
     end
   end

lib/overcommit/hook/pre_commit/haml_lint.rb

@@ -13,7 +13,7 @@ def run
 
       extract_messages(
         result.stdout.split("\n"),
-        /^(?<file>[^:]+):(?<line>\d+)[^ ]* (?<type>[^ ]+)/,
+        /^(?<file>(?:\w:)?[^:]+):(?<line>\d+)[^ ]* (?<type>[^ ]+)/,
         MESSAGE_TYPE_CATEGORIZER,
       )
     end

lib/overcommit/hook/pre_commit/hard_tabs.rb

@@ -6,7 +6,7 @@ def run
 
       extract_messages(
         result.stdout.split("\n"),
-        /^(?<file>[^:]+):(?<line>\d+)/,
+        /^(?<file>(?:\w:)?[^:]+):(?<line>\d+)/,
       )
     end
   end

lib/overcommit/hook/pre_commit/html_hint.rb

@@ -13,7 +13,7 @@ def run
         file = lines[0][/(.+):/, 1]
         extract_messages(
           lines[1..-1].map { |msg| "#{file}: #{msg}" },
-          /^(?<file>[^:]+): line (?<line>\d+)/
+          /^(?<file>(?:\w:)?[^:]+): line (?<line>\d+)/
         )
       end.flatten
     end

lib/overcommit/hook/pre_commit/html_tidy.rb

@@ -4,7 +4,7 @@ module Overcommit::Hook::PreCommit
   # @see http://www.html-tidy.org/
   class HtmlTidy < Base
     MESSAGE_REGEX = /
-      ^(?<file>[^:]+):\s
+      ^(?<file>(?:\w:)?[^:]+):\s
       line\s(?<line>\d+)\s
       column\s(?<col>\d+)\s-\s
       (?<type>Error|Warning):\s(?<message>.+)$

lib/overcommit/hook/pre_commit/java_checkstyle.rb

@@ -3,7 +3,7 @@ module Overcommit::Hook::PreCommit
   #
   # @see http://checkstyle.sourceforge.net/
   class JavaCheckstyle < Base
-    MESSAGE_REGEX = /^(?<file>[^:]+):(?<line>\d+)/
+    MESSAGE_REGEX = /^(?<file>(?:\w:)?[^:]+):(?<line>\d+)/
 
     def run
       result = execute(command + applicable_files)

lib/overcommit/hook/pre_commit/js_hint.rb

@@ -13,7 +13,7 @@ def run
       #   path/to/file.js: line 1, col 0, Error message (E001)
       extract_messages(
         output.split("\n").grep(/E|W/),
-        /^(?<file>[^:]+):[^\d]+(?<line>\d+).+\((?<type>E|W)\d+\)/,
+        /^(?<file>(?:\w:)?[^:]+):[^\d]+(?<line>\d+).+\((?<type>E|W)\d+\)/,
         lambda { |type| type.include?('W') ? :warning : :error }
       )
     end

lib/overcommit/hook/pre_commit/js_lint.rb

@@ -3,7 +3,7 @@ module Overcommit::Hook::PreCommit
   #
   # @see http://www.jslint.com/
   class JsLint < Base
-    MESSAGE_REGEX = /(?<file>[^:]+):(?<line>\d+)/
+    MESSAGE_REGEX = /(?<file>(?:\w:)?[^:]+):(?<line>\d+)/
 
     def run
       result = execute(command + applicable_files)

lib/overcommit/hook/pre_commit/jscs.rb

@@ -17,7 +17,7 @@ def run
       #   path/to/file.js: line 7, col 0, ruleName: Error message
       extract_messages(
         result.stdout.split("\n"),
-        /^(?<file>[^:]+):[^\d]+(?<line>\d+)/,
+        /^(?<file>(?:\w:)?[^:]+):[^\d]+(?<line>\d+)/,
       )
     end
   end

lib/overcommit/hook/pre_commit/jsl.rb

@@ -3,7 +3,7 @@ module Overcommit::Hook::PreCommit
   #
   # @see http://www.javascriptlint.com/
   class Jsl < Base
-    MESSAGE_REGEX = /(?<file>.+)\((?<line>\d+)\):(?<type>[^:]+)/
+    MESSAGE_REGEX = /(?<file>(?:\w:)?.+)\((?<line>\d+)\):(?<type>[^:]+)/
 
     MESSAGE_TYPE_CATEGORIZER = lambda do |type|
       type =~ /warning/ ? :warning : :error

lib/overcommit/hook/pre_commit/pep257.rb

@@ -14,7 +14,7 @@ def run
       #           D102: Docstring missing
       extract_messages(
         output.gsub(/:\s+/, ': ').split("\n"),
-        /^(?<file>[^:]+):(?<line>\d+)/
+        /^(?<file>(?:\w:)?[^:]+):(?<line>\d+)/
       )
     end
   end

lib/overcommit/hook/pre_commit/pep8.rb

@@ -13,7 +13,7 @@ def run
       #   path/to/file.py:88:5: E301 expected 1 blank line, found 0
       extract_messages(
         output.split("\n"),
-        /^(?<file>[^:]+):(?<line>\d+):\d+:\s(?<type>E|W)/,
+        /^(?<file>(?:\w:)?[^:]+):(?<line>\d+):\d+:\s(?<type>E|W)/,
         lambda { |type| type.include?('W') ? :warning : :error }
       )
     end

lib/overcommit/hook/pre_commit/puppet_lint.rb

@@ -3,7 +3,7 @@ module Overcommit::Hook::PreCommit
   #
   # @see http://puppet-lint.com/
   class PuppetLint < Base
-    MESSAGE_REGEX = /(?<file>.+):(?<line>\d+):\d+:(?<type>\w+)/
+    MESSAGE_REGEX = /(?<file>(?:\w:)?.+):(?<line>\d+):\d+:(?<type>\w+)/
 
     MESSAGE_TYPE_CATEGORIZER = lambda do |type|
       type == 'ERROR' ? :error : :warning

lib/overcommit/hook/pre_commit/pyflakes.rb

@@ -3,7 +3,7 @@ module Overcommit::Hook::PreCommit
   #
   # @see https://pypi.python.org/pypi/pyflakes
   class Pyflakes < Base
-    MESSAGE_REGEX = /^(?<file>[^:]+):(?<line>\d+):/
+    MESSAGE_REGEX = /^(?<file>(?:\w:)?[^:]+):(?<line>\d+):/
 
     def run
       result = execute(command + applicable_files)

lib/overcommit/hook/pre_commit/pylint.rb

@@ -3,7 +3,7 @@ module Overcommit::Hook::PreCommit
   #
   # @see http://www.pylint.org/
   class Pylint < Base
-    MESSAGE_REGEX = /^(?<file>.+):(?<line>\d+):(?<type>[CEFRW])/
+    MESSAGE_REGEX = /^(?<file>(?:\w:)?.+):(?<line>\d+):(?<type>[CEFRW])/
 
     # Classify 'E' and 'F' message codes as errors,
     # everything else as warnings.

lib/overcommit/hook/pre_commit/python_flake8.rb

@@ -3,7 +3,7 @@ module Overcommit::Hook::PreCommit
   #
   # @see https://pypi.python.org/pypi/flake8
   class PythonFlake8 < Base
-    MESSAGE_REGEX = /^(?<file>.+):(?<line>\d+):\d+:\s(?<type>\w\d+)/
+    MESSAGE_REGEX = /^(?<file>(?:\w:)?.+):(?<line>\d+):\d+:\s(?<type>\w\d+)/
 
     # Classify 'Exxx' and 'Fxxx' message codes as errors,
     # everything else as warnings.

lib/overcommit/hook/pre_commit/reek.rb

@@ -11,7 +11,7 @@ def run
 
       extract_messages(
         output,
-        /^\s*(?<file>[^:]+):(?<line>\d+):/,
+        /^\s*(?<file>(?:\w:)?[^:]+):(?<line>\d+):/,
       )
     end
 

lib/overcommit/hook/pre_commit/rubo_cop.rb

@@ -13,7 +13,7 @@ def run
 
       extract_messages(
         result.stdout.split("\n"),
-        /^(?<file>[^:]+):(?<line>\d+):[^ ]+ (?<type>[^ ]+)/,
+        /^(?<file>(?:\w:)?[^:]+):(?<line>\d+):[^ ]+ (?<type>[^ ]+)/,
         MESSAGE_TYPE_CATEGORIZER,
       )
     end

lib/overcommit/hook/pre_commit/ruby_lint.rb

@@ -13,7 +13,7 @@ def run
 
       extract_messages(
         result.stdout.split("\n"),
-        /^(?<file>[^:]+):(?<type>[^:]+):(?<line>\d+)/,
+        /^(?<file>(?:\w:)?[^:]+):(?<type>[^:]+):(?<line>\d+)/,
         MESSAGE_TYPE_CATEGORIZER
       )
     end

lib/overcommit/hook/pre_commit/scalariform.rb

@@ -3,7 +3,7 @@ module Overcommit::Hook::PreCommit
   #
   # @see https://github.com/mdr/scalariform
   class Scalariform < Base
-    MESSAGE_REGEX = /^\[(?<type>FAILED|ERROR)\]\s+(?<file>.+)/
+    MESSAGE_REGEX = /^\[(?<type>FAILED|ERROR)\]\s+(?<file>(?:\w:)?.+)/
 
     def run
       result = execute(command + applicable_files)

lib/overcommit/hook/pre_commit/scalastyle.rb

@@ -5,7 +5,7 @@ module Overcommit::Hook::PreCommit
   class Scalastyle < Base
     MESSAGE_REGEX = /
       ^(?<type>error|warning)\s
-      file=(?<file>.+)\s
+      file=(?<file>(?:\w:)?.+)\s
       message=.+\s
       line=(?<line>\d+)
     /x

lib/overcommit/hook/pre_commit/scss_lint.rb

@@ -20,7 +20,7 @@ def run
 
       extract_messages(
         result.stdout.split("\n"),
-        /^(?<file>[^:]+):(?<line>\d+)[^ ]* (?<type>[^ ]+)/,
+        /^(?<file>(?:\w:)?[^:]+):(?<line>\d+)[^ ]* (?<type>[^ ]+)/,
         MESSAGE_TYPE_CATEGORIZER,
       )
     end

lib/overcommit/hook/pre_commit/semi_standard.rb

@@ -12,7 +12,7 @@ def run
       #   path/to/file.js:1:1: Error message (ruleName)
       extract_messages(
         output.split("\n")[1..-1], # ignore header line
-        /^\s*(?<file>[^:]+):(?<line>\d+)/
+        /^\s*(?<file>(?:\w:)?[^:]+):(?<line>\d+)/
       )
     end
   end

lib/overcommit/hook/pre_commit/shell_check.rb

@@ -13,7 +13,7 @@ def run
 
       extract_messages(
         result.stdout.split("\n"),
-        /^(?<file>[^:]+):(?<line>\d+):[^ ]+ (?<type>[^ ]+)/,
+        /^(?<file>(?:\w:)?[^:]+):(?<line>\d+):[^ ]+ (?<type>[^ ]+)/,
         MESSAGE_TYPE_CATEGORIZER,
       )
     end

lib/overcommit/hook/pre_commit/slim_lint.rb

@@ -13,7 +13,7 @@ def run
 
       extract_messages(
         result.stdout.split("\n"),
-        /^(?<file>[^:]+):(?<line>\d+)[^ ]* (?<type>[^ ]+)/,
+        /^(?<file>(?:\w:)?[^:]+):(?<line>\d+)[^ ]* (?<type>[^ ]+)/,
         MESSAGE_TYPE_CATEGORIZER,
       )
     end

lib/overcommit/hook/pre_commit/sqlint.rb

@@ -3,7 +3,7 @@ module Overcommit::Hook::PreCommit
   #
   # @see https://github.com/purcell/sqlint
   class Sqlint < Base
-    MESSAGE_REGEX = /(?<file>.+):(?<line>\d+):\d+:(?<type>\w+)/
+    MESSAGE_REGEX = /(?<file>(?:\w:)?.+):(?<line>\d+):\d+:(?<type>\w+)/
 
     MESSAGE_TYPE_CATEGORIZER = lambda do |type|
       type == 'ERROR' ? :error : :warning

lib/overcommit/hook/pre_commit/standard.rb

@@ -12,7 +12,7 @@ def run
       #   path/to/file.js:1:1: Error message (ruleName)
       extract_messages(
         output.split("\n")[1..-1], # ignore header line
-        /^\s*(?<file>[^:]+):(?<line>\d+)/
+        /^\s*(?<file>(?:\w:)?[^:]+):(?<line>\d+)/
       )
     end
   end

lib/overcommit/hook/pre_commit/trailing_whitespace.rb

@@ -6,7 +6,7 @@ def run
 
       extract_messages(
         result.stdout.split("\n"),
-        /^(?<file>[^:]+):(?<line>\d+)/,
+        /^(?<file>(?:\w:)?[^:]+):(?<line>\d+)/,
       )
     end
   end

lib/overcommit/hook/pre_commit/vint.rb

@@ -13,7 +13,7 @@ def run
       #   path/to/file.vim:1:1: Error message
       extract_messages(
         result.stdout.split("\n"),
-        /^(?<file>[^:]+):(?<line>\d+)/
+        /^(?<file>(?:\w:)?[^:]+):(?<line>\d+)/
       )
     end
   end

lib/overcommit/hook/pre_commit/xml_lint.rb

@@ -3,7 +3,7 @@ module Overcommit::Hook::PreCommit
   #
   # @see http://xmlsoft.org/xmllint.html
   class XmlLint < Base
-    MESSAGE_REGEX = /^(?<file>[^:]+):(?<line>\d+):/
+    MESSAGE_REGEX = /^(?<file>(?:\w:)?[^:]+):(?<line>\d+):/
 
     def run
       result = execute(command + applicable_files)