gistory 0.1.8 → 0.2.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
-SHA1:
-  metadata.gz: 2d3139f7b6f07fd2b6de135eb63c8ada4baaa1c4
-  data.tar.gz: 95de85e6a6f1b4809295db975d33449d12dd23cd
+SHA256:
+  metadata.gz: '09f3c9251a676aa3fe78d5487e7cb60ee6c8348de21ae7ad4d91ee7a16623f69'
+  data.tar.gz: bb6b913eb6a330dea5db7e8f8bdd4e1db87324ff72abdeb48c835c00c87ab0a3
 SHA512:
-  metadata.gz: f8eae6a6a4b52a452b45f837594646b8a57c6aa4f101c86612b56462a1234025355861c8a6fe0bb22de55c54908c9d422c15634c92c0c22b5ed52b6c5f6ebb21
-  data.tar.gz: cb9c8bd90a5f722217a792b4b3890394894ed23a242d95e1453470ab408f86ba83c145364a8c2f47367552900635ad56d9299809ee64ff76f179ea35f29538b6
+  metadata.gz: 5f0931cb9e7673d6e7d88e1eb80e892ae3f482b1c3c8e4feae59b1b28f20b45a37bb6c08d51e4d15ec0d7115b780c0cc2f4dc4c530739197c4e9c48975ef19db
+  data.tar.gz: 137c0bb24fa19cea60221ab8a57994bc7a4e46c99aa4666c6b3d35b133aeb0559541b26090c9d3f92f71d87daacc471b5fe283e0824433d98d1b8af30c3b3223

data/.rubocop.yml

@@ -1,5 +1,4 @@
-Documentation:
-  Enabled: false
+require: rubocop-rspec
 
 Metrics/AbcSize:
   Max: 25
@@ -13,18 +12,73 @@ Metrics/BlockLength:
 Metrics/MethodLength:
   Max: 20
 
-Metrics/LineLength:
-  Max: 120
-
 Layout/EmptyLineAfterGuardClause:
   Enabled: false
 
+Layout/LineLength:
+  Max: 120
+
 Style/BlockDelimiters:
   Exclude:
     - 'spec/**/*_spec.rb'
 
+Style/Documentation:
+  Enabled: false
+
 Style/IfUnlessModifier:
   Enabled: false
 
 Style/FrozenStringLiteralComment:
   EnforcedStyle: always
+
+# rubocop-rspec
+
+RSpec/MultipleExpectations:
+  Max: 15
+
+RSpec/ExampleLength:
+  Max: 15
+
+# starting rubocop 0.80 newly added cops need to be explicitly enabled:
+
+Lint/DeprecatedOpenSSLConstant:
+  Enabled: true
+
+Layout/EmptyLinesAroundAttributeAccessor:
+  Enabled: true
+
+Lint/MixedRegexpCaptureTypes:
+  Enabled: true
+
+Lint/RaiseException:
+  Enabled: true
+
+Layout/SpaceAroundMethodCallOperator:
+  Enabled: true
+
+Lint/StructNewOverride:
+  Enabled: true
+
+Style/ExponentialNotation:
+  Enabled: true
+
+Style/HashEachMethods:
+  Enabled: true
+
+Style/HashTransformKeys:
+  Enabled: true
+
+Style/HashTransformValues:
+  Enabled: true
+
+Style/RedundantFetchBlock:
+  Enabled: true
+
+Style/RedundantRegexpCharacterClass:
+  Enabled: true
+
+Style/RedundantRegexpEscape:
+  Enabled: true
+
+Style/SlicingWithRange:
+  Enabled: true

data/.travis.yml

@@ -3,7 +3,7 @@ language: ruby
 script:
   - bundle exec rspec
 rvm:
-  - 2.4.0
-  - 2.3.3
-  - 2.2.6
+  - 2.6.6
+  - 2.5.8
+  - 2.4.10
 before_install: gem install bundler -v 1.17.3

data/README.md

