Contents

module Overcommit::GitHook
  class RubyStyle < HookSpecificCheck
    include HookRegistry
    file_type :rb

    def run_check
      unless in_path?('rubocop')
        return :warn, 'rubocop not installed -- run `gem install rubocop`'
      end

      paths_to_staged_files = Hash[staged.map { |s| [s.path, s] }]
      staged_files = paths_to_staged_files.keys

      output = `rubocop --format=emacs --silent #{staged_files.join(' ')} 2>&1`
      return :good if $?.success?

      # Keep lines from the output for files that we actually modified
      error_lines, warning_lines = output.lines.partition do |output_line|
        if match = output_line.match(/^([^:]+):(\d+)/)
          file = match[1]
          line = match[2]
        end
        paths_to_staged_files[file].modified_lines.include?(line.to_i)
      end

      return :bad, error_lines.join unless error_lines.empty?
      return :warn, "Modified files have style lints (on lines you didn't modify)\n" <<
                    warning_lines.join
    end
  end
end

Version data entries

2 entries across 2 versions & 1 rubygems

Version	Path
overcommit-0.2.1	lib/overcommit/plugins/pre_commit/ruby_style.rb
overcommit-0.2.0	lib/overcommit/plugins/pre_commit/ruby_style.rb