standard 0.10.2 → 1.0.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: d321223bc503dbd20a61bbfedcc5b5add5f4d327e2807646d06b9e0dac215094
-  data.tar.gz: 55a9caa17e48d3bd6d3d1e79fe36209520ff9347e73e50e8f9938d501610d9c2
+  metadata.gz: ad16fa94acb2de73ebed6090c37d6e73791f7db9588ce441606963930381d36a
+  data.tar.gz: 2a649119dd14d36e72f0eb159373cba7b173e43b92566a27d92c8d4582564a89
 SHA512:
-  metadata.gz: 671a7fbb708de9c209a2da970d33666557823a6b87d4ba59b1c74d591347300d1b50aa3d50808f5a16b0530482dfc85eed3f80eb84df3f1649b44d29681382a0
-  data.tar.gz: 1ccc5a54fe626bca0a4aff7b7e24419a6114aa846a2494bffe5d45dc838f8922d3572ec40d2fbe6c615b499798be1417bbc6c1efc8d4e1eb80733dd7bb5f9fcf
+  metadata.gz: 6190b3e6ffd7bc1f7254c42d085fa42e4e7e5d16032c46841be6c7149a7ee3d3b01ea07487380237bc7745d2d52112b9c1f8434fac55ce11935212ed6f081b08
+  data.tar.gz: d3c470c9d92695eaa9acb8532b2959176e2c6caa42115cb5781eb6efff81ebab9f711ead27598b7acc3c643a193a9848c7b81dc26637f1d239d38cb8131f2009

data/.github/workflows/test.yml

@@ -9,15 +9,29 @@ on:
 jobs:
   test:
 
-    runs-on: ubuntu-latest
+    strategy:
+      matrix:
+        os: [ ubuntu-latest ]
+        ruby-version: [2.4, 2.7, 3.0]
+
+    runs-on: ${{ matrix.os }}
 
     steps:
     - uses: actions/checkout@v2
-    - name: Set up Ruby
+    - name: Set up Ruby ${{ matrix.ruby-version }}
       uses: ruby/setup-ruby@v1
       with:
-        ruby-version: 2.7
+        ruby-version: ${{ matrix.ruby-version }}
+    - name: Set up caching
+      uses: actions/cache@v1
+      with:
+        path: vendor/bundle
+        key: bundle-user-ruby-${{ matrix.os }}-${{matrix.ruby-version }}-${{ hashFiles('**/Gemfile.lock') }}
+        restore-keys: |
+          bundle-use-ruby-${{ matrix.os }}-${{ matrix.ruby-version }}-
     - name: Install dependencies
-      run: bundle
-    - name: Run tests
+      run: |
+        bundle config path vendor/bundle
+        bundle install --jobs 4 --retry 3
+    - name: Run tests for Ruby ${{ matrix.ruby-version }} on ${{ matrix.os }}
       run: bundle exec rake

data/CHANGELOG.md

@@ -1,5 +1,38 @@
 # Changelog
 
+## 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
+
+* 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)
+* Changed `Style/NegatedIf` to `postfix`
+
 ## 0.10.2
 
 * Remove

data/Gemfile

@@ -1,3 +1,13 @@
 source "https://rubygems.org"
 
 gemspec
+
+gem "bundler"
+gem "minitest", "~> 5.0"
+gem "pry"
+gem "rake", "~> 13.0"
+gem "gimme"
+
+if Gem::Version.new(RUBY_VERSION) >= Gem::Version.new("2.5")
+  gem "simplecov"
+end

data/Gemfile.lock

@@ -1,52 +1,55 @@
 PATH
   remote: .
   specs:
-    standard (0.10.2)
-      rubocop (= 1.4.2)
-      rubocop-performance (= 1.9.1)
+    standard (1.0.1)
+      rubocop (= 1.11.0)
+      rubocop-performance (= 1.10.1)
 
 GEM
   remote: https://rubygems.org/
   specs:
-    ast (2.4.1)
+    ast (2.4.2)
     coderay (1.1.3)
-    docile (1.3.2)
+    docile (1.3.5)
     gimme (0.5.0)
     method_source (1.0.0)
-    minitest (5.14.2)
+    minitest (5.14.4)
     parallel (1.20.1)
-    parser (2.7.2.0)
+    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.1)
-    regexp_parser (2.0.0)
+    rake (13.0.3)
+    regexp_parser (2.1.1)
     rexml (3.2.4)
-    rubocop (1.4.2)
+    rubocop (1.11.0)
       parallel (~> 1.10)
-      parser (>= 2.7.1.5)
+      parser (>= 3.0.0.0)
       rainbow (>= 2.2.2, < 4.0)
-      regexp_parser (>= 1.8)
+      regexp_parser (>= 1.8, < 3.0)
       rexml
-      rubocop-ast (>= 1.1.1)
+      rubocop-ast (>= 1.2.0, < 2.0)
       ruby-progressbar (~> 1.7)
-      unicode-display_width (>= 1.4.0, < 2.0)
-    rubocop-ast (1.3.0)
+      unicode-display_width (>= 1.4.0, < 3.0)
+    rubocop-ast (1.4.1)
       parser (>= 2.7.1.5)
-    rubocop-performance (1.9.1)
+    rubocop-performance (1.10.1)
       rubocop (>= 0.90.0, < 2.0)
       rubocop-ast (>= 0.4.0)
-    ruby-progressbar (1.10.1)
-    simplecov (0.19.0)
+    ruby-progressbar (1.11.0)
+    simplecov (0.21.2)
       docile (~> 1.1)
       simplecov-html (~> 0.11)
-    simplecov-html (0.12.2)
-    unicode-display_width (1.7.0)
+      simplecov_json_formatter (~> 0.1)
+    simplecov-html (0.12.3)
+    simplecov_json_formatter (0.1.2)
+    unicode-display_width (2.0.0)
 
 PLATFORMS
   ruby
