pronto 0.1.7 → 0.2.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: a1fa95ec101ccf2570dbe2092aa71d0153f3128c
-  data.tar.gz: 82de58f6378b124f8e12093f20ca43678bd999a4
+  metadata.gz: ccb63635760620106d5ff1d436d22ca6b13848fc
+  data.tar.gz: c4a6efc7cba539d3834515f014dac0345822199b
 SHA512:
-  metadata.gz: a5fe03a1de231b2188dae91205b74a1912dd68c1fc849a8145039af8ca920fb0e190315ead426a25c3f0a384f470f84b671edc933d1a18b9bf94b319a6b19b8e
-  data.tar.gz: a5792d5886aa4ad19361d20075ae3d47334d1590fc89b089a1a3788c5823c4c87e9bb27e46e4f06c3869b5da8304389cda8b960925086ca542088c01b9985674
+  metadata.gz: 0b938196c2df754b0a3f193047a54a8f36f016c6759e2ba3434fff04baf7292410d18bc1cd24ffdb48fd2dfedb688887679a6fdd68fc49b6fdb1aab9b24a4dc1
+  data.tar.gz: 5883eb8f88031f6568dc086ccbe30064f8a8768a4097386f73031765a2ae1ba1193429daa14d8b5e4e1c11cf9b29b8a96cf39920ac3947c5ef915a62d8ed0ca3

data/README.md

