pronto 0.0.2 → 0.1.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 32afa53492ee02c977a4c7def6376ff63f1789af
-  data.tar.gz: c62b2494b9f26207d368e4788b432e971c0f81c5
+  metadata.gz: cd8b0fda3d023e477bd597284d3e03929203c630
+  data.tar.gz: c288acc2abfe04edfaf9019ab9a88e6e31ff1546
 SHA512:
-  metadata.gz: 4ccaf764dfd8eb44ac04dfd5e48d65c119ef98c0a1388cb2936450bc9d7f644a8c32714b8ee9b8a43d94868ce342aa9553dc834b5952f860737dbf25250ad30b
-  data.tar.gz: 94ee6628d7ac7443f4992afc3ae0a0694dfa00d16580edd5d86c3a00996dbe1c890de2386e092ed7c7826c50cfe6e62923efeeaec491d7cb121aabe859f82b64
+  metadata.gz: fb4693ed64d9c46496d2df7414613609120b56dc3673b97729dd4dcb6fc2866624f1b49c800575562634fff7e7dc824a0bf22db2f77c7766d2a205828a4695ce
+  data.tar.gz: ea08d936438b0af186d709736166b88043a0487acf100ca6f0af2d51a5b4b676f3e2ba537c1ed2e83eb3afb7f01d679546198c11e95dee8d29f4852dff201a51

data/README.md

@@ -1,6 +1,9 @@
 # pronto
 
