boxt_ruby_style_guide 7.6.0 → 7.9.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 7fdb78910aef325a37d80abf635b2c675d190e63669133b14f19ebcad1c480d9
-  data.tar.gz: 5c58cb56cfeb23d4769f8681bd21c16b5f39a3401ecbb6a016b3ba7cdf1c68f7
+  metadata.gz: df02aaff031561dd78d64ead4f814ff7b3dca32ed71c6e5ca80d0921a7a57ba6
+  data.tar.gz: c9e037578d06d41dafeaf714db0f1c2db9de4a63ab718a2188a943090301885c
 SHA512:
-  metadata.gz: be5aa640996292e3d526e91a27fb2365eebb84c2e2c02f32f0a1d2b47dbe04340e0e11f15a30d9b6c60a2b25a56430dce48a950cf2522b227ce0f51acb99c58d
-  data.tar.gz: 667f275e3560ec91b6c20ca8bdd6f58cfc7f079c1207812d891af364e5a086823e0f7679c8e0537598c9dd4f03d230c7853b8828787a481eb2a8e5b023adfc28
+  metadata.gz: 70be516ec5d55e77c4b0afde9fccff43f5ec2cc3f32dd57e14fca433c1e4cac544f655361994bc4e891db2d7090122e1c2c1165684776ac26f4a62c4b91344c5
+  data.tar.gz: b727e3d6d32dc4d329f559c0bf7a482408c1f166e9f3868bd511c5734f4c740539710cdd55a99bad327a3d18cc291f1a057509283483b562b9df7227f1c481a9

data/README.md

@@ -24,13 +24,6 @@ And then execute:
 bundle
 ```
 
-To make the lint rake tasks available to non Rails apps and gems, add the following to the project's `Rakefile`:
-
-```ruby
-require "boxt_ruby_style_guide"
-BoxtRubyStyleGuide.install_tasks
-```
-
 Rails apps should have access to the lint tasks by default.
 
 ## Usage
@@ -41,10 +34,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 +53,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 +68,17 @@ Lint tasks to run against files listed as changed by Git.
 To run `rubocop` against any changed files use:
 
 ```sh
-rake lint:rubocop
+RUBOCOP_LINT_BASE=your-base-branch rake lint:rubocop 
 ```
 
+To run `rubocop` with autofix, use one of the following:
+
+```sh
+RUBOCOP_LINT_BASE=your-base-branch rake lint:rubocop -a 
+RUBOCOP_LINT_BASE=your-base-branch rake lint:rubocop -A
+```
+
+
 If there are no changed files the commands will run against all files.
 
 ## Editor Plugins

data/Rakefile

@@ -10,4 +10,5 @@ RSpec::Core::RakeTask.new(:spec) do |t|
   t.pattern = Dir.glob("spec/**/*_spec.rb")
 end
 
+desc "Map rake test to rake spec"
 task test: :spec

data/VERSION

@@ -1 +1 @@
-7.6.0
+7.9.0

data/default.yml

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

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:)
+      @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/tasks/lint.rake

@@ -1,64 +1,30 @@
 # frozen_string_literal: true
 
 require "boxt_ruby_style_guide"
+require "boxt_ruby_style_guide/filepath_matcher"
+require "boxt_ruby_style_guide/git_diff"
 require "rubocop"
 
-##
-# The base branch to compare HEAD with for changes
-BASE_BRANCH = "develop"
-
-##
-# Directories containing Ruby files to lint
-RUBY_DIRS = %w[app lib test spec].freeze
-
-##
-# Grep file pattern to make sure we only check files that are Ruby files
-# This pattern matches the following:
-#
-# - Gemfile
-# - Rakefile
-# - *.gemspec
-# - app/**/*.{rb,rake}
-# - lib/**/*.{rb,rake}
-# - test/**/*.{rb,rake}
-# - spec/**/*.{rb,rake}
-#
-GREP_PATTERN = <<~STRING.delete("\n").delete("\s")
-  (
-    (#{RUBY_DIRS.join('|')})\\/.+\\.(rb|rake)
-    |^
-    (Gemfile|Rakefile|.+\\.gemspec)
-  \Z)
-STRING
-
 namespace :lint do
   desc "Runs rubocop against all files with committed changes different from base branch"
   task :rubocop do
-    exec(<<~BASH)
-      RUBOCOP_CHANGED_FILES=`#{diff_file_paths}`
-      if [ -z "$RUBOCOP_CHANGED_FILES" ]; then
-        echo "No matching Ruby files changed"
-      else
-        bundle exec rubocop $RUBOCOP_CHANGED_FILES
-      fi
-    BASH
+    file_paths = sanitized_file_paths
+    puts "File paths: #{file_paths.join(', ')}"
+
+    if file_paths.any?
+      auto_flag_opt = ARGV.select { |a| ["-a", "-A"].include?(a) }.first
+      exec("bundle exec rubocop #{file_paths.join(' ')} #{auto_flag_opt}".strip)
+    else
+      puts "No Ruby files changed"
+    end
   end
+end
 
-  private
+private
 
-  # A list of the local file paths of Ruby files with committed changes.
-  #
-  #   Run a git diff-tree command with the following otions:
-  #     -r recursive
-  #     --name-only Only return the name of the files
-  #     --diff-filter Filter out results that have been deleted on HEAD
-  #
-  #   Pipe the output from this command through grep to match only Ruby files in the
-  #     desired directories
-  #
-  # Returns String
-  def diff_file_paths
-    base_branch = ENV.fetch("RUBOCOP_LINT_BASE", BASE_BRANCH)
-    "git diff -r --name-only --diff-filter=d #{base_branch} | egrep '#{GREP_PATTERN}'"
-  end
+# Returns an array of files
+def sanitized_file_paths
+  base = ENV.fetch("RUBOCOP_LINT_BASE", "main")
+  changed_files = BoxtRubyStyleGuide::GitDiff.new(base: base).all
+  BoxtRubyStyleGuide::FilepathMatcher.new(*changed_files).all_matches
 end

metadata

@@ -1,49 +1,43 @@
 --- !ruby/object:Gem::Specification
 name: boxt_ruby_style_guide
 version: !ruby/object:Gem::Version
-  version: 7.6.0
+  version: 7.9.0
 platform: ruby
 authors:
 - Boxt
-autorequire: 
+autorequire:
 bindir: bin
 cert_chain: []
-date: 2020-10-12 00:00:00.000000000 Z
+date: 2021-02-02 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
-  name: rubocop
+  name: git
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - '='
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.92.0
+        version: '1.4'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - '='
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.92.0
+        version: '1.4'
 - !ruby/object:Gem::Dependency
-  name: rails
+  name: rubocop
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ">"
-      - !ruby/object:Gem::Version
-        version: '5'
-    - - "<"
+    - - '='
       - !ruby/object:Gem::Version
-        version: '7'
-  type: :development
+        version: 1.9.1
+  type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ">"
-      - !ruby/object:Gem::Version
-        version: '5'
-    - - "<"
+    - - '='
       - !ruby/object:Gem::Version
-        version: '7'
+        version: 1.9.1
 - !ruby/object:Gem::Dependency
   name: rubocop-faker
   requirement: !ruby/object:Gem::Requirement
@@ -64,28 +58,42 @@ dependencies:
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 2.8.1
+        version: 2.9.1
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - '='
+      - !ruby/object:Gem::Version
+        version: 2.9.1
+- !ruby/object:Gem::Dependency
+  name: rubocop-rake
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - '='
+      - !ruby/object:Gem::Version
+        version: 0.5.1
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 2.8.1
+        version: 0.5.1
 - !ruby/object:Gem::Dependency
   name: rubocop-rspec
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 1.43.2
+        version: 2.1.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 1.43.2
+        version: 2.1.0
 - !ruby/object:Gem::Dependency
   name: bundler
   requirement: !ruby/object:Gem::Requirement
@@ -100,6 +108,54 @@ 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: 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: rake
   requirement: !ruby/object:Gem::Requirement
@@ -170,6 +226,8 @@ 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
@@ -181,7 +239,7 @@ homepage: https://github.com/boxt/ruby-style-guide
 licenses:
 - MIT
 metadata: {}
-post_install_message: 
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -196,8 +254,8 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.1.2
-signing_key: 
+rubygems_version: 3.1.4
+signing_key:
 specification_version: 4
 summary: Ruby styleguide info for the BOXT Ruby projects
 test_files: []