rails_best_practices 1.20.1 → 1.23.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: e7a2b67944c27b7a11e48f2c490a4ea1080fc4df39ad1d59b20cdeaedfb4b083
-  data.tar.gz: d96f6d7da9909bac895162f70ae1247a98a8df39da7fd2ce17365e959568c4f6
+  metadata.gz: 6130c27d1f5ad49baeed9bd6595188a99f77e10260f709e7b2dc49bce191d06e
+  data.tar.gz: 856313c260d0e66d2625885bdb6f4f1614fa2184303a182d7fb016fb76dac304
 SHA512:
-  metadata.gz: 244acdd4067af5f87364236d4fe5bc40d5a4d68b98d1c6786aaf0e53c7766626ef670c4aaa61d793a47bba643fddcd2cbd545a4697dff8d0a3d6f7238912c2b5
-  data.tar.gz: 9b4ca4c95172317d9017691399fcb9211295bad98959477fa8fa24a054a7a6283dc7cdbabd926e925ccf0f3ea9bfe1dde3e594cc42a2db7bd3b37758cb7bdc6e
+  metadata.gz: 66c463631584235d020ea1ee12d748119827de1c9347660ccf5fde82191038ab75cd07f9266933c199a2fd6c722abf566eb75ec906f1c717a557c4e19e327278
+  data.tar.gz: 453991a40ccdf9f0549dbe6842e8823034a41e5b253efdccde0e4442a0ea17b654046747dea4de54ac4312eae89bd4ec946c45ded98c356d7f12465294b34d8d

data/.github/workflows/main.yml

@@ -0,0 +1,31 @@
+# This workflow uses actions that are not certified by GitHub.
+# They are provided by a third-party and are governed by
+# separate terms of service, privacy policy, and support
+# documentation.
+# This workflow will download a prebuilt Ruby version, install dependencies and run tests with Rake
+# For more information see: https://github.com/marketplace/actions/setup-ruby-jruby-and-truffleruby
+
+name: CI
+
+on:
+  push:
+    branches: [ master ]
+  pull_request:
+    branches: [ master ]
+
+jobs:
+  test:
+    runs-on: ubuntu-latest
+    strategy:
+      matrix:
+        ruby-version: ['2.6', '2.7', '3.0', '3.1']
+
+    steps:
+    - uses: actions/checkout@v2
+    - name: Set up Ruby
+      uses: ruby/setup-ruby@v1
+      with:
+        ruby-version: ${{ matrix.ruby-version }}
+        bundler-cache: true
+    - name: Run tests
+      run: bundle exec rspec spec

data/.ruby-version

@@ -0,0 +1 @@
+3.1.0

data/CHANGELOG.md

@@ -1,5 +1,18 @@
 # Next Release
 
+## 1.23.0 (2022-01-31)
+
+* Support hash short syntax
+
+## 1.22.0 (2021-10-27)
+
+* Support atom editor
+
+## 1.21.0 (2021-06-15)
+
+* Disable check by inline comment
+* Add type check for string
+
 ## 1.20.0 (2020-02-29)
 
 * Support ruby 2.7

data/Gemfile

@@ -8,4 +8,3 @@ gem 'coveralls', require: false
 gem 'guard'
 gem 'guard-rspec'
 gem 'pry'
-gem 'rspec'

data/Gemfile.lock

@@ -1,9 +1,9 @@
 PATH
   remote: .
   specs:
-    rails_best_practices (1.20.1)
-      activesupport
-      code_analyzer (>= 0.5.2)
+    rails_best_practices (1.23.0)
+      activesupport (~> 6.1.4)
+      code_analyzer (~> 0.5.5)
       erubis
       i18n
       json
@@ -13,79 +13,81 @@ PATH
 GEM
   remote: https://rubygems.org/
   specs:
-    activesupport (6.1.3.2)
+    activesupport (6.1.4.4)
       concurrent-ruby (~> 1.0, >= 1.0.2)
       i18n (>= 1.6, < 2)
       minitest (>= 5.1)
       tzinfo (~> 2.0)
       zeitwerk (~> 2.3)
     awesome_print (1.8.0)
-    code_analyzer (0.5.2)
+    code_analyzer (0.5.5)
       sexp_processor
-    coderay (1.1.2)
-    concurrent-ruby (1.1.8)
+    coderay (1.1.3)
+    concurrent-ruby (1.1.9)
     coveralls (0.8.23)
       json (>= 1.8, < 3)
       simplecov (~> 0.16.1)
       term-ansicolor (~> 1.3)
       thor (>= 0.19.4, < 2.0)
       tins (~> 1.6)
-    diff-lcs (1.3)
+    diff-lcs (1.4.4)
     docile (1.3.2)
     erubis (2.7.0)
-    ffi (1.11.3)
-    formatador (0.2.5)
-    guard (2.16.1)
+    ffi (1.15.4)
+    formatador (0.3.0)
+    guard (2.18.0)
       formatador (>= 0.2.4)
       listen (>= 2.7, < 4.0)
       lumberjack (>= 1.0.12, < 2.0)
       nenv (~> 0.1)
       notiffany (~> 0.0)
-      pry (>= 0.9.12)
+      pry (>= 0.13.0)
       shellany (~> 0.0)
       thor (>= 0.18.1)
-    guard-rspec (4.3.1)
+    guard-compat (1.2.1)
+    guard-rspec (4.7.3)
       guard (~> 2.1)
-      rspec (>= 2.14, < 4.0)
+      guard-compat (~> 1.1)
+      rspec (>= 2.99.0, < 4.0)
     haml (5.1.2)
       temple (>= 0.8.0)
       tilt
-    i18n (1.8.10)
+    i18n (1.9.1)
       concurrent-ruby (~> 1.0)
     json (2.3.0)
-    listen (3.2.1)
+    listen (3.7.0)
       rb-fsevent (~> 0.10, >= 0.10.3)
       rb-inotify (~> 0.9, >= 0.9.10)
-    lumberjack (1.0.13)
-    method_source (0.9.2)
-    minitest (5.14.4)
+    lumberjack (1.2.8)
+    method_source (1.0.0)
+    minitest (5.15.0)
     nenv (0.3.0)
     notiffany (0.1.3)
       nenv (~> 0.1)
       shellany (~> 0.0)
-    pry (0.12.2)
-      coderay (~> 1.1.0)
-      method_source (~> 0.9.0)
+    pry (0.14.1)
+      coderay (~> 1.1)
+      method_source (~> 1.0)
     rake (13.0.1)
-    rb-fsevent (0.10.3)
+    rb-fsevent (0.11.0)
     rb-inotify (0.10.1)
       ffi (~> 1.0)
     require_all (3.0.0)
-    rspec (3.9.0)
-      rspec-core (~> 3.9.0)
-      rspec-expectations (~> 3.9.0)
-      rspec-mocks (~> 3.9.0)
-    rspec-core (3.9.1)
-      rspec-support (~> 3.9.1)
-    rspec-expectations (3.9.1)
+    rspec (3.10.0)
+      rspec-core (~> 3.10.0)
+      rspec-expectations (~> 3.10.0)
+      rspec-mocks (~> 3.10.0)
+    rspec-core (3.10.1)
+      rspec-support (~> 3.10.0)
+    rspec-expectations (3.10.1)
       diff-lcs (>= 1.2.0, < 2.0)
-      rspec-support (~> 3.9.0)
-    rspec-mocks (3.9.1)
+      rspec-support (~> 3.10.0)
+    rspec-mocks (3.10.2)
       diff-lcs (>= 1.2.0, < 2.0)
-      rspec-support (~> 3.9.0)
-    rspec-support (3.9.2)
+      rspec-support (~> 3.10.0)
+    rspec-support (3.10.2)
     ruby-progressbar (1.11.0)
-    sexp_processor (4.15.2)
+    sexp_processor (4.16.0)
     shellany (0.0.1)
     simplecov (0.16.1)
       docile (~> 1.1)
@@ -98,12 +100,12 @@ GEM
     temple (0.8.2)
     term-ansicolor (1.7.1)
       tins (~> 1.0)