+[![Code Climate](https://codeclimate.com/github/mmozuras/pronto.png)](https://codeclimate.com/github/mmozuras/pronto)
 [![Build Status](https://secure.travis-ci.org/mmozuras/pronto.png)](http://travis-ci.org/mmozuras/pronto)
+[![Dependency Status](https://gemnasium.com/mmozuras/pronto.png)](https://gemnasium.com/mmozuras/pronto)
+
 
 ## Usage
 

data/lib/pronto.rb

@@ -1,19 +1,28 @@
 require 'rugged'
 require_relative 'rugged/diff'
-require_relative 'rugged/tree'
 require_relative 'rugged/diff/delta'
+require_relative 'rugged/diff/patch'
+require_relative 'rugged/diff/line'
+require_relative 'rugged/tree'
+require_relative 'rugged/remote'
+require_relative 'rugged/repository'
 
 require 'pronto/plugin'
 require 'pronto/message'
 require 'pronto/runner'
 
 require 'pronto/formatter/text_formatter'
+require 'pronto/formatter/json_formatter'
+require 'pronto/formatter/github_formatter'
+require 'pronto/formatter/formatter'
 
 module Pronto
-  def self.run(commit1 = nil, commit2 = 'master', repo_path = '.')
-    patches = diff(repo_path, commit1, commit2)
+  def self.run(commit = 'master', repo_path = '.', formatter = nil)
+    patches = diff(repo_path, commit)
     result = run_all_runners(patches)
-    Formatter::TextFormatter.new.format(result)
+
+    formatter ||= default_formatter
+    formatter.format(result)
   end
 
   def self.gem_names
@@ -33,11 +42,10 @@ module Pronto
 
   private
 
-  def self.diff(repo_path, commit1, commit2)
+  def self.diff(repo_path, commit)
     repo = Rugged::Repository.new(repo_path)
-    commit1 ||= repo.head.target
-    commit2 ||= 'master'
-    repo.diff(commit1, commit2)
+    commit ||= 'master'
+    repo.diff(commit, repo.head.target)
   end
 
   def self.run_all_runners(patches)
@@ -45,4 +53,8 @@ module Pronto
       runner.new.run(patches)
     end.flatten.compact
   end
+
+  def default_formatter
+    Formatter::TextFormatter.new
+  end
 end

data/lib/pronto/cli.rb

@@ -7,17 +7,11 @@ module Pronto
 
     desc 'exec', 'Run Pronto'
 
-    method_option :commit1,
+    method_option :commit,
                   type: :string,
                   default: nil,
-                  aliases: '-f',
-                  banner: 'First commit for the diff, defaults to current HEAD'
-
-    method_option :commit2,
-                  type: :string,
-                  default: nil,
-                  aliases: '-s',
-                  banner: 'Second commit for the diff, defaults to master'
+                  aliases: '-c',
+                  banner: 'Commit for the diff, defaults to master'
 
     method_option :runner,
                   type: :array,
@@ -25,17 +19,34 @@ module Pronto
                   aliases: '-r',
                   banner: 'Run only the passed runners'
 
+    method_option :formatter,
+                  type: :string,
+                  default: nil,
+                  aliases: '-f',
+                  banner: "Formatter, defaults to text. Available: #{::Pronto::Formatter.names.join(', ')}"
+
+    method_option :access_token,
+                  type: :string,
+                  default: nil,
+                  aliases: '-t',
+                  banner: 'Github access token, used for github formatter'
+
     def exec
       gem_names = options[:runner].any? ? options[:runner]
                                         : ::Pronto.gem_names
       gem_names.each do |gem_name|
         require "pronto/#{gem_name}"
       end
-      puts ::Pronto.run(options[:commit1], options[:commit2])
+
+      formatter = ::Pronto::Formatter.get(options[:formatter])
+      if formatter.is_a? ::Pronto::Formatter::GithubFormatter
+        access_token = options[:access_token]
+        formatter.client = Octokit::Client.new(access_token: access_token)
+      end
+
+      puts ::Pronto.run(options[:commit], '.', formatter)
     rescue Rugged::RepositoryError
       puts '"pronto" should be run from a git repository'
-    rescue => e
-      puts e.message
     end
 
     desc 'list', 'Lists pronto runners that are available to be used'

data/lib/pronto/formatter/formatter.rb

@@ -0,0 +1,18 @@
+module Pronto
+  module Formatter
+    def self.get(name)
+      formatter = FORMATTERS[name.to_s] || TextFormatter
+      formatter.new
+    end
+
+    def self.names
+      FORMATTERS.keys
+    end
+
+    FORMATTERS = {
+      'github' => GithubFormatter,
+      'json' => JsonFormatter,
+      'text' => TextFormatter
+    }
+  end
+end

data/lib/pronto/formatter/github_formatter.rb

@@ -0,0 +1,42 @@
+require 'octokit'
+
+module Pronto
+  module Formatter
+    class GithubFormatter
+      attr_writer :client
+
+      def format(messages)
+        messages.each do |message|
+          @client.create_commit_comment(github_slug(message),
+                                        sha(message),
+                                        message.msg,
+                                        message.path,
+                                        message.line.new_lineno)
+        end
+
+        "#{messages.count} pronto messages posted to GitHub"
+      end
+
+      private
+
+      def github_slug(message)
+        message.repo.remotes.map(&:github_slug).compact.first
+      end
+
+      def sha(message)
+        blamelines = blame(message).lines
+        lineno = message.line.new_lineno
+
+        blameline = blamelines.detect { |line| line.lineno == lineno }
+
+        blameline.commit.id if blameline
+      end
+
+      def blame(message)
+        @blames ||= {}
+        @blames[message.path] ||= message.repo.blame(message.path)
+        @blames[message.path]
+      end
+    end
+  end
+end

data/lib/pronto/formatter/json_formatter.rb

@@ -0,0 +1,18 @@
+require 'json'
+
+module Pronto
+  module Formatter
+    class JsonFormatter
+      def format(messages)
+        messages.map do |message|
+          {
+            path: message.path,
+            line: message.line.new_lineno,
+            level: message.level[0].upcase,
+            message: message.msg
+          }
+        end.to_json
+      end
+    end
+  end
+end

data/lib/pronto/message.rb

@@ -11,5 +11,9 @@ module Pronto
 
       @path, @line, @level, @msg = path, line, level, msg
     end
+
+    def repo
+      line.patch.delta.repo
+    end
   end
 end

data/lib/pronto/runner.rb

@@ -1,5 +1,3 @@
-require 'tempfile'
-
 module Pronto
   class Runner
     include Plugin
@@ -8,50 +6,17 @@ module Pronto
       repository
     end
 
-    def create_tempfiles(blobs)
-      return [] if blobs.nil?
-
-      files = blobs.map { |blob| write(blob) }.compact
-      return [] if files.empty?
-
-      begin
-        if block_given?
-          files.each(&:flush)
-          yield(files)
-        else
-          files
-        end
-      ensure
-        files.each do |file|
-          file.close unless file.closed?
-        end
-      end
-    end
-
-    def create_tempfile(blob)
-      file = write(blob)
-      return if file.nil?
-
-      begin
-        if block_given?
-          file.flush
-          yield(file)
-        else
-          file
-        end
-      ensure
-        file.close if file && !file.closed?
-      end
+    def ruby_file?(path)
+      File.extname(path) == '.rb' || ruby_executable?(path)
     end
 
     private
 
-    def write(blob)
-      return if blob.nil?
-
-      file = Tempfile.new(blob.oid)
-      file.write(blob.text)
-      file
+    def ruby_executable?(path)
+      line = File.open(path) { |file| file.readline }
+      line =~ /#!.*ruby/
+    rescue ArgumentError, EOFError
+      false
     end
   end
 end

data/lib/pronto/version.rb

@@ -1,3 +1,3 @@
 module Pronto
-  VERSION = '0.0.2'
+  VERSION = '0.1.0'
 end

data/lib/rugged/diff/delta.rb

@@ -1,25 +1,13 @@
 module Rugged
   class Diff
     class Delta
-      def new_blob
-        blob(new_file)
-      end
-
-      def old_blob
-        blob(old_file)
-      end
-
       def repo
         diff.tree.repo
       end
 
-      private
-
-      def blob(file)
-        return if file.nil?
-        return if file[:oid] == '0000000000000000000000000000000000000000'
-
-        repo.lookup(file[:oid])
+      def new_file_full_path
+        repo_path = Pathname.new(repo.path).parent
+        repo_path.join(new_file[:path])
       end
     end
   end

data/lib/rugged/diff/line.rb

@@ -0,0 +1,9 @@
+module Rugged
+  class Diff
+    class Line
+      def patch
+        hunk.owner
+      end
+    end
+  end
+end

data/lib/rugged/diff/patch.rb

@@ -0,0 +1,23 @@
+module Rugged
+  class Diff
+    class Patch
+      def added_lines
+        lines.select(&:addition?)
+      end
+
+      def deleted_lines
+        lines.select(&:deletion?)
+      end
+
+      def new_file_full_path
+        delta.new_file_full_path
+      end
+
+      private
+
+      def lines
+        map(&:lines).flatten.compact
+      end
+    end
+  end
+end

data/lib/rugged/remote.rb

@@ -0,0 +1,8 @@
+module Rugged
+  class Remote
+    def github_slug
+      match = /.*github.com(:|\/)(?<slug>.*).git/.match(url)
+      match[:slug] if match
+    end
+  end
+end

data/lib/rugged/repository.rb

@@ -0,0 +1,11 @@
+require 'grit'
+
+module Rugged
+  class Repository
+    def blame(filepath)
+      # TODO: Using grit blame implementation for now.
+      # Replace it with rugged implementation when it's available.
+      ::Grit::Repo.new(path).blame(filepath)
+    end
+  end
+end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: pronto
 version: !ruby/object:Gem::Version
-  version: 0.0.2
+  version: 0.1.0
 platform: ruby
 authors:
 - Mindaugas Mozūras
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-09-01 00:00:00.000000000 Z
+date: 2013-09-22 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rugged
@@ -38,6 +38,34 @@ dependencies:
     - - ~>
       - !ruby/object:Gem::Version
         version: 0.18.0
+- !ruby/object:Gem::Dependency
+  name: octokit
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ~>
+      - !ruby/object:Gem::Version
+        version: 2.1.1
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ~>
+      - !ruby/object:Gem::Version
+        version: 2.1.1
+- !ruby/object:Gem::Dependency
+  name: grit
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ~>
+      - !ruby/object:Gem::Version
+        version: 2.5.0
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ~>
+      - !ruby/object:Gem::Version
+        version: 2.5.0
 - !ruby/object:Gem::Dependency
   name: rake
   requirement: !ruby/object:Gem::Requirement
@@ -58,14 +86,14 @@ dependencies:
     requirements:
     - - ~>
       - !ruby/object:Gem::Version
-        version: 2.13.0
+        version: 2.14.0
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ~>
       - !ruby/object:Gem::Version
-        version: 2.13.0
+        version: 2.14.0
 description: pronto
 email: mindaugas.mozuras@gmail.com
 executables:
@@ -74,6 +102,9 @@ extensions: []
 extra_rdoc_files: []
 files:
 - lib/pronto/cli.rb
+- lib/pronto/formatter/formatter.rb
+- lib/pronto/formatter/github_formatter.rb
+- lib/pronto/formatter/json_formatter.rb
 - lib/pronto/formatter/text_formatter.rb
 - lib/pronto/message.rb
 - lib/pronto/plugin.rb
@@ -81,7 +112,11 @@ files:
 - lib/pronto/version.rb
 - lib/pronto.rb
 - lib/rugged/diff/delta.rb
+- lib/rugged/diff/line.rb
+- lib/rugged/diff/patch.rb
 - lib/rugged/diff.rb
+- lib/rugged/remote.rb
+- lib/rugged/repository.rb
 - lib/rugged/tree.rb
 - LICENSE
 - README.md
@@ -111,3 +146,4 @@ signing_key:
 specification_version: 4
 summary: pronto
 test_files: []
+has_rdoc: