boxt_ruby_style_guide 7.3.0 → 7.7.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: db37fda2704f058428f66ef3227058dc67c4abd08df1829c5674ff827079bbf6
-  data.tar.gz: 90302060b3ae806b43f4cce6165744e6b5af59566176dffbc878875a876038c9
+  metadata.gz: 28eca289d2e48ea7d90efe79f6815253121c1876e5066033cfe11ad802f8c901
+  data.tar.gz: 1608aaac5753d5f00865542d18f1f0daf9b0bf7852aff5139ab65079feca23f8
 SHA512:
-  metadata.gz: 5e2bcb595827b288a7d6f9d068ff46036f25fc93bd3e39f3f481d8e5eb6ddbe94f23f65dd8554383cfc009dc4ff9ecf19d00a6564aa45181af77971660826bf3
-  data.tar.gz: 147a875753ac204afb7b063f00e10047ec7ef1c8a39c28827ded7d2515b0f6076c418a1e11fb9b3c2ab2a93b2220200bc573fa04f96c9523bf545a90b1f4e223
+  metadata.gz: 431b69e3a82bf76119f8e7b7936042dc3ef64ee6dcdce6d6c6e9244140cab9b4e7fa8e5cfffbd31245e9026af5fe4a127e8166c9261a0a0d6fd80710423bcf88
+  data.tar.gz: 6440d530e3bde18b2645cac87e48c3a987bfdc0992f11b905db068ea51ba3ba955733ba207c3719793e8e71ec6c8eee52a1b93d2e5e6d373ed476ede390a2973

data/README.md

@@ -41,10 +41,10 @@ Add a `.rubocop.yml` file to the root of your project with the following setting
 inherit_gem:
   boxt_ruby_style_guide:
     - default.yml # use default cops
-    - pending.yml # use pending cops 
+    - pending.yml # use pending cops
     - rails.yml # use Rails cops - see Additional Extensions/Cops
-    - rails-pending.yml # use pending rails cops 
-    - rspec.yml # use rspec cops 
+    - rails-pending.yml # use pending rails cops
+    - rspec.yml # use rspec cops
 ```
 
 ### Additional Extensions/Cops
@@ -60,7 +60,7 @@ To enable add the following to your `.rubocop.yml` file.
 ```yml
 inherit_gem:
   boxt_ruby_style_guide:
-  # .... add cops 
+  # .... add cops
 
 require:
   - rubocop-faker # if your project is using the Faker gem then add this
@@ -75,9 +75,17 @@ Lint tasks to run against files listed as changed by Git.
 To run `rubocop` against any changed files use:
 
 ```sh
-rake lint:rubocop
+rake lint:rubocop RUBOCOP_LINT_BASE=your-base-branch
 ```
 
+To run `rubocop` with autofix, use one of the following:
+
+```sh
+rake lint:rubocop_a RUBOCOP_LINT_BASE=your-base-branch
+rake lint:rubocop_A RUBOCOP_LINT_BASE=your-base-branch
+```
+
+
 If there are no changed files the commands will run against all files.
 
 ## Editor Plugins

data/Rakefile

@@ -4,7 +4,7 @@ require "bundler/gem_tasks"
 require "rake/testtask"
 require "rspec/core/rake_task"
 
-import "./tasks/lint.rake"
+import "lib/tasks/lint.rake"
 
 RSpec::Core::RakeTask.new(:spec) do |t|
   t.pattern = Dir.glob("spec/**/*_spec.rb")

data/VERSION

@@ -1 +1 @@
-7.3.0
+7.7.2

data/default.yml

@@ -1,10 +1,10 @@
 AllCops:
   Exclude:
     - "**/*/schema.rb"
-    - Gemfile.lock
-    - node_modules/**/*
-    - tmp/**/*
-    - vendor/**/*
+    - "Gemfile.lock"
+    - "node_modules/**/*"
+    - "tmp/**/*"
+    - "vendor/**/*"
   TargetRubyVersion: 2.7.1
 Layout/LineLength:
   Exclude:

data/lib/boxt_ruby_style_guide.rb

@@ -4,17 +4,13 @@ require "boxt_ruby_style_guide/railtie" if defined?(Rails)
 require "boxt_ruby_style_guide/version"
 
 module BoxtRubyStyleGuide
