standard 0.12.0 → 1.0.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: ece6905a02f43b8e9787ca829e9940f6cd811ce19cc5c73aaa29b39308e86de3
-  data.tar.gz: 420fa134ef9a26d7ab7d6ce72cebb28ca951d03795e95fdf43072af4b5039848
+  metadata.gz: b1ffe5112549df3b2c10c013937dc42d459d6ee098971d214ece1d00482d3b97
+  data.tar.gz: e796e1c215cc354d9098583b5d1f69860852096c79d293ff935b9c4d3fda9fe6
 SHA512:
-  metadata.gz: 70f35192a65faf716fed5b62ebafd6c6de6e8d54e06e38a8d07bd071b2cff8ff4aab81b8cb9089fd7941ff14b19edf19a37b5468e1eb9eaa7a29e0ece6b925d5
-  data.tar.gz: 00b5ed175cc6e8a47cee92b132dd1a9bbd32f1a2d21284bec0c07be75341eeedc6a44761b1f5f7978517ab4830704b5324df9c28b110371c81ed308589b7641a
+  metadata.gz: 7925123a27c8140624f9ec4903f3546c7e7aa9e0ba48244f362e570ed4a4a2a34304621f78e964d6445e15ef1046ee4fd410e80e05ff308886be1d77d2324d6c
+  data.tar.gz: f00df60430fabfddd8e2f389a5b7e4d8701d2ce0b0ca92c828e28331d64ee5ca2616053a04094f23ab3d5a340e276561be6a97d0f21bd351d407e2bd94b2598e

data/CHANGELOG.md

@@ -1,11 +1,44 @@
 # Changelog
 