-    thor (1.0.1)
+    thor (1.1.0)
     tilt (2.0.10)
     tins (1.22.2)
     tzinfo (2.0.4)
       concurrent-ruby (~> 1.0)
-    zeitwerk (2.4.2)
+    zeitwerk (2.5.4)
 
 PLATFORMS
   ruby
@@ -122,4 +124,4 @@ DEPENDENCIES
   slim
 
 BUNDLED WITH
-   2.1.4
+   2.2.22

data/README.md

@@ -284,7 +284,7 @@ Follow us on twitter: <http://twitter.com/railsbp>
 Send us email: <team@railsbp.com>
 
 
-Copyright © 2009 - 2013 Richard Huang (flyerhzm@gmail.com), released under the MIT license
+Copyright © 2009 - 2022 Richard Huang (flyerhzm@gmail.com), released under the MIT license
 
 
 [1]: https://github.com/railsbp/rails_best_practices/wiki/How-to-write-your-own-check-list

data/assets/result.html.erb

@@ -115,6 +115,8 @@
             <td class='filename'>
               <% if @github %>
                 <a href='<%= @github_name %>/blob/<%= @last_commit_id %>/<%= error.short_filename %>#L<%= error.first_line_number %>' target='_blank'><%= error.short_filename %></a>
+              <% elsif @atom %>
+                <a href='atom://core/open/file?filename=<%= File.expand_path(error.filename) %>&amp;line=<%= error.line_number %>'><%= error.short_filename %></a>
               <% elsif @textmate %>
                 <a href='txmt://open/?url=file://<%= File.expand_path(error.filename) %>&amp;line=<%= error.line_number %>'><%= error.short_filename %></a>
               <% elsif @sublime %>

data/lib/rails_best_practices/analyzer.rb

@@ -249,6 +249,7 @@ module RailsBestPractices
         file.puts eruby.evaluate(
                     errors: errors,
                     error_types: error_types,
+                    atom: @options['with-atom'],
                     textmate: @options['with-textmate'],
                     vscode: @options['with-vscode'],
                     sublime: @options['with-sublime'],
@@ -326,8 +327,8 @@ module RailsBestPractices
 
     # analyze source codes.
     def analyze_source_codes
-      @bar = ProgressBar.create(title: 'Source Code', total: parse_files.size * 3) if display_bar?
-      %w[lexical prepare review].each { |process| send(:process, process) }
+      @bar = ProgressBar.create(title: 'Source Code', total: parse_files.size * 4) if display_bar?
+      %w[lexical prepare review inline_disable].each { |process| send(:process, process) }
       @bar.finish if display_bar?
     end
 

data/lib/rails_best_practices/core/check.rb

@@ -204,7 +204,7 @@ module RailsBestPractices
                               :command_call,
                               :command,
                               :alias,
-                              :bare_assoc_hash,
+                              :assoc_new,
                               :method_add_arg
 
             # remembe the message of call node.
@@ -269,9 +269,12 @@ module RailsBestPractices
             #     def to_xml(options = {})
             #       super options.merge(exclude: :visible, methods: [:is_discussion_conversation])
             #     end
-            add_callback :start_bare_assoc_hash do |node|
-              if node.hash_keys.include? 'methods'
-                mark_used(node.hash_value('methods'))
+            add_callback :start_assoc_new do |node|
+              if node.key == 'methods'
+                mark_used(node.value)
+              end
+              if node.value.nil?
+                mark_used(node.key)
               end
             end
 
@@ -294,7 +297,9 @@ module RailsBestPractices
             def mark_used(method_node)
               return if method_node == :call
 
-              if method_node.sexp_type == :bare_assoc_hash
+              if method_node.is_a?(String)
+                method_name = method_node
+              elsif method_node.sexp_type == :bare_assoc_hash
                 method_node.hash_values.each { |value_node| mark_used(value_node) }
               elsif method_node.sexp_type == :array
                 method_node.array_values.each { |value_node| mark_used(value_node) }

data/lib/rails_best_practices/core/runner.rb

@@ -65,6 +65,9 @@ module RailsBestPractices
             checkers: @prepares.reject { |checker| checker.is_a? Prepares::GemfilePrepare }
           )
         @review_checker ||= CodeAnalyzer::CheckingVisitor::Default.new(checkers: @reviews)
+
+        @inlnie_disable ||= InlineDisables::InlineDisable.new
+        @inline_disable_checker ||= CodeAnalyzer::CheckingVisitor::Plain.new(checkers: [@inlnie_disable])
       end
 
       # lexical analysis the file.
@@ -106,11 +109,27 @@ module RailsBestPractices
         @review_checker.after_check
       end
 
+      # disable check by inline comment the file.
+      #
+      # @param [String] filename of the file
+      # @param [String] content of the file
+      def inline_disable(filename, content)
+        content = parse_html_template(filename, content)
+        @inline_disable_checker.check(filename, content)
+      end
+
+      def after_inline_disable
+        @inline_disable_checker.after_check
+      end
+
       # get all errors from lexicals and reviews.
       #
       # @return [Array] all errors from lexicals and reviews
       def errors
-        @errors ||= (@reviews + @lexicals).collect(&:errors).flatten
+        @errors ||= begin
+          reported_errors = (@reviews + @lexicals).collect(&:errors).flatten
+          reported_errors.reject { |error| @inlnie_disable.disabled?(error) }
+        end
       end
 
       private

data/lib/rails_best_practices/inline_disables/comment_ripper.rb

@@ -0,0 +1,19 @@
+# frozen_string_literal: true
+
+module RailsBestPractices
+  module InlineDisables
+    class CommentRipper < Ripper::SexpBuilder
+      attr_reader :comments
+
+      def initialize(*arg)
+        super
+        @comments = []
+      end
+
+      def on_comment(*arg)
+        # [sexp_type, statement, [lineno, column]] = super
+        comments << super
+      end
+    end
+  end
+end

data/lib/rails_best_practices/inline_disables/inline_disable.rb

@@ -0,0 +1,50 @@
+# frozen_string_literal: true
+
+module RailsBestPractices
+  module InlineDisables
+    class InlineDisable < Core::Check
+      interesting_files ALL_FILES
+      url '#'
+
+      def initialize(*args)
+        super
+        @disabled_errors = []
+      end
+
+      def check(filename, content)
+        comments = CommentRipper.new(content).tap(&:parse).comments
+        comments.each do |_sexp_type, statement, (line_number, _column)|
+          add_as_disable_errors(filename, statement, line_number)
+        end
+      end
+
+      def disabled?(error)
+        error_key = [error.filename, error.line_number, error.type.split('::').last].join('-')
+        disabled_error_keys.include?(error_key)
+      end
+
+      private
+
+      def disabled_error_keys
+        @disabled_error_keys ||= Set.new(@disabled_errors.map { |e| [e.filename, e.line_number, e.type].join('-') })
+      end
+
+      def add_as_disable_errors(filename, statement, line_number)
+        match = statement.match(/rails_b(?:est_)?p(?:ractices)?:disable (.*)/)
+        return unless match
+
+        check_names = match[1].split(',')
+        check_names.each do |check_name|
+          add_as_disable_error(filename, check_name.gsub(/Check$/, 'Review'), line_number)
+        end
+      end
+
+      def add_as_disable_error(filename, check_name, line_number)
+        @disabled_errors <<
+          RailsBestPractices::Core::Error.new(
+            filename: filename, line_number: line_number, message: 'disable by inline comment', type: check_name, url: url
+          )
+      end
+    end
+  end
+end

data/lib/rails_best_practices/inline_disables.rb

@@ -0,0 +1,3 @@
+# frozen_string_literal: true
+
+require_rel 'inline_disables'

data/lib/rails_best_practices/option_parser.rb

@@ -10,6 +10,7 @@ module RailsBestPractices
     #    -f, --format FORMAT              output format (text, html, yaml, json, xml)
     #        --output-file FILE           output html file for the analyzing result
     #        --without-color              only output plain text without color
+    #        --with-atom                  open file by atom in html format
     #        --with-textmate              open file by textmate in html format
     #        --with-vscode                open file by vscode in html format
     #        --with-sublime               open file by sublime in html format (requires subl-handler)
@@ -49,6 +50,10 @@ module RailsBestPractices
           options['without-color'] = true
         end
 
+        opts.on('--with-atom', 'open file by atom in html format') do
+          options['with-atom'] = true
+        end
+
         opts.on('--with-textmate', 'open file by textmate in html format') do
           options['with-textmate'] = true
         end

data/lib/rails_best_practices/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module RailsBestPractices
-  VERSION = '1.20.1'
+  VERSION = '1.23.0'
 end

data/lib/rails_best_practices.rb

@@ -8,6 +8,7 @@ require 'rails_best_practices/analyzer'
 require 'rails_best_practices/lexicals'
 require 'rails_best_practices/prepares'
 require 'rails_best_practices/reviews'
+require 'rails_best_practices/inline_disables'
 require 'rails_best_practices/option_parser'
 require 'rails_best_practices/cli'
 

data/rails_best_practices.gemspec

@@ -1,6 +1,8 @@
 # frozen_string_literal: true
 
-require File.expand_path('lib/rails_best_practices/version', __dir__)
+lib = File.expand_path('lib', __dir__)
+$LOAD_PATH.unshift(lib) unless $LOAD_PATH.include?(lib)
+require 'rails_best_practices/version'
 
 Gem::Specification.new do |spec|
   spec.name        = 'rails_best_practices'
@@ -16,13 +18,13 @@ Gem::Specification.new do |spec|
   spec.required_ruby_version = '>= 1.9.0'
   spec.required_rubygems_version = '>= 1.3.6'
 
-  spec.add_dependency('activesupport')
-  spec.add_dependency('code_analyzer', '>= 0.5.2')
-  spec.add_dependency('erubis')
-  spec.add_dependency('i18n')
-  spec.add_dependency('json')
-  spec.add_dependency('require_all', '~> 3.0')
-  spec.add_dependency('ruby-progressbar')
+  spec.add_runtime_dependency('activesupport', '~> 6.1.4')
+  spec.add_runtime_dependency('code_analyzer', '~> 0.5.5')
+  spec.add_runtime_dependency('erubis')
+  spec.add_runtime_dependency('i18n')
+  spec.add_runtime_dependency('json')
+  spec.add_runtime_dependency('require_all', '~> 3.0')
+  spec.add_runtime_dependency('ruby-progressbar')
 
   spec.add_development_dependency('awesome_print')
   spec.add_development_dependency('bundler')

data/spec/rails_best_practices/inline_disables/inline_disable_spec.rb

@@ -0,0 +1,62 @@
+# frozen_string_literal: true
+
+require 'spec_helper'
+
+module RailsBestPractices
+  module InlineDisables
+    describe InlineDisable do
+      let(:runner) { Core::Runner.new(reviews: Reviews::MoveModelLogicIntoModelReview.new) }
+
+      it 'moves model logic into model' do
+        content = <<-EOF
+        class PostsController < ApplicationController
+          def publish
+            @post = Post.find(params[:id])
+            @post.update_attributes(:is_published, true)
+            @post.approved_by = current_user
+
+            if @post.created_at > Time.now - 7.days
+              @post.popular = 100
+            else
+              @post.popular = 0
+            end
+
+            redirect_to post_url(@post)
+          end
+        end
+        EOF
+        runner.review('app/controllers/posts_controller.rb', content)
+        runner.inline_disable('app/controllers/posts_controller.rb', content)
+
+        expect(runner.errors.size).to eq(1)
+        expect(runner.errors[0].to_s).to eq(
+          'app/controllers/posts_controller.rb:2 - move model logic into model (@post use_count > 4)'
+        )
+      end
+
+      it 'is no error is output' do
+        content = <<-EOF
+        class PostsController < ApplicationController
+          def publish # rails_best_practices:disable MoveModelLogicIntoModelCheck
+            @post = Post.find(params[:id])
+            @post.update_attributes(:is_published, true)
+            @post.approved_by = current_user
+
+            if @post.created_at > Time.now - 7.days
+              @post.popular = 100
+            else
+              @post.popular = 0
+            end
+
+            redirect_to post_url(@post)
+          end
+        end
+        EOF
+        runner.review('app/controllers/posts_controller.rb', content)
+        runner.inline_disable('app/controllers/posts_controller.rb', content)
+
+        expect(runner.errors.size).to eq(0)
+      end
+    end
+  end
+end

