Fix HardTabs pre-commit hook to use extract_messages

Similar to 06d5d75, we change `HardTabs` to use `extract_messages` so
line information is captured.

Author: sds

CHANGELOG.md

@@ -11,9 +11,9 @@
   [Scalariform](https://mdr.github.io/scalariform/)
 * Add `SlimLint` pre-commit hook that analyzes Slim templates with
   [Slim-Lint](https://github.com/sds/slim-lint)
-* Enhance `TrailingWhitespace` pre-commit hook to including line information in
-  errors, making it work as expected when `problem_on_unmodified_line` is set
-  to something other than `report`
+* Enhance `HardTabs` and `TrailingWhitespace` pre-commit hooks to including
+  line information in errors, making it work as expected when
+  `problem_on_unmodified_line` is set to something other than `report`
 
 ## 0.24.0
 

lib/overcommit/hook/pre_commit/hard_tabs.rb

@@ -2,13 +2,12 @@ module Overcommit::Hook::PreCommit
   # Checks for hard tabs in files.
   class HardTabs < Base
     def run
-      # Catches hard tabs
       result = execute(command + applicable_files)
-      unless result.stdout.empty?
-        return :fail, "Hard tabs detected:\n#{result.stdout}"
-      end
 
-      :pass
+      extract_messages(
+        result.stdout.split("\n"),
+        /^(?<file>[^:]+):(?<line>\d+)/,
+      )
     end
   end
 end