@@ -2,6 +2,7 @@
 
 [![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)
+[![Gem Version](https://badge.fury.io/rb/pronto.png)](http://badge.fury.io/rb/pronto)
 [![Dependency Status](https://gemnasium.com/mmozuras/pronto.png)](https://gemnasium.com/mmozuras/pronto)
 
 ## Usage
@@ -53,3 +54,4 @@ Currently available runners:
 * [pronto-rubocop](https://github.com/mmozuras/pronto-rubocop)
 * [pronto-flay](https://github.com/mmozuras/pronto-flay)
 * [pronto-brakeman](https://github.com/mmozuras/pronto-brakeman)
+* [pronto-rails_best_practices](https://github.com/mmozuras/pronto-rails_best_practices)

data/lib/pronto.rb

@@ -20,7 +20,7 @@ require 'pronto/formatter/formatter'
 module Pronto
   def self.run(commit = 'master', repo_path = '.', formatter = nil)
     patches = diff(repo_path, commit)
-    result = run_all_runners(patches)
+    result = run_all_runners(patches, commit)
 
     formatter ||= default_formatter
     formatter.format(result)
@@ -50,9 +50,9 @@ module Pronto
     repo.diff(merge_base, repo.head.target)
   end
 
-  def self.run_all_runners(patches)
+  def self.run_all_runners(patches, commit)
     Runner.runners.map do |runner|
-      runner.new.run(patches)
+      runner.new.run(patches, commit)
     end.flatten.compact
   end
 

data/lib/pronto/cli.rb

@@ -5,7 +5,14 @@ module Pronto
     require 'pronto'
     require 'pronto/version'
 
-    desc 'exec', 'Run Pronto'
+    class << self
+      def is_thor_reserved_word?(word, type)
+        return false if word == 'run'
+        super
+      end
+    end
+
+    desc 'run', 'Run Pronto'
 
     method_option :commit,
                   type: :string,
@@ -25,7 +32,7 @@ module Pronto
                   aliases: '-f',
                   banner: "Pick output formatter. Available: #{::Pronto::Formatter.names.join(', ')}"
 
-    def exec
+    def run
       gem_names = options[:runner].any? ? options[:runner] : ::Pronto.gem_names
       gem_names.each do |gem_name|
         require "pronto/#{gem_name}"

data/lib/pronto/formatter/github_formatter.rb

@@ -6,7 +6,7 @@ module Pronto
       def format(messages)
         commit_messages = messages.map do |message|
           repo = github_slug(message)
-          sha = message.line.commit_sha
+          sha = message.commit_sha
           position = message.line.commit_line.position
           path = message.path
           body = message.msg

data/lib/pronto/formatter/json_formatter.rb

@@ -5,12 +5,13 @@ module Pronto
     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
-          }
+          lineno = message.line.new_lineno if message.line
+
+          result = { level: message.level[0].upcase, message: message.msg }
+          result[:path] = message.path if message.path
+          result[:line] = lineno if lineno
+          result[:commit_sha] = message.commit_sha if message.commit_sha
+          result
         end.to_json
       end
     end

data/lib/pronto/formatter/text_formatter.rb

@@ -5,7 +5,12 @@ module Pronto
         messages.map do |message|
           level = message.level[0].upcase
           line = message.line
-          "#{message.path}:#{line.new_lineno} #{level}: #{message.msg}"
+          lineno = line.new_lineno if line
+          path = message.path
+          commit_sha = message.commit_sha[0..6] if message.commit_sha
+
+          location = (path.nil? && lineno.nil?) ? commit_sha : "#{path}:#{lineno}"
+          "#{location} #{level}: #{message.msg}"
         end
       end
     end

data/lib/pronto/message.rb

@@ -1,15 +1,20 @@
 module Pronto
   class Message
-    attr_reader :path, :line, :level, :msg
+    attr_reader :path, :line, :level, :msg, :commit_sha
 
     LEVELS = [:info, :warning, :error, :fatal]
 
-    def initialize(path, line, level, msg)
+    def initialize(path, line, level, msg, commit_sha = nil)
       unless LEVELS.include?(level)
         raise ::ArgumentError, "level should be set to one of #{LEVELS}"
       end
 
-      @path, @line, @level, @msg = path, line, level, msg
+      @path = path
+      @line = line
+      @level = level
+      @msg = msg
+      @commit_sha = commit_sha
+      @commit_sha ||= line.commit_sha if line
     end
 
     def repo

data/lib/pronto/rugged/diff/line.rb

@@ -26,13 +26,14 @@ module Rugged
 
       def commit_line
         @commit_line ||= begin
-          commit_patch = commit.show.patches.find do |p|
+          diff = commit.show
+          patches = diff ? diff.patches : []
+          commit_patch = patches.find do |p|
             patch.new_file_full_path == p.new_file_full_path
           end
 
-          result = commit_patch.lines.find do |l|
-            blameline.lineno == l.new_lineno
-          end
+          lines = commit_patch ? commit_patch.lines : []
+          result = lines.find { |l| blameline.lineno == l.new_lineno }
 
           result || self # no commit_line means that it was just added
         end

data/lib/pronto/version.rb

@@ -1,3 +1,3 @@
 module Pronto
-  VERSION = '0.1.7'
+  VERSION = '0.2.0'
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: pronto
 version: !ruby/object:Gem::Version
-  version: 0.1.7
+  version: 0.2.0
 platform: ruby
 authors:
 - Mindaugas Mozūras
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-10-08 00:00:00.000000000 Z
+date: 2013-10-20 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rugged
@@ -94,34 +94,6 @@ dependencies:
     - - ~>
       - !ruby/object:Gem::Version
         version: 2.14.0
-- !ruby/object:Gem::Dependency
-  name: pronto-rubocop
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - ~>
-      - !ruby/object:Gem::Version
-        version: 0.1.0
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - ~>
-      - !ruby/object:Gem::Version
-        version: 0.1.0
-- !ruby/object:Gem::Dependency
-  name: pronto-flay
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - ~>
-      - !ruby/object:Gem::Version
-        version: 0.1.0
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - ~>
-      - !ruby/object:Gem::Version
-        version: 0.1.0
 description: 
 email: mindaugas.mozuras@gmail.com
 executables:
@@ -171,8 +143,9 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: 1.3.6
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.0.6
+rubygems_version: 2.0.7
 signing_key: 
 specification_version: 4
 summary: Pronto runs analysis quickly by checking only the introduced changes
 test_files: []
+has_rdoc: