lintrunner 0.0.1

data/lib/lintrunner/reporter/file.rb

@@ -0,0 +1,11 @@
+module Lintrunner
+  module Reporter
+    class File < Base
+
+      def finish(messages)
+        puts messages.collect(&:filename).uniq.join("\n") if messages.any?
+      end
+
+    end
+  end
+end

data/lib/lintrunner/reporter/text.rb

@@ -0,0 +1,29 @@
+module Lintrunner
+  module Reporter
+    class Text < Base
+
+      def start(name)
+        puts "Running #{name} linters".underline
+      end
+
+      def report(message)
+        puts "#{location(message)} #{message.description} #{message_name(message)}"
+      end
+
+      def finish(messages)
+        puts "No messages found".color(:green) if messages.empty?
+        puts
+      end
+
+      private
+
+      def message_name(message)
+        "(#{message.name})".color(:yellow)
+      end
+
+      def location(message)
+        "#{message.filename.to_s.color(:cyan)}:#{message.line.to_s.color(:green)}"
+      end
+    end
+  end
+end

data/lib/lintrunner/runner/base.rb

@@ -0,0 +1,21 @@
+module Lintrunner
+  module Runner
+    class Base
+      include Lintrunner::GitHelpers
+
+      attr_accessor :path, :match, :executor, :git
+
+      def initialize(path:, match:, executor:)
+        self.path = path
+        self.executor = executor
+        self.match = Regexp.new(Regexp.escape(match))
+        self.git = Rugged::Repository.new(path)
+      end
+
+      def run(reporter = nil)
+        raise "Runner must implement #run method"
+      end
+
+    end
+  end
+end

data/lib/lintrunner/runner/changed_file.rb

@@ -0,0 +1,24 @@
+module Lintrunner
+  module Runner
+    class ChangedFile < Base
+
+      def run(reporter)
+        warnings = []
+        git_changeset.each do |patch|
+          filename = patch.delta.new_file[:path]
+          next if patch.delta.binary?
+          next unless filename =~ match
+          full_path = File.join(path, filename)
+
+          messages = executor.execute(full_path, filename: filename)
+          warnings.concat messages
+          output = messages.collect do |message|
+            reporter.report(message)
+          end
+        end
+        warnings
+      end
+
+    end
+  end
+end

data/lib/lintrunner/runner/diff.rb

@@ -0,0 +1,89 @@
+module Lintrunner
+  module Runner
+    class Diff < Base
+
+      def run(reporter)
+        warnings = []
+        git_changeset.each do |patch|
+          filename = patch.delta.new_file[:path]
+          next if patch.delta.binary?
+          next if filename.end_with?('app/styles/pattern-guide/pattern-guide.scss')
+          next unless filename =~ match
+
+          before_contents = contents_for(patch.delta, :src)
+          after_contents = contents_for(patch.delta, :dst)
+
+          if before_contents && after_contents
+            before_file = tempfile(before_contents)
+            after_file = tempfile(after_contents)
+
+            before_messages = executor.execute(before_file, filename: filename)
+            after_messages = executor.execute(after_file, filename: filename)
+
+            line_map = line_map_for(patch)
+            after_messages.each do |lint|
+              same_lint = before_messages.find do |b_lint|
+                b_lint.line == line_map[lint.line] &&
+                  b_lint.name == lint.name && b_lint.description == lint.description
+              end
+              warnings << lint unless same_lint
+              reporter.report(lint) unless same_lint
+            end
+          elsif after_contents
+            after_file = tempfile(after_contents)
+            after_messages = executor.execute(after_file, filename: filename)
+            warnings.concat after_messages
+            output = after_messages.collect do |message|
+              reporter.report(message)
+            end
+          end
+        end
+        warnings
+      end
+
+      private
+
+      # Get the file contents of a delta
+      # if type=:src, return original file
+      # if type=:dst, return modified file
+      def contents_for(delta, type=:dst)
+        ref = if type == :src
+          delta.old_file[:oid]
+        elsif type == :dst
+          delta.new_file[:oid]
+        end
+        @git.lookup(ref).content if ref != "0000000000000000000000000000000000000000"
+      end
+
+      def line_map_for(patch)
+        line_map = []
+        patch.hunks.first.lines.each do |line|
+          if line.new_lineno != -1
+            line_map[line.new_lineno] = line.old_lineno == -1 ? nil : line.old_lineno
+          end
+        end
+        line_map
+      end
+
+      def tempfile(string)
+        t = Tempfile.new('diff_file')
+        # ensure tempfiles aren't unlinked when GC runs by maintaining a reference to them.
+        @tempfiles ||= []
+        @tempfiles.push(t)
+        t.print(string)
+        t.flush
+        t.close
+        t.path
+      end
+
+      def invert(array)
+        result = []
+        array.each_with_index do |el, i|
+          result[el] = i if el
+        end
+        result
+      end
+
+    end
+  end
+end