@@ -5,7 +5,7 @@
 [![Build Status](https://travis-ci.org/serch/gistory.svg?branch=master)](https://travis-ci.org/serch/gistory)
 [![Coverage Status](https://coveralls.io/repos/github/serch/gistory/badge.svg?branch=master)](https://coveralls.io/github/serch/gistory?branch=master)
 
-If you use bundler and git, and want to know when a gem you are using was updated, `gistory` comes to your rescue, simply:
+If you use bundler and git and you want to know when a gem was updated, `gistory` comes to the rescue, simply:
 
 ```shell
 gem install gistory
@@ -14,6 +14,7 @@ gistory sidekiq
 ```
 
 and you'll see something like:
+
 ```
 Gem: sidekiq
 Current version: 4.2.7
@@ -25,11 +26,17 @@ Change history:
 4.1.4 on Wed,  9 Nov 2016 14:31 +01:00 (commit 05a3c549)
 ```
 
-By default `gistory` only looks at the last 100 changes to Gemfile.lock
-if you want to see farther in the past run:
+By default `gistory` only looks at the last 100 commits made to the current branch.
+If you want to see farther back in the past run:
+
+```shell
+gistory sidekiq -m1000
+```
+
+If you want to look at all changes to Gemfile.lock in all branches, use the `-a` switch:
 
 ```shell
-gistory sidekiq -m10000
+gistory sidekiq -a
 ```
 
 Note that if the gem was added, then removed, and then added again, `gistory` will
@@ -44,4 +51,3 @@ only show the latest version changes up until it was removed.
 - remove bundler dep
 - add yard doc
 - do not print the warning text if there were no more changes in the lock file
-- find version with the longest length and pad all others to match it (f.i. rails has 4.2.8 and 4.2.7.1)

data/bin/console

@@ -8,7 +8,7 @@ require 'pry'
 root_path = "#{File.dirname(__FILE__)}/.."
 
 # require all files in lib for quick access to them in the console
-Dir["#{root_path}/lib/**/*.rb"].each { |file| require file }
+Dir["#{root_path}/lib/**/*.rb"].sort.each { |file| require file }
 
 def reload!
   verbosity = $VERBOSE

data/gistory.gemspec

@@ -8,7 +8,7 @@ Gem::Specification.new do |spec|
   spec.name = 'gistory'
   spec.version = Gistory::VERSION
   spec.platform = Gem::Platform::RUBY
-  spec.required_ruby_version = '>= 2.2'
+  spec.required_ruby_version = '>= 2.4'
   spec.authors = ['Sergio Medina']
   spec.email = ['medinasergio@gmail.com']
 
@@ -27,6 +27,10 @@ Gem::Specification.new do |spec|
   spec.add_dependency 'bundler', '~> 1.0'
   spec.add_dependency 'colorize'
 
-  spec.add_development_dependency 'rake', '~> 10.0'
-  spec.add_development_dependency 'rspec', '~> 3.8'
+  spec.add_development_dependency 'pry'
+  spec.add_development_dependency 'pry-byebug'
+  spec.add_development_dependency 'rake', '~> 13.0'
+  spec.add_development_dependency 'rspec', '~> 3.9'
+  spec.add_development_dependency 'rubocop'
+  spec.add_development_dependency 'rubocop-rspec'
 end

data/lib/gistory/change_log.rb

@@ -10,17 +10,17 @@ module Gistory
 
     def changelog_for_gem(gem_name)
       version_changes = []
-      lockfile_changes = @repo.changes_to_file(LOCKFILE)
+      commits_with_changes = @repo.changes_to_file(LOCKFILE)
 
       # no lockfile found or no changes to the lockfile found
-      return [] if lockfile_changes.empty?
+      return [] if commits_with_changes.empty?
 
-      previous_commit = lockfile_changes.shift
+      previous_commit = commits_with_changes.shift
       previous_gem_spec = gem_version_at_commit_hash(previous_commit.short_hash, gem_name)
       # only one change to the lockfile was found and the gem was not there
       return [] if previous_gem_spec.nil?
 
-      lockfile_changes.each do |current_commit|
+      commits_with_changes.each do |current_commit|
         current_gem_spec = gem_version_at_commit_hash(current_commit.short_hash, gem_name)
 
         # we reached the end, the gem didn't exist back then

data/lib/gistory/cli/arg_parser.rb

@@ -38,33 +38,35 @@ module Gistory
         parser = OptionParser.new
         parser.banner = 'Usage: gistory <gem_name> [options]'
 
-        add_specific_options(parser, config)
-        add_common_options(parser)
+        add_options(parser, config)
 
         parser
       end
 
-      def add_specific_options(parser, config)
+      def add_options(parser, config)
         parser.separator ''
-        parser.separator 'Specific options:'
-
-        add_max_lockfile_changes(parser, config)
-      end
-
-      def add_common_options(parser)
-        parser.separator ''
-        parser.separator 'Common options:'
+        parser.separator 'Options:'
 
+        add_max_fetched_commits(parser, config)
+        add_use_commits_from_all_branches(parser, config)
         add_help(parser)
         add_version(parser)
       end
 
-      def add_max_lockfile_changes(parser, config)
-        default = config.max_lockfile_changes
-        description = "max number of changes to the lock file (default #{default})"
-        parser.on('-m', '--max-lockfile-changes [INTEGER]', Integer, description) do |m|
-          raise(Gistory::ParserError, 'argument --max-lockfile-changes must be an integer') if m.nil?
-          config.max_lockfile_changes = m
+      def add_max_fetched_commits(parser, config)
+        default = config.max_fetched_commits
+        description = "max number of commits to be fetched (default #{default})"
+        parser.on('-m', '--max-fetched-commits [Integer]', Integer, description) do |m|
+          raise(Gistory::ParserError, 'argument --max-fetched-commits must be an integer') if m.nil?
+          config.max_fetched_commits = m
+        end
+      end
+
+      def add_use_commits_from_all_branches(parser, config)
+        description = 'use commits from all branches ' \
+                      '(by default it uses only commits made to the current branch)'
+        parser.on('-a', '--all-branches', description) do |a|
+          config.all_branches = a
         end
       end
 

data/lib/gistory/cli/main.rb

@@ -34,14 +34,32 @@ module Gistory
         @io.puts "Current version: #{changes.first.version}"
         @io.puts ''
 
+        print_change_history(changes)
+
+        @io.puts ''
+
+        print_configuration_info
+      end
+
+      def print_change_history(changes)
         @io.puts 'Change history:'
+        max_length = changes.map { |c| c.version.length }.max
+
         changes.each do |change|
-          @io.puts "#{change.version} on #{change.date.strftime('%a, %e %b %Y %H:%M %Z')} (commit #{change.short_hash})"
+          @io.puts "#{change.version.ljust(max_length)} on #{change.date.strftime('%a, %e %b %Y %H:%M %Z')} " \
+                   "(commit #{change.short_hash})"
         end
+      end
 
-        @io.puts ''
-        max = Gistory.config.max_lockfile_changes
-        @io.puts "The last #{max} changes to the lock file were taken into account, to see farther in the past use the -m switch" # rubocop:disable Metrics/LineLength
+      def print_configuration_info
+        max = Gistory.config.max_fetched_commits
+        if Gistory.config.all_branches?
+          @io.puts "The last #{max} changes to the lock file were fetched."
+        else
+          @io.puts "The last #{max} commits made to the current branch were fetched."
+        end
+
+        @io.puts 'To see farther in the past use the -m switch'
       end
     end
   end

data/lib/gistory/commit.rb

@@ -11,5 +11,9 @@ module Gistory
       @date = DateTime.parse(date.to_s)
       freeze
     end
+
+    def to_s
+      "Commit #{short_hash} on #{date}"
+    end
   end
 end

data/lib/gistory/configuration.rb

@@ -2,10 +2,16 @@
 
 module Gistory
   class Configuration
-    attr_accessor :gem_name, :max_lockfile_changes
+    attr_accessor :gem_name, :max_fetched_commits
+    attr_writer :all_branches
 
     def initialize
-      @max_lockfile_changes = 100
+      @max_fetched_commits = 100
+      @all_branches = false
+    end
+
+    def all_branches?
+      @all_branches
     end
   end
 end

data/lib/gistory/git_repo.rb

@@ -10,8 +10,9 @@ module Gistory
     end
 
     def changes_to_file(filename)
-      max_count = Gistory.config.max_lockfile_changes
-      hashes_and_dates = git("log --pretty=format:'%h|%cD' --max-count=#{max_count} --follow #{filename}")
+      max_count = Gistory.config.max_fetched_commits
+      strategy = git_log_strategy(filename)
+      hashes_and_dates = git("log --pretty=format:'%h|%cD' --max-count=#{max_count} #{strategy}")
       to_commits(hashes_and_dates.split("\n"))
     end
 
@@ -21,6 +22,15 @@ module Gistory
 
     private
 
+    def git_log_strategy(filename)
+      if Gistory.config.all_branches?
+        "--follow #{filename}"
+      else
+        # TODO: filter out commits that did not introduce changes to the lock file
+        '--first-parent'
+      end
+    end
+
     def git_cli_available?
       system('which git > /dev/null 2>&1')
     end

data/lib/gistory/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Gistory
-  VERSION = '0.1.8'
+  VERSION = '0.2.0'
 end

data/lib/gistory/version_change.rb

@@ -14,5 +14,9 @@ module Gistory
       @version = version
       freeze
     end
+
+    def to_s
+      "Version #{version} (on #{date} by #{short_hash})"
+    end
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: gistory
 version: !ruby/object:Gem::Version
-  version: 0.1.8
+  version: 0.2.0
 platform: ruby
 authors:
 - Sergio Medina
-autorequire: 
+autorequire:
 bindir: exe
 cert_chain: []
-date: 2019-08-17 00:00:00.000000000 Z
+date: 2020-06-27 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -38,34 +38,90 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
+- !ruby/object:Gem::Dependency
+  name: pry
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: pry-byebug
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: rake
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '10.0'
+        version: '13.0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '10.0'
+        version: '13.0'
 - !ruby/object:Gem::Dependency
   name: rspec
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '3.8'
+        version: '3.9'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '3.8'
+        version: '3.9'
+- !ruby/object:Gem::Dependency
+  name: rubocop
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: rubocop-rspec
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 description: 'Gistory: Know exactly when a gem was updated in your Gemfile.lock'
 email:
 - medinasergio@gmail.com
@@ -104,7 +160,7 @@ homepage: https://www.github.com/serch/gistory
 licenses:
 - MIT
 metadata: {}
-post_install_message: 
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -112,16 +168,15 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: '2.2'
+      version: '2.4'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubyforge_project: 
-rubygems_version: 2.6.14.1
-signing_key: 
+rubygems_version: 3.0.3
+signing_key:
 specification_version: 4
 summary: 'Gistory: Know exactly when a gem was updated in your Gemfile.lock'
 test_files: []