+  x86_64-darwin-20
 
 DEPENDENCIES
   bundler
@@ -58,4 +61,4 @@ DEPENDENCIES
   standard!
 
 BUNDLED WITH
-   2.1.4
+   2.2.6

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`
@@ -302,10 +298,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?
 
@@ -367,7 +403,7 @@ Refer to RuboCop's [documentation on
 formatters](https://rubocop.readthedocs.io/en/latest/formatters/) for more
 information.
 
-## How do I run standard in my editor?
+## How do I run Standard in my editor?
 
 It can be very handy to know about failures while editing to shorten the
 feedback loop. Some editors support asynchronously running linters.
@@ -378,6 +414,10 @@ feedback loop. Some editors support asynchronously running linters.
 - [vim (via ALE)](https://github.com/testdouble/standard/wiki/IDE:-vim)
 - [VS Code](https://github.com/testdouble/standard/wiki/IDE:-vscode)
 
+## How do I use Standard with Rubocop extensions?
+
+This is not officially supported by Standard. However, Evil Martians wrote up [a regularly updated guide](https://evilmartians.com/chronicles/rubocoping-with-legacy-bring-your-ruby-code-up-to-standard) on how to do so.
+
 ## Does Standard work with [Insert other tool name here]?
 
 Maybe! Start by searching the repository to see if there's an existing issue open for

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
@@ -939,7 +917,7 @@ Style/MultilineWhenThen:
 
 Style/NegatedIf:
   Enabled: true
-  EnforcedStyle: both
+  EnforcedStyle: postfix
 
 Style/NegatedWhile:
   Enabled: true
@@ -1102,6 +1080,9 @@ Style/SingleLineMethods:
   Enabled: true
   AllowIfMethodIsEmpty: false
 
+Style/SlicingWithRange:
+  Enabled: true
+
 Style/StabbyLambdaParentheses:
   Enabled: true
   EnforcedStyle: require_parentheses

data/config/ruby-2.3.yml

@@ -1,4 +1,4 @@
-inherit_from: ./base.yml
+inherit_from: ./ruby-2.5.yml
 
 AllCops:
   TargetRubyVersion: 2.4 # The oldest supported

data/config/ruby-2.5.yml

@@ -0,0 +1,7 @@
+inherit_from: ./ruby-2.7.yml
+
+AllCops:
+  TargetRubyVersion: 2.5
+
+Style/SlicingWithRange:
+  Enabled: false

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/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

@@ -18,6 +18,10 @@ class Standard::CreatesConfigStore
         "ruby-2.2.yml"
       elsif desired_version < Gem::Version.new("2.4")
         "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/rubocop.rb

@@ -28,7 +28,11 @@ module Standard
       def print_corrected_code_if_fixing_stdin(rubocop_options)
         return unless rubocop_options[:stdin] && rubocop_options[:auto_correct]
 
-        puts "=" * 20
+        if rubocop_options[:stderr]
+          warn "=" * 20
+        else
+          puts "=" * 20
+        end
         print rubocop_options[:stdin]
       end
     end

data/lib/standard/version.rb

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

data/standard.gemspec

@@ -19,13 +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.4.2"
-  spec.add_dependency "rubocop-performance", "1.9.1"
-
-  spec.add_development_dependency "bundler"
-  spec.add_development_dependency "minitest", "~> 5.0"
-  spec.add_development_dependency "pry"
-  spec.add_development_dependency "rake", "~> 13.0"
-  spec.add_development_dependency "simplecov"
-  spec.add_development_dependency "gimme"
+  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.10.2
+  version: 1.0.1
 platform: ruby
 authors:
 - Justin Searls
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2020-12-04 00:00:00.000000000 Z
+date: 2021-03-04 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rubocop
@@ -16,112 +16,28 @@ dependencies:
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 1.4.2
+        version: 1.11.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 1.4.2
+        version: 1.11.0
 - !ruby/object:Gem::Dependency
   name: rubocop-performance
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 1.9.1
+        version: 1.10.1
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 1.9.1
-- !ruby/object:Gem::Dependency
-  name: bundler
-  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: minitest
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '5.0'
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '5.0'
-- !ruby/object:Gem::Dependency
-  name: pry
-  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
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '13.0'
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '13.0'
-- !ruby/object:Gem::Dependency
-  name: simplecov
-  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: gimme
-  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'
+        version: 1.10.1
 description:
 email:
 - searls@gmail.com
@@ -146,12 +62,14 @@ files:
 - config/ruby-1.9.yml
 - 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
@@ -190,7 +108,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.1.2
+rubygems_version: 3.1.4
 signing_key:
 specification_version: 4
 summary: Ruby Style Guide, with linter & automatic code fixer

data/lib/standard/cop/semantic_blocks.rb

@@ -1,170 +0,0 @@
-module RuboCop::Cop
-  module Standard
-    class SemanticBlocks < RuboCop::Cop::Cop
-      include RuboCop::Cop::IgnoredMethods
-
-      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, location: :begin)
-      end
-
-      def autocorrect(node)
-        return if correction_would_break_code?(node)
-
-        if node.braces?
-          replace_braces_with_do_end(node.loc)
-        else
-          replace_do_end_with_braces(node.loc)
-        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(loc)
-        b = loc.begin
-        e = loc.end
-
-        lambda do |corrector|
-          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
-      end
-
-      def replace_do_end_with_braces(loc)
-        b = loc.begin
-        e = loc.end
-
-        lambda do |corrector|
-          corrector.insert_after(b, " ") unless whitespace_after?(b, 2)
-
-          corrector.replace(b, "{")
-          corrector.replace(e, "}")
-        end
-      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