data/lib/lintrunner/runner/new_file.rb

@@ -0,0 +1,26 @@
+module Lintrunner
+  module Runner
+    class NewFile < Base
+
+      def run(reporter)
+        warnings = []
+        git_changeset.each do |patch|
+          filename = patch.delta.new_file[:path]
+          next if patch.delta.binary?
+          next unless patch.delta.added?
+          next unless filename =~ match
+
+          full_path = File.join(path, filename)
+
+          messages = executor.execute(full_path, filename: filename)
+          warnings.concat messages
+          output = messages.collect do |message|
+            reporter.report(message)
+          end
+        end
+        warnings
+      end
+
+    end
+  end
+end

data/lib/lintrunner/runner/repo.rb

@@ -0,0 +1,31 @@
+module Lintrunner
+  module Runner
+    class Repo < Base
+
+      def run(reporter = nil)
+        warnings = []
+        files.each do |filename|
+          # next if patch.delta.binary?
+          full_path = File.join(path, filename)
+          next unless filename =~ match
+
+          messages = executor.execute(full_path, filename: filename)
+          warnings.concat messages
+          output = messages.collect do |message|
+            reporter.report(message)
+          end
+        end
+        warnings
+      end
+
+      private
+
+      def files
+        files = []
+        git.head.target.tree.walk_blobs { |root, entry| files << "#{root}#{entry[:name]}" }
+        files
+      end
+
+    end
+  end
+end

data/lib/lintrunner/version.rb

@@ -0,0 +1,3 @@
+module Lintrunner
+  VERSION = "0.0.1"
+end

data/lib/lintrunner.rb

@@ -0,0 +1,34 @@
+# Dependencies
+require "json"
+require "rugged"
+require "tempfile"
+require "stringio"
+require "pathname"
+require "rainbow/ext/string"
+
+require "lintrunner/version"
+require "lintrunner/git_helpers"
+require "lintrunner/message"
+require "lintrunner/plugin"
+require "lintrunner/executor"
+
+# Parsers
+require "lintrunner/parser/base"
+require "lintrunner/parser/eslint"
+require "lintrunner/parser/scss_lint"
+require "lintrunner/parser/rubocop"
+
+# Runners
+require "lintrunner/runner/base"
+require "lintrunner/runner/diff"
+require "lintrunner/runner/new_file"
+require "lintrunner/runner/changed_file"
+require "lintrunner/runner/repo"
+
+# Reporters
+require "lintrunner/reporter/base"
+require "lintrunner/reporter/text"
+require "lintrunner/reporter/context"
+require "lintrunner/reporter/file"
+
+module Lintrunner; end

data/lintrunner.gemspec

@@ -0,0 +1,26 @@
+# coding: utf-8
+lib = File.expand_path('../lib', __FILE__)
+$LOAD_PATH.unshift(lib) unless $LOAD_PATH.include?(lib)
+require 'lintrunner/version'
+
+Gem::Specification.new do |spec|
+  spec.name          = "lintrunner"
+  spec.version       = Lintrunner::VERSION
+  spec.authors       = ["Ivan Tse"]
+  spec.email         = ["ivan.tse1@gmail.com"]
+  spec.summary       = %q{Run multiple lints with various runners}
+  spec.description   = %q{lintrunner}
+  spec.homepage      = ""
+  spec.license       = "MIT"
+
+  spec.files         = `git ls-files -z`.split("\x0")
+  spec.executables   = spec.files.grep(%r{^bin/}) { |f| File.basename(f) }
+  spec.test_files    = spec.files.grep(%r{^(test|spec|features)/})
+  spec.require_paths = ["lib"]
+
+  spec.add_dependency "rugged", "~> 0.21"
+  spec.add_dependency "rainbow", " ~> 2.0"
+
+  spec.add_development_dependency "bundler", "~> 1.7"
+  spec.add_development_dependency "rake", "~> 10.0"
+end