-  class << self
-    def install_tasks
-      Dir[File.join(gem_spec.gem_dir, "tasks/*.rake")].each do |file|
-        load(file)
-      end
-    end
+  module_function
 
-    private
-
-    def gem_spec
-      Gem::Specification.find_by_name("boxt_ruby_style_guide")
-    end
+  ##
+  # Provide a root path helper for the gem root dir
+  #
+  # Returns Pathname
+  def root
+    Pathname.new(File.dirname(__dir__))
   end
 end

data/lib/boxt_ruby_style_guide/filepath_matcher.rb

@@ -0,0 +1,44 @@
+# frozen_string_literal: true
+
+module BoxtRubyStyleGuide
+  ##
+  # Sanitizes a list of filepaths based on Rubocops exclusions
+  class FilepathMatcher
+    require "yaml"
+
+    ##
+    # Compare a given filepath with a grep-style filename pattern
+    FILEPATH_PATTERN_MATCH = proc do |filepath, pattern|
+      File.fnmatch(pattern, filepath, File::FNM_PATHNAME)
+    end
+
+    ##
+    # Array of file patterns to make sure we only check files that are Ruby files
+    INCLUDE_PATTERNS = %w[
+      Gemfile
+      Rakefile
+      *.gemspec
+      **/*.rb
+      **/*.rake
+    ].freeze
+
+    ##
+    # Array of the excluded files from Rubocop default.yml config
+    EXCLUDE_PATTERNS = YAML.load_file(
+      BoxtRubyStyleGuide.root.join("default.yml")
+    ).dig("AllCops", "Exclude")
+
+    attr_reader :filepaths
+
+    def initialize(*filepaths)
+      @filepaths = filepaths
+    end
+
+    def all_matches
+      filepaths.select do |filepath|
+        filepath_proc = FILEPATH_PATTERN_MATCH.curry.call(filepath)
+        INCLUDE_PATTERNS.any?(filepath_proc) && EXCLUDE_PATTERNS.none?(filepath_proc)
+      end
+    end
+  end
+end

data/lib/boxt_ruby_style_guide/git_diff.rb

@@ -0,0 +1,38 @@
+# frozen_string_literal: true
+
+module BoxtRubyStyleGuide
+  # Returns a list of files that have changed, as detected by `git-diff`
+  #
+  # TODO: Write tests for this to ensure we're pulling the desired diff files
+  # see: https://github.com/ruby-git/ruby-git
+  class GitDiff
+    require "git"
+
+    ##
+    # List of Git statuses we should test
+    # See: https://git-scm.com/docs/git-status#_short_format
+    TEST_STATUSES = %w[M A U].freeze
+
+    attr_reader :base
+
+    def initialize(base = "master")
+      @base = base
+    end
+
+    ##
+    # A list of the local file paths of Ruby files with committed changes.
+    #
+    # Returns Array
+    def all
+      @all ||= begin
+        git.diff(base).name_status.select { |_, stat| TEST_STATUSES.include?(stat) }.keys
+      end
+    end
+
+    private
+
+    def git
+      @git ||= Git.open(".")
+    end
+  end
+end

data/lib/boxt_ruby_style_guide/railtie.rb

@@ -5,7 +5,7 @@ require "rails"
 module BoxtRubyStyleGuide
   class Railtie < Rails::Railtie
     rake_tasks do
-      files = File.join(File.dirname(__FILE__), "../../tasks/*.rake")
+      files = BoxtRubyStyleGuide.root.join("lib", "tasks", "*.rake")
       Dir[files].each { |file| load(file) }
     end
   end

data/lib/tasks/lint.rake