+## 1.0.3
+
+* Fix an exit code bug introduced in 1.0.2
+  ([#271](https://github.com/testdouble/standard/pull/272)
+
+## 1.0.2
+
+* Preserve RuboCop's CLI exit codes
+  ([#270](https://github.com/testdouble/standard/pull/270)) by
+  [@nicksieger](https://github.com/nicksieger)
+
+## 1.0.1
+
+* Update rubocop from 1.10.0 to [1.11.0](https://github.com/rubocop-hq/rubocop/releases/tag/v1.11.0)
+* Update rubocop-performance from 1.9.2 to [1.10.1](https://github.com/rubocop-hq/rubocop-performance/releases/tag/v1.10.1)
+
+## 1.0.0
+
+* Relax multi-line block rules, moving away from enforcing semantic blocks to
+  instead allowing code to adhere to whatever multi-line format the author deems
+  best [#263](https://github.com/testdouble/standard/pull/263)
+* Allow a `standard:disable` comment directive in addition to `rubocop:disable`
+  [#186](https://github.com/testdouble/standard/pull/186)
+* Remove the banner text that standard outputs after failure
+  [#264](https://github.com/testdouble/standard/pull/264)
+
+## 0.13.0
+
+* Update rubocop from 1.7.0 to [1.10.0](https://github.com/rubocop-hq/rubocop/releases/tag/v1.10.0) enabling:
+  * [`Lint/AmbiguousAssignment`](https://github.com/rubocop-hq/rubocop/issues/9223)
+  * [`Style/HashExcept`](https://github.com/rubocop-hq/rubocop/pull/9283)
+  * [`Lint/DeprecatedConstants`](https://github.com/rubocop-hq/rubocop/pull/9324)
+
 ## 0.12.0
 
 * Update rubocop from 1.7.0 to [1.8.1](https://github.com/rubocop-hq/rubocop/releases/tag/v1.8.1)
 * Enabled [`Style/SlicingWithRange`](https://github.com/testdouble/standard/issues/175)
 
-## 0.11.0 
+## 0.11.0
 
 * Update rubocop-performance from 1.9.1 to [1.9.2](https://github.com/rubocop-hq/rubocop-performance/releases/tag/v1.9.2)
 * Update rubocop from 1.4.2 to [1.7.0](https://github.com/rubocop-hq/rubocop/releases/tag/v1.7.0)

data/Gemfile.lock

@@ -1,9 +1,9 @@
 PATH
   remote: .
   specs:
-    standard (0.12.0)
-      rubocop (= 1.8.1)
-      rubocop-performance (= 1.9.2)
+    standard (1.0.3)
+      rubocop (= 1.11.0)
+      rubocop-performance (= 1.10.1)
 
 GEM
   remote: https://rubygems.org/
@@ -13,18 +13,18 @@ GEM
     docile (1.3.5)
     gimme (0.5.0)
     method_source (1.0.0)
-    minitest (5.14.3)
+    minitest (5.14.4)
     parallel (1.20.1)
     parser (3.0.0.0)
       ast (~> 2.4.1)
-    pry (0.13.1)
+    pry (0.14.0)
       coderay (~> 1.1)
       method_source (~> 1.0)
     rainbow (3.0.0)
     rake (13.0.3)
-    regexp_parser (2.0.3)
+    regexp_parser (2.1.1)
     rexml (3.2.4)
-    rubocop (1.8.1)
+    rubocop (1.11.0)
       parallel (~> 1.10)
       parser (>= 3.0.0.0)
       rainbow (>= 2.2.2, < 4.0)
@@ -35,7 +35,7 @@ GEM
       unicode-display_width (>= 1.4.0, < 3.0)
     rubocop-ast (1.4.1)
       parser (>= 2.7.1.5)
-    rubocop-performance (1.9.2)
+    rubocop-performance (1.10.1)
       rubocop (>= 0.90.0, < 2.0)
       rubocop-ast (>= 0.4.0)
     ruby-progressbar (1.11.0)
@@ -49,6 +49,7 @@ GEM
 
 PLATFORMS
   ruby
+  x86_64-darwin-20
 
 DEPENDENCIES
   bundler
@@ -60,4 +61,4 @@ DEPENDENCIES
   standard!
 
 BUNDLED WITH
-   2.2.6
+   2.2.3

data/LICENSE.txt

@@ -2,7 +2,7 @@ Copyright (c) 2019 Test Double, LLC
 
 Portions of these files Copyright (c) 2012-18 Bozhidar Batsov:
   - config/base.yml
-  - lib/standard/cop/semantic_blocks.rb
+  - lib/standard/cop/block_delimiters.rb
   - test/cop_invoker.rb
 
 Permission is hereby granted, free of charge, to any person obtaining

data/README.md

@@ -40,10 +40,12 @@ flag.
   you'll need interpolation in a string slows people down
 - **1.9 hash syntax** - When all the keys in a hash literal are symbols,
   Standard enforces Ruby 1.9's `{hash: syntax}`
-- **Semantic blocks** - `{`/`}` for functional blocks that return a value, and
-  `do`/`end` for procedural blocks that have side effects. More
-  [here](http://www.virtuouscode.com/2011/07/26/the-procedurefunction-block-convention-in-ruby/)
-  and [here](https://github.com/rubocop-hq/ruby-style-guide/issues/162)
+- **Braces for single-line blocks** - Require `{`/`}` for one-line blocks, but
+  allow either braces or `do`/`end` for multiline blocks. Like using `do`/`end`
+  for multiline blocks? Prefer `{`/`}` when chaining? A fan of expressing intent
+  with Jim Weirich's [semantic
+  block](http://www.virtuouscode.com/2011/07/26/the-procedurefunction-block-convention-in-ruby/)
+  approach? Standard lets you do you!
 - **Leading dots on multi-line method chains** - chosen for
   [these](https://github.com/testdouble/standard/issues/75) reasons.
 - **Spaces inside blocks, but not hash literals** - In Ruby, the `{` and `}`
@@ -58,12 +60,6 @@ can look at Standard's current base configuration in
 significant changes to the configuration will be documented as [GitHub release
 notes](https://github.com/testdouble/standard/releases).
 
-**[NOTE: until StandardRB hits 1.0.0, we consider this configuration to be a
-non-final work in progress and we encourage you to submit your opinions (and
-reasoned arguments) for the addition, removal, or change to a rule by [opening
-an issue](https://github.com/testdouble/standard/issues/new). If you start using
-Standard, don't be shocked if things change a bit!]**
-
 ## Usage
 
 Once you've installed Standard, you should be able to use the `standardrb`
@@ -235,11 +231,6 @@ style.
 
 ## I disagree with rule X, can you change it?
 
-**[NOTE: until StandardRB hits 1.0.0, the answer is yes! It just requires
-[opening an issue](https://github.com/testdouble/standard/issues/new) and
-convincing [@searls](https://twitter.com/searls) (the BDFNow) to make the
-change.]**
-
 No. The whole point of Standard is to save you time by avoiding
 [bikeshedding](https://www.freebsd.org/doc/en/books/faq/misc.html#bikeshed-painting)
 about code style. There are lots of debates online about tabs vs. spaces, etc.
@@ -302,10 +293,50 @@ ignore:
     - Style/BlockDelimiters
 ```
 
+## How do I disable a warning within my source code?
+
 You can also use special comments to disable all or certain rules within your
-source code. See [RuboCop's
-docs](https://docs.rubocop.org/en/latest/configuration/#disabling-cops-within-source-code)
-for details.
+source code.
+
+Given this source listing `foo.rb`:
+
+```ruby
+baz = 42
+```
+
+Running `standard foo.rb` would fail:
+
+```
+foo.rb:1:1: Lint/UselessAssignment: Useless assignment to variable - `baz`.
+```
+
+If we wanted to make an exception, we could add the following comment:
+
+```ruby
+baz = 42 # standard:disable Lint/UselessAssignment
+```
+
+The comment directives (both `standard:disable` and `rubocop:disable`) will
+suppress the error and Standard would succeed.
+
+If, however, you needed to disable standard for multiple lines, you could use
+open and closing directives like this:
+
+```ruby
+# standard:disable Layout/IndentationWidth
+def foo
+    123
+end
+# standard:enable Layout/IndentationWidth
+```
+
+And if you don't know or care which rule is being violated, you can also
+substitute its name for "all". This line actually triggers three different
+violations, so we can suppress them like this:
+
+```ruby
+baz = ['a'].each do end # standard:disable all
+```
 
 ## How do I specify a Ruby version? What is supported?
 

data/config/base.yml

@@ -329,6 +329,9 @@ Layout/TrailingWhitespace:
   Enabled: true
   AllowInHeredoc: true
 
+Lint/AmbiguousAssignment:
+  Enabled: true
+
 Lint/AmbiguousOperator:
   Enabled: true
 
@@ -360,6 +363,9 @@ Lint/Debugger:
 Lint/DeprecatedClassMethods:
   Enabled: true
 
+Lint/DeprecatedConstants:
+  Enabled: true
+
 Lint/DeprecatedOpenSSLConstant:
   Enabled: true
 
@@ -732,39 +738,8 @@ Security/YAMLLoad:
   Enabled: true
   SafeAutoCorrect: false
 
-Standard/SemanticBlocks:
-  ProceduralMethods:
-    - benchmark
-    - bm
-    - bmbm
-    - tap
-    # Enumerable
-    - cycle
-    - each
-    - each_cons
-    - each_entry
-    - each_slice
-    - each_with_index
-    # Rails
-    - transaction
-  FunctionalMethods:
-    - given
-    - given!
-    - let
-    - let!
-    - subject
-    - watch
-    - Given
-    - Given!
-    - Invariant
-    - Then
-    - And
-  IgnoredMethods:
-    - lambda
-    - proc
-    - describe
-    - it
-    - When
+Standard/BlockDelimiters:
+  Enabled: true
 
 Style/Alias:
   Enabled: true
@@ -880,6 +855,9 @@ Style/GlobalVars:
   Enabled: true
   AllowedVariables: []
 
+Style/HashExcept:
+  Enabled: true
+
 Style/HashSyntax:
   Enabled: true
   EnforcedStyle: ruby19_no_mixed_keys

data/config/ruby-2.5.yml

@@ -1,4 +1,4 @@
-inherit_from: ./base.yml
+inherit_from: ./ruby-2.7.yml
 
 AllCops:
   TargetRubyVersion: 2.5

data/config/ruby-2.7.yml

@@ -0,0 +1,7 @@
+inherit_from: ./base.yml
+
+AllCops:
+  TargetRubyVersion: 2.7
+
+Style/HashExcept:
+  Enabled: true

data/docs/RELEASE.md

@@ -23,7 +23,7 @@ really the only one we'll need for releasing the gem to
 
 1. Make sure git is up to date and `bundle exec rake` exits cleanly
 2. If you upgraded a Rubocop dependency, be sure to lock it down in
-   `standard.gemspec`. Until Rubocop hits 1.0, we're going to stick to exact
+   `standard.gemspec`. To avoid being broken transitively, we stick to exact
    release dependencies (e.g. "0.91.0" instead of "~> 0.91")
 3. Bump the appropriate version segment in `lib/standard/version.rb` (basic
    semantic versioning rules apply; if the release updates Rubocop, follow its

data/lib/standard.rb

@@ -7,7 +7,7 @@ require "standard/cli"
 require "standard/railtie" if defined?(Rails) && defined?(Rails::Railtie)
 
 require "standard/formatter"
-require "standard/cop/semantic_blocks"
+require "standard/cop/block_delimiters"
 
 module Standard
 end

data/lib/standard/cli.rb

@@ -3,9 +3,6 @@ require_relative "loads_runner"
 
 module Standard
   class Cli
-    SUCCESS_STATUS_CODE = 0
-    FAILURE_STATUS_CODE = 1
-
     def initialize(argv)
       @argv = argv
       @builds_config = BuildsConfig.new
@@ -14,10 +11,7 @@ module Standard
 
     def run
       config = @builds_config.call(@argv)
-
-      success = @loads_runner.call(config.runner).call(config)
-
-      success ? SUCCESS_STATUS_CODE : FAILURE_STATUS_CODE
+      @loads_runner.call(config.runner).call(config).to_i
     end
   end
 end

data/lib/standard/cop/block_delimiters.rb

@@ -0,0 +1,96 @@
+module RuboCop::Cop
+  module Standard
+    # Check for uses of braces around single line blocks, but allows either
+    # braces or do/end for multi-line blocks.
+    #
+    # @example
+    #   # bad - single line block
+    #   items.each do |item| item / 5 end
+    #
+    #   # good - single line block
+    #   items.each { |item| item / 5 }
+    #
+    class BlockDelimiters < RuboCop::Cop::Base
+      extend RuboCop::Cop::AutoCorrector
+
+      def on_send(node)
+        return unless node.arguments?
+        return if node.parenthesized?
+        return if node.operator_method? || node.assignment_method?
+
+        node.arguments.each do |arg|
+          get_blocks(arg) do |block|
+            # If there are no parentheses around the arguments, then braces
+            # and do-end have different meaning due to how they bind, so we
+            # allow either.
+            ignore_node(block)
+          end
+        end
+      end
+
+      def on_block(node)
+        return if ignored_node?(node)
+        return if proper_block_style?(node)
+
+        message = message(node)
+        add_offense(node.loc.begin, message: message) do |corrector|
+          autocorrect(corrector, node)
+        end
+      end
+
+      private
+
+      def get_blocks(node, &block)
+        case node.type
+        when :block
+          yield node
+        when :send
+          get_blocks(node.receiver, &block) if node.receiver
+        when :hash
+          # A hash which is passed as method argument may have no braces
+          # In that case, one of the K/V pairs could contain a block node
+          # which could change in meaning if do...end replaced {...}
+          return if node.braces?
+
+          node.each_child_node { |child| get_blocks(child, &block) }
+        when :pair
+          node.each_child_node { |child| get_blocks(child, &block) }
+        end
+      end
+
+      def proper_block_style?(node)
+        node.multiline? || node.braces?
+      end
+
+      def message(node)
+        "Prefer `{...}` over `do...end` for single-line blocks."
+      end
+
+      def autocorrect(corrector, node)
+        return if correction_would_break_code?(node)
+
+        replace_do_end_with_braces(corrector, node.loc)
+      end
+
+      def correction_would_break_code?(node)
+        return unless node.keywords?
+
+        node.send_node.arguments? && !node.send_node.parenthesized?
+      end
+
+      def replace_do_end_with_braces(corrector, loc)
+        b = loc.begin
+        e = loc.end
+
+        corrector.insert_after(b, " ") unless whitespace_after?(b, 2)
+
+        corrector.replace(b, "{")
+        corrector.replace(e, "}")
+      end
+
+      def whitespace_after?(range, length = 1)
+        /\s/.match?(range.source_buffer.source[range.begin_pos + length, 1])
+      end
+    end
+  end
+end

data/lib/standard/creates_config_store/assigns_rubocop_yaml.rb

@@ -20,6 +20,8 @@ class Standard::CreatesConfigStore
         "ruby-2.3.yml"
       elsif desired_version < Gem::Version.new("2.6")
         "ruby-2.5.yml"
+      elsif desired_version < Gem::Version.new("3.0")
+        "ruby-2.7.yml"
       else
         "base.yml"
       end

data/lib/standard/formatter.rb

@@ -8,11 +8,6 @@ module Standard
       standard: Use Ruby Standard Style (https://github.com/testdouble/standard)
     MSG
 
-    CALL_TO_ACTION_MESSAGE = <<-MSG.gsub(/^ {6}/, "")
-      Notice: Disagree with these rules? While StandardRB is pre-1.0.0, feel free to submit suggestions to:
-        https://github.com/testdouble/standard/issues/new
-    MSG
-
     def self.fixable_error_message(command)
       <<-MSG.gsub(/^ {8}/, "")
         standard: Run `#{command}` to automatically fix some problems.
@@ -24,7 +19,6 @@ module Standard
       @detects_fixability = DetectsFixability.new
       @header_printed_already = false
       @fix_suggestion_printed_already = false
-      @any_uncorrected_offenses = false
     end
 
     def started(_target_files)
@@ -33,7 +27,6 @@ module Standard
 
     def file_finished(file, offenses)
       return unless (uncorrected_offenses = offenses.reject(&:corrected?)).any?
-      @any_uncorrected_offenses = true
 
       print_header_once
       print_fix_suggestion_once(uncorrected_offenses)
@@ -43,10 +36,6 @@ module Standard
       end
     end
 
-    def finished(_)
-      print_call_for_feedback if @any_uncorrected_offenses
-    end
-
     private
 
     def print_header_once
@@ -90,11 +79,6 @@ module Standard
       Pathname.new(file).relative_path_from(Pathname.new(Dir.pwd))
     end
 
-    def print_call_for_feedback
-      output.print "\n"
-      output.print CALL_TO_ACTION_MESSAGE
-    end
-
     def auto_correct_option_provided?
       options[:auto_correct] || options[:safe_auto_correct]
     end

data/lib/standard/rubocop/ext.rb

@@ -5,4 +5,12 @@ module RuboCop
       "Wrap assignment in parentheses if intentional"
     end
   end
+
+  class CommentConfig
+    remove_const :COMMENT_DIRECTIVE_REGEXP
+    COMMENT_DIRECTIVE_REGEXP = Regexp.new(
+      ('# (?:standard|rubocop) : ((?:disable|enable|todo))\b ' + COPS_PATTERN)
+        .gsub(" ", '\s*')
+    )
+  end
 end

data/lib/standard/runners/genignore.rb

@@ -13,7 +13,7 @@ module Standard
           config.rubocop_options[:formatters] = [["files", temp_file.path]]
           config.rubocop_options[:format] = "files"
           config.rubocop_options[:out] = temp_file.path
-          Runners::Rubocop.new.call(config)
+          exit_code = Runners::Rubocop.new.call(config)
 
           # Read in the files with errors.  It will have the absolute paths
           # of the files but we only want the relative path.
@@ -37,6 +37,7 @@ module Standard
             file.puts "# Remove from this list as you refactor files."
             file.write(yaml_format_errors.to_yaml)
           end
+          exit_code
         ensure
           # Clean up temp file.
           temp_file.close

data/lib/standard/runners/rubocop.rb

@@ -4,32 +4,15 @@ module Standard
   module Runners
     class Rubocop
       def call(config)
-        rubocop_runner = RuboCop::Runner.new(
-          config.rubocop_options,
-          config.rubocop_config_store
+        rubocop_runner = RuboCop::CLI::Command::ExecuteRunner.new(
+          RuboCop::CLI::Environment.new(
+            config.rubocop_options,
+            config.rubocop_config_store,
+            config.paths
+          )
         )
 
-        rubocop_runner.run(config.paths).tap do |success|
-          print_errors_and_warnings(success, rubocop_runner)
-          print_corrected_code_if_fixing_stdin(config.rubocop_options)
-        end
-      end
-
-      private
-
-      def print_errors_and_warnings(success, rubocop_runner)
-        return unless success
-
-        (rubocop_runner.warnings + rubocop_runner.errors).each do |message|
-          warn message
-        end
-      end
-
-      def print_corrected_code_if_fixing_stdin(rubocop_options)
-        return unless rubocop_options[:stdin] && rubocop_options[:auto_correct]
-
-        puts "=" * 20
-        print rubocop_options[:stdin]
+        rubocop_runner.run
       end
     end
   end

data/lib/standard/version.rb

@@ -1,3 +1,3 @@
 module Standard
-  VERSION = Gem::Version.new("0.12.0")
+  VERSION = Gem::Version.new("1.0.3")
 end

data/standard.gemspec

@@ -19,6 +19,6 @@ Gem::Specification.new do |spec|
   spec.executables = spec.files.grep(%r{^exe/}) { |f| File.basename(f) }
   spec.require_paths = ["lib"]
 
-  spec.add_dependency "rubocop", "1.8.1"
-  spec.add_dependency "rubocop-performance", "1.9.2"
+  spec.add_dependency "rubocop", "1.11.0"
+  spec.add_dependency "rubocop-performance", "1.10.1"
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: standard
 version: !ruby/object:Gem::Version
-  version: 0.12.0
+  version: 1.0.3
 platform: ruby
 authors:
 - Justin Searls
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2021-01-29 00:00:00.000000000 Z
+date: 2021-03-06 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rubocop
@@ -16,28 +16,28 @@ dependencies:
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 1.8.1
+        version: 1.11.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 1.8.1
+        version: 1.11.0
 - !ruby/object:Gem::Dependency
   name: rubocop-performance
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 1.9.2
+        version: 1.10.1
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 1.9.2
+        version: 1.10.1
 description:
 email:
 - searls@gmail.com
@@ -46,7 +46,6 @@ executables:
 extensions: []
 extra_rdoc_files: []
 files:
-- ".github/dependabot.yml"
 - ".github/workflows/test.yml"
 - ".gitignore"
 - ".standard.yml"
@@ -64,12 +63,13 @@ files:
 - config/ruby-2.2.yml
 - config/ruby-2.3.yml
 - config/ruby-2.5.yml
+- config/ruby-2.7.yml
 - docs/RELEASE.md
 - exe/standardrb
 - lib/standard.rb
 - lib/standard/builds_config.rb
 - lib/standard/cli.rb
-- lib/standard/cop/semantic_blocks.rb
+- lib/standard/cop/block_delimiters.rb
 - lib/standard/creates_config_store.rb
 - lib/standard/creates_config_store/assigns_rubocop_yaml.rb
 - lib/standard/creates_config_store/configures_ignored_paths.rb
@@ -108,7 +108,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.1.4
+rubygems_version: 3.2.3
 signing_key:
 specification_version: 4
 summary: Ruby Style Guide, with linter & automatic code fixer

data/.github/dependabot.yml

@@ -1,8 +0,0 @@
-version: 2
-updates:
-- package-ecosystem: bundler
-  directory: "/"
-  schedule:
-    interval: weekly
-    time: "10:00"
-  open-pull-requests-limit: 10

data/lib/standard/cop/semantic_blocks.rb

@@ -1,165 +0,0 @@
-module RuboCop::Cop
-  module Standard
-    class SemanticBlocks < RuboCop::Cop::Base
-      include RuboCop::Cop::IgnoredMethods
-      extend RuboCop::Cop::AutoCorrector
-
-      def on_send(node)
-        return unless node.arguments?
-        return if node.parenthesized? || node.operator_method?
-
-        node.arguments.each do |arg|
-          get_blocks(arg) do |block|
-            # If there are no parentheses around the arguments, then braces
-            # and do-end have different meaning due to how they bind, so we
-            # allow either.
-            ignore_node(block)
-          end
-        end
-      end
-
-      def on_block(node)
-        return if ignored_node?(node) ||
-          proper_block_style?(node) ||
-          (!node.braces? && rescue_child_block?(node))
-
-        add_offense(node.loc.begin, message: message(node)) do |corrector|
-          return if correction_would_break_code?(node)
-
-          if node.braces?
-            replace_braces_with_do_end(corrector, node.loc)
-          else
-            replace_do_end_with_braces(corrector, node.loc)
-          end
-        end
-      end
-
-      private
-
-      def message(node)
-        if node.single_line?
-          "Prefer `{...}` over `do...end` for single-line blocks."
-        elsif node.loc.begin.source == "{"
-          "Prefer `do...end` over `{...}` for procedural blocks."
-        else
-          "Prefer `{...}` over `do...end` for functional blocks."
-        end
-      end
-
-      def replace_braces_with_do_end(corrector, loc)
-        b = loc.begin
-        e = loc.end
-
-        corrector.insert_before(b, " ") unless whitespace_before?(b)
-        corrector.insert_before(e, " ") unless whitespace_before?(e)
-        corrector.insert_after(b, " ") unless whitespace_after?(b)
-        corrector.replace(b, "do")
-        corrector.replace(e, "end")
-      end
-
-      def replace_do_end_with_braces(corrector, loc)
-        b = loc.begin
-        e = loc.end
-
-        corrector.insert_after(b, " ") unless whitespace_after?(b, 2)
-
-        corrector.replace(b, "{")
-        corrector.replace(e, "}")
-      end
-
-      def whitespace_before?(range)
-        range.source_buffer.source[range.begin_pos - 1, 1] =~ /\s/
-      end
-
-      def whitespace_after?(range, length = 1)
-        range.source_buffer.source[range.begin_pos + length, 1] =~ /\s/
-      end
-
-      def get_blocks(node, &block)
-        case node.type
-        when :block
-          yield node
-        when :send
-          get_blocks(node.receiver, &block) if node.receiver
-        when :hash
-          # A hash which is passed as method argument may have no braces
-          # In that case, one of the K/V pairs could contain a block node
-          # which could change in meaning if do...end replaced {...}
-          return if node.braces?
-
-          node.each_child_node { |child| get_blocks(child, &block) }
-        when :pair
-          node.each_child_node { |child| get_blocks(child, &block) }
-        end
-      end
-
-      def proper_block_style?(node)
-        method_name = node.method_name
-
-        if ignored_method?(method_name)
-          true
-        elsif node.single_line?
-          node.braces?
-        elsif node.braces?
-          !procedural_method?(method_name) &&
-            (functional_method?(method_name) || functional_block?(node))
-        else
-          procedural_method?(method_name) || !return_value_used?(node)
-        end
-      end
-
-      def correction_would_break_code?(node)
-        rescue_child_block?(node) || non_parenthesized_keyword_args?(node)
-      end
-
-      def rescue_child_block?(node)
-        node.children.any? { |node| node.rescue_type? || node.ensure_type? }
-      end
-
-      def non_parenthesized_keyword_args?(node)
-        return unless node.keywords?
-
-        node.send_node.arguments? && !node.send_node.parenthesized?
-      end
-
-      def functional_method?(method_name)
-        cop_config["FunctionalMethods"].map(&:to_sym).include?(method_name)
-      end
-
-      def functional_block?(node)
-        return_value_used?(node) || return_value_of_scope?(node)
-      end
-
-      def procedural_method?(method_name)
-        cop_config["ProceduralMethods"].map(&:to_sym).include?(method_name)
-      end
-
-      def return_value_used?(node)
-        return unless node.parent
-
-        # If there are parentheses around the block, check if that
-        # is being used.
-        if node.parent.begin_type?
-          return_value_used?(node.parent)
-        else
-          node.parent.assignment? || node.parent.send_type?
-        end
-      end
-
-      def return_value_of_scope?(node)
-        return unless node.parent
-
-        conditional?(node.parent) || array_or_range?(node.parent) ||
-          node.parent.children.last == node
-      end
-
-      def conditional?(node)
-        node.if_type? || node.or_type? || node.and_type?
-      end
-
-      def array_or_range?(node)
-        node.array_type? || node.irange_type? || node.erange_type?
-      end
-    end
-  end
-end