data/test.rb

@@ -0,0 +1,43 @@
+module Lintrunner
+  module Plugin
+    class BindingPryCheck
+
+      attr_accessor :filename
+
+      def initialize(filename)
+        self.filename = filename
+        @binding_pry_matcher = /binding\.pry/
+      end
+
+      def run(options = {})
+        lineno = 1
+        warnings = []
+        return warnings if binary?(filename)
+        File.read(filename).each_line do |line|
+          warnings << create_message(options[:filename] || filename, lineno) if line[@binding_pry_matcher]
+          lineno += 1
+        end
+        warnings
+      end
+
+      def create_message(filename, lineno)
+        Lintrunner::Message.new(
+          filename: filename,
+          line: lineno,
+          name: "BindingPryCheck",
+          description: "binding.pry detected"
+        )
+      end
+
+      def binary?(file)
+        # return false if image?(file)
+        bytes = File.stat(file).blksize
+        bytes = 4096 if bytes > 4096
+        s = (File.read(file, bytes) || "")
+        s = s.encode('US-ASCII', :undef => :replace).split(//)
+        ((s.size - s.grep(" ".."~").size) / s.size.to_f) > 0.30
+      end
+
+    end
+  end
+end

data/test2.rb

@@ -0,0 +1,7 @@
+class Test2
+  <<<<<<< HEAD:file.txt
+  Hello world
+  =======
+  Goodbye
+  >>>>>>> 77976da35a11db4580b80ae27e8d65caf5208086:file.txt
+end

metadata

@@ -0,0 +1,134 @@
+--- !ruby/object:Gem::Specification
+name: lintrunner
+version: !ruby/object:Gem::Version
+  version: 0.0.1
+platform: ruby
+authors:
+- Ivan Tse
+autorequire: 
+bindir: bin
+cert_chain: []
+date: 2015-07-12 00:00:00.000000000 Z
+dependencies:
+- !ruby/object:Gem::Dependency
+  name: rugged
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.21'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.21'
+- !ruby/object:Gem::Dependency
+  name: rainbow
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '2.0'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '2.0'
+- !ruby/object:Gem::Dependency
+  name: bundler
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '1.7'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '1.7'
+- !ruby/object:Gem::Dependency
+  name: rake
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '10.0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '10.0'
+description: lintrunner
+email:
+- ivan.tse1@gmail.com
+executables:
+- lintrunner
+extensions: []
+extra_rdoc_files: []
+files:
+- ".gitignore"
+- ".lintrunner_config"
+- ".rubocop.yml"
+- ".scss-lint.yml"
+- Gemfile
+- LICENSE.txt
+- README.md
+- Rakefile
+- bin/lintrunner
+- lib/lintrunner.rb
+- lib/lintrunner/cli.rb
+- lib/lintrunner/executor.rb
+- lib/lintrunner/git_helpers.rb
+- lib/lintrunner/message.rb
+- lib/lintrunner/options.rb
+- lib/lintrunner/parser/base.rb
+- lib/lintrunner/parser/eslint.rb
+- lib/lintrunner/parser/rubocop.rb
+- lib/lintrunner/parser/scss_lint.rb
+- lib/lintrunner/plugin.rb
+- lib/lintrunner/reporter/base.rb
+- lib/lintrunner/reporter/context.rb
+- lib/lintrunner/reporter/file.rb
+- lib/lintrunner/reporter/text.rb
+- lib/lintrunner/runner/base.rb
+- lib/lintrunner/runner/changed_file.rb
+- lib/lintrunner/runner/diff.rb
+- lib/lintrunner/runner/new_file.rb
+- lib/lintrunner/runner/repo.rb
+- lib/lintrunner/version.rb
+- lintrunner.gemspec
+- test.rb
+- test2.rb
+homepage: ''
+licenses:
+- MIT
+metadata: {}
+post_install_message: 
+rdoc_options: []
+require_paths:
+- lib
+required_ruby_version: !ruby/object:Gem::Requirement
+  requirements:
+  - - ">="
+    - !ruby/object:Gem::Version
+      version: '0'
+required_rubygems_version: !ruby/object:Gem::Requirement
+  requirements:
+  - - ">="
+    - !ruby/object:Gem::Version
+      version: '0'
+requirements: []
+rubyforge_project: 
+rubygems_version: 2.2.2
+signing_key: 
+specification_version: 4
+summary: Run multiple lints with various runners
+test_files: []