data/spec/rails_best_practices/reviews/remove_unused_methods_in_models_review_spec.rb

@@ -556,6 +556,28 @@ module RailsBestPractices
         end
       end
 
+      if RUBY_VERSION.to_f > 3.0
+        context 'short syntax value' do
+          it 'does not remove unused method' do
+            content = <<-EOF
+            class Post < ActiveRecord::Base
+              def build
+                new(value:)
+              end
+
+              def value
+                'value'
+              end
+            end
+            EOF
+            runner.prepare('app/models/post.rb', content)
+            runner.review('app/models/post.rb', content)
+            runner.after_review
+            expect(runner.errors.size).to eq(1)
+          end
+        end
+      end
+
       context 'callbacks' do
         it 'does not remove unused method' do
           content = <<-EOF

metadata

@@ -1,43 +1,43 @@
 --- !ruby/object:Gem::Specification
 name: rails_best_practices
 version: !ruby/object:Gem::Version
-  version: 1.20.1
+  version: 1.23.0
 platform: ruby
 authors:
 - Richard Huang
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2021-05-08 00:00:00.000000000 Z
+date: 2022-01-30 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: 6.1.4
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: 6.1.4
 - !ruby/object:Gem::Dependency
   name: code_analyzer
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.5.2
+        version: 0.5.5
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.5.2
+        version: 0.5.5
 - !ruby/object:Gem::Dependency
   name: erubis
   requirement: !ruby/object:Gem::Requirement
@@ -200,9 +200,10 @@ executables:
 extensions: []
 extra_rdoc_files: []
 files:
+- ".github/workflows/main.yml"
 - ".gitignore"
 - ".rspec"
-- ".travis.yml"
+- ".ruby-version"
 - CHANGELOG.md
 - Gemfile
 - Gemfile.lock
@@ -235,6 +236,9 @@ files:
 - lib/rails_best_practices/core/routes.rb
 - lib/rails_best_practices/core/runner.rb
 - lib/rails_best_practices/core_ext/erubis.rb
+- lib/rails_best_practices/inline_disables.rb
+- lib/rails_best_practices/inline_disables/comment_ripper.rb
+- lib/rails_best_practices/inline_disables/inline_disable.rb
 - lib/rails_best_practices/lexicals.rb
 - lib/rails_best_practices/lexicals/long_line_check.rb
 - lib/rails_best_practices/lexicals/remove_tab_check.rb
@@ -314,6 +318,7 @@ files:
 - spec/rails_best_practices/core/routes_spec.rb
 - spec/rails_best_practices/core/runner_spec.rb
 - spec/rails_best_practices/core_ext/erubis_spec.rb
+- spec/rails_best_practices/inline_disables/inline_disable_spec.rb
 - spec/rails_best_practices/lexicals/long_line_check_spec.rb
 - spec/rails_best_practices/lexicals/remove_tab_check_spec.rb
 - spec/rails_best_practices/lexicals/remove_trailing_whitespace_check_spec.rb
@@ -399,7 +404,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: 1.3.6
 requirements: []
-rubygems_version: 3.1.6
+rubygems_version: 3.3.3
 signing_key:
 specification_version: 4
 summary: a code metric tool for rails codes.
@@ -424,6 +429,7 @@ test_files:
 - spec/rails_best_practices/core/routes_spec.rb
 - spec/rails_best_practices/core/runner_spec.rb
 - spec/rails_best_practices/core_ext/erubis_spec.rb
+- spec/rails_best_practices/inline_disables/inline_disable_spec.rb
 - spec/rails_best_practices/lexicals/long_line_check_spec.rb
 - spec/rails_best_practices/lexicals/remove_tab_check_spec.rb
 - spec/rails_best_practices/lexicals/remove_trailing_whitespace_check_spec.rb

data/.travis.yml

@@ -1,4 +0,0 @@
-language: ruby
-rvm:
-  - 2.6.4
-script: bundle exec rspec spec