gitarro 0.1.53 → 0.1.54

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: ed7728ce69392b49eb64ed39ead778e1c57c1ff0
-  data.tar.gz: 238ca7702d9a0253f578240a17f225c3abc8c1d4
+  metadata.gz: ea196155b60e599328efe518da225836b264f0fd
+  data.tar.gz: 74cef83a6730617b66c2e71b4191bd7f7563a214
 SHA512:
-  metadata.gz: 762011a475cc4c1dd77b788cce6d3f0fa6a491be52f7c723ad842689d2cb88121a9b6e128d9d8677bbe35fc76cb1956cb2b73c90b6349d8a2f4c43fb17aba68a
-  data.tar.gz: 2c2d61721b17aa0e1f381e45a441c298d463ca3bb3899fb61c143b2c60196ad698be3a0e6a7b0d8d9b2933dca97558b496ecaa79078b211e198f69b544ace585
+  metadata.gz: de2965fe6377e68a1677e21f88c82c10c85165f009fae1b1c0506cb27e938fa555feece15a0bf970363a17bc0d14f7d262cef86461e7963e7fa3eafd2d866826
+  data.tar.gz: 0ce1d7d7380317c34dc6fb3b98c13048bb4d496975ba8fd1bfeeb1ef5f8a8ebd8a00683eeaebe090a035681cabf4143347ad0a8014c0711ed17910fc5d0bfa44

data/lib/gitarro/backend.rb

@@ -4,10 +4,8 @@ require 'octokit'
 require 'optparse'
 require 'time'
 require 'English'
-require 'faraday-http-cache'
 require_relative 'opt_parser'
 require_relative 'git_op'
-require 'active_support'
 
 # this module perform basic operations
 # on prs and contain helper functions
@@ -73,36 +71,6 @@ module GitHubPrOperations
   end
 end
 
-# by default enabled (faraday_cache in memory, will be deleted
-# after gitarro run 2nd time. usefull for -C check option
-# and performance, since we cache)
-module CachingOctokit
-  def create_dir_store(cache_path)
-    cache_name = 'httpcache'
-    full_cache_dir = "#{cache_path}/#{cache_name}"
-    ActiveSupport::Cache::FileStore.new(full_cache_dir)
-  end
-
-  def generate_cache(httpcache_path)
-    # changed_since cannot work with cache
-    return false unless generate_cache?
-    stack = Faraday::RackBuilder.new do |builder|
-      builder.use Faraday::HttpCache,
-                  store: create_dir_store(httpcache_path),
-                  serializer: Marshal,
-                  shared_cache: false
-      builder.use Octokit::Response::RaiseError
-      builder.adapter Faraday.default_adapter
-    end
-    Octokit.middleware = stack
-  end
-
-  # if true we dont need the cache
-  def generate_cache?
-    @cachehttp
-  end
-end
-
 # This is a private class, which has the task to execute/run tests
 # called by Backend
 class TestExecutor
@@ -154,7 +122,6 @@ end
 # were we execute the tests and so on
 class Backend
   attr_accessor :options, :client, :gbexec
-  include CachingOctokit
   include GitHubPrOperations
 
   # public method of backend
@@ -165,8 +132,6 @@ class Backend
       instance_variable_set("@#{key}", value)
       self.class.send(:attr_accessor, key)
     end
-    # if changed_since option on, dont generate cache.
-    generate_cache(@cachehttp)
     Octokit.auto_paginate = true
     @client = Octokit::Client.new(netrc: true)
     @gbexec = TestExecutor.new(@options)

data/lib/gitarro/opt_parser.rb

@@ -73,14 +73,6 @@ module OptionalOptions
     opt.on('--https', https_desc) { |https| @options[:https] = https }
   end
 
-  def cachehttp_opt(opt)
-    desc = 'Custom path where http cache for gitarro is stored' \
-           'by default is set to /tmp/gitarro/httpcache'
-    opt.on('-k', "--cachepath 'CACHEPATH'", desc) do |cache_http|
-      @options[:cachehttp] = cache_http
-    end
-  end
-
   def pr_number(opt)
     desc = 'Specify the PR number instead of checking all of them. ' \
            'Force to rerun against a specific PR number,' \
@@ -107,7 +99,6 @@ module OptionalOptions
     url_opt(opt)
     pr_number(opt)
     https_opt(opt)
-    cachehttp_opt(opt)
     changed_since(opt)
   end
 end
@@ -138,9 +129,6 @@ class OptParserInternal
     parse_options(opt_parser)
     mandatory_options = %w[repo context test_file git_dir]
     mandatory_options.each { |opt| ck_mandatory_option(opt) }
-    if @options[:test_file].nil? && @options[:changelog_test].nil?
-      raise_incorrect_syntax('Incorrect syntax (use -h for help)')
-    end
     defaults_false
     defaults_to_text
   end
@@ -172,7 +160,6 @@ class OptParserInternal
     @options[:target_url] = '' if @options[:target_url].nil?
     @options[:https] = false if @options[:https].nil?
     @options[:changed_since] = -1 if @options[:changed_since].nil?
-    @options[:cachehttp] = false if @options[:cachehttp].nil?
   end
 
   def defaults_to_text

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: gitarro
 version: !ruby/object:Gem::Version
-  version: 0.1.53
+  version: 0.1.54
 platform: ruby
 authors:
 - Dario Maiocchi
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2017-10-18 00:00:00.000000000 Z
+date: 2017-10-19 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: english
@@ -52,34 +52,6 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '4.7'
-- !ruby/object:Gem::Dependency
-  name: activesupport
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '5.1'
-  type: :runtime
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '5.1'
-- !ruby/object:Gem::Dependency
-  name: faraday-http-cache
-  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: minitest
   requirement: !ruby/object:Gem::Requirement
@@ -123,33 +95,33 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '10.5'
 - !ruby/object:Gem::Dependency
-  name: rubocop
+  name: rspec
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.49'
+        version: '3.6'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.49'
+        version: '3.6'
 - !ruby/object:Gem::Dependency
-  name: rspec
+  name: rubocop
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '3.6'
+        version: '0.49'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '3.6'
+        version: '0.49'
 description: gitarro run tests on GitHub PRs using almost any script,language or binary,
   it integrate easy with other tools.
 email: dmaiocchi@suse.com