@@ -0,0 +1,55 @@
+# frozen_string_literal: true
+
+require "boxt_ruby_style_guide"
+require "rubocop"
+
+require "boxt_ruby_style_guide/git_diff"
+require "boxt_ruby_style_guide/filepath_matcher"
+
+##
+# The base branch to compare HEAD with for changes
+BASE_BRANCH = "develop"
+
+##
+# Name of the master Rubocop lint task to run
+RUBOCOP_TASK_NAME = :"lint:execute_rubocop"
+
+##
+# Pattern for matching autofix options
+AUTO_REGEX = /\A-a\Z/i.freeze
+
+namespace :lint do
+  desc "Runs rubocop against all files with committed changes different from base branch"
+  task :rubocop do
+    Rake::Task[RUBOCOP_TASK_NAME].invoke
+  end
+
+  desc "Runs rubocop against all files using -a (soft autofix) option"
+  task :rubocop_a do
+    Rake::Task[RUBOCOP_TASK_NAME].invoke("-a")
+  end
+
+  desc "Runs rubocop against all files using -A (hard autofix) option"
+  task :rubocop_A do
+    Rake::Task[RUBOCOP_TASK_NAME].invoke("-A")
+  end
+
+  task :execute_rubocop, [:auto_flag] do |_t, args|
+    if sanitized_file_paths.any?
+      # Sanitize args to make sure only a single "a" or "A" is accepted
+      auto_flag = AUTO_REGEX.match(args[:auto_flag])
+      exec("bundle exec rubocop #{sanitized_file_paths.join(" ")} #{auto_flag}".strip)
+    else
+      puts "No matching Ruby files changed"
+    end
+  end
+end
+
+private
+
+# Returns Array
+def sanitized_file_paths
+  base = ENV.fetch("RUBOCOP_LINT_BASE", BASE_BRANCH)
+  changed_files = BoxtRubyStyleGuide::GitDiff.new(base).all
+  BoxtRubyStyleGuide::FilepathMatcher.new(*changed_files).all_matches
+end

metadata

@@ -1,15 +1,29 @@
 --- !ruby/object:Gem::Specification
 name: boxt_ruby_style_guide
 version: !ruby/object:Gem::Version
-  version: 7.3.0
+  version: 7.7.2
 platform: ruby
 authors:
 - Boxt
-autorequire:
+autorequire: 
 bindir: bin
 cert_chain: []
-date: 2020-09-28 00:00:00.000000000 Z
+date: 2020-10-19 00:00:00.000000000 Z
 dependencies:
+- !ruby/object:Gem::Dependency
+  name: git
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - '='
+      - !ruby/object:Gem::Version
+        version: 1.4.0
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - '='
+      - !ruby/object:Gem::Version
+        version: 1.4.0
 - !ruby/object:Gem::Dependency
   name: rubocop
   requirement: !ruby/object:Gem::Requirement
@@ -24,6 +38,26 @@ dependencies:
     - - '='
       - !ruby/object:Gem::Version
         version: 0.92.0
+- !ruby/object:Gem::Dependency
+  name: rails
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">"
+      - !ruby/object:Gem::Version
+        version: '5'
+    - - "<"
+      - !ruby/object:Gem::Version
+        version: '7'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">"
+      - !ruby/object:Gem::Version
+        version: '5'
+    - - "<"
+      - !ruby/object:Gem::Version
+        version: '7'
 - !ruby/object:Gem::Dependency
   name: rubocop-faker
   requirement: !ruby/object:Gem::Requirement
@@ -80,6 +114,34 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '2.1'
+- !ruby/object:Gem::Dependency
+  name: bundler-audit
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 0.7.0.1
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 0.7.0.1
+- !ruby/object:Gem::Dependency
+  name: 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
@@ -128,14 +190,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.18'
+        version: '0.19'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.18'
+        version: '0.19'
 description: Ruby styleguide info for the BOXT projects, as well as config settings
   for Rubocop
 email:
@@ -150,8 +212,11 @@ files:
 - VERSION
 - default.yml
 - lib/boxt_ruby_style_guide.rb
+- lib/boxt_ruby_style_guide/filepath_matcher.rb
+- lib/boxt_ruby_style_guide/git_diff.rb
 - lib/boxt_ruby_style_guide/railtie.rb
 - lib/boxt_ruby_style_guide/version.rb
+- lib/tasks/lint.rake
 - pending.yml
 - rails-pending.yml
 - rails.yml
@@ -160,7 +225,7 @@ homepage: https://github.com/boxt/ruby-style-guide
 licenses:
 - MIT
 metadata: {}
-post_install_message:
+post_install_message: 
 rdoc_options: []
 require_paths:
 - lib
@@ -175,8 +240,8 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.1.4
-signing_key:
+rubygems_version: 3.1.2
+signing_key: 
 specification_version: 4
 summary: Ruby styleguide info for the BOXT Ruby projects
 test_files: []