datarockets-style 0.8.1 → 1.1.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
-SHA1:
-  metadata.gz: 4f7d9668fbc210ee4a62d6e430941cb41f0c76f1
-  data.tar.gz: f940fdde78d921b6fed3ebe586c136525a986397
+SHA256:
+  metadata.gz: 7ef40260e95ced035f4c8f86b18c807997a4e79228c72e75a880477fecb8030e
+  data.tar.gz: b35d77e092533855df4092585e8af229653859c6cce685bed399f50880d5bc49
 SHA512:
-  metadata.gz: c78d9af47a6f03ea18358b9a41547fd6e5afd61aa51abffb3ee5319cb5c3762359e5bb2b611387abfe4f9c7bbf5beafbaa3573c5b418ee9c00f1b865dea5f840
-  data.tar.gz: 72897ad0da40c29848d78a0663004fcb968c51e67f84a6480e0c6953b2a35cc83949bfae4aba2bbe84575bfb842959da9cc0e806e2f5b5ffb010cdaa83b33b25
+  metadata.gz: a8b0a2bcb81d48837a708f337e3f37baba4e26d86f1be42fa02c12f441e18286b1164996a0a0026f870886a887c7664300683de2b80dee0e7fc6c4721934fce2
+  data.tar.gz: 0dd7f1c7bd2a72a7cbebe4fb5eaf4b399f53d014b0ecfe701be979917e09e7bc7467367d0dced76d25f9a76ad369b88da746285774e092b0078eea180209c913

data/.github/ISSUE_TEMPLATE/update-dependencies.md

@@ -14,3 +14,4 @@ To-Do list:
 - [ ] update dependencies to the latest version
 - [ ] investigate how it changes and affect our current styles and cops
 - [ ] check the required ruby version
+- [ ] enable pending cops

data/.gitignore

@@ -4,3 +4,6 @@
 /_yardoc/
 /pkg/
 /tmp/
+
+.byebug_history
+.idea

data/.rubocop.yml

@@ -7,7 +7,7 @@ inherit_mode:
     - Exclude
 
 AllCops:
-  TargetRubyVersion: 2.3
+  TargetRubyVersion: 2.4
 
 # for checking cops with interpolation
 Lint/InterpolationCheck:

data/CHANGELOG.md

@@ -2,7 +2,73 @@
 
 The format is described in [Contributing notes](CONTRIBUTING.md#changelog-entry-format).
 
-## master (unreleased)
+## master
+
+## 1.1.0 (2021-02-09)
+
+### Changed
+
+* Setup `EnforcedStyleForMultiline` for `Style/TrailingCommaInHashLiteral` rule. ([@r.dubrovsky][])
+* Update rubocop to ```1.9.1```.([@paydaylight][])
+* Update rubocop-rails requirement to ```>= 2.8.0, < 2.10.0```. ([@paydaylight][])
+* Update rubocop-rspec to `2.2.0`. ([@paydaylight][])
+* Change ```Datarockets::Style``` module to ```DatarocketsStyle```. ([@paydaylight][])
+
+### Fixed
+
+* [#177](https://github.com/datarockets/datarockets-style/issues/177) set ```Layout/MultilineOperationIndentation``` to indented ([@paydaylight][])
+
+## 1.0.0 (2020-11-10)
+
+### Changed
+
+* Update rubocop to `1.2.0`.
+* Update rubocop-rails to `2.8.1`.
+* Update rubocop-rspec tp `2.0.0`.
+
+## 0.11.0 (2020-11-07)
+
+### Changed
+
+* Update rubocop to '0.93.1'. ([@r.dubrovsky][])
+  * Enable new cops `Lint/BinaryOperatorWithIdenticalOperands`, `Lint/DuplicateRescueException`, `Lint/EmptyConditionalBody`, `Lint/FloatComparison`, `Lint/MissingSuper
+`, `Lint/OutOfRangeRegexpRef`, `Lint/SelfAssignment`, `Lint/TopLevelReturnWithArgument`, `Lint/UnreachableLoop`, `Style/ExplicitBlockArgument`, `Style/GlobalStdStream`, `Style/OptionalBooleanParameter`, `Style/SingleArgumentDig`, `Style/SoleNestedConditional` and `Style/StringConcatenation` in version `0.89`.
+  * Allow to add all new lints automatically.
+  * Add auto-correctable count notification.
+  * Fix tests
+* Update rubocop-rails to `2.7.1`.
+* Update rubocop-rspec tp `1.43.2`.
+  * Disable `RSpec/MultipleMemoizedHelpers` cop.
+
+## 0.10.0 (2020-07-17)
+
+### Changed
+
+* Update rubocop to `0.88.0`. ([@r.dubrovsky][])
+  * Enable new cops `Lint/MixedRegexpCaptureTypes`, `Style/RedundantRegexpCharacterClass` and `Style/RedundantRegexpEscape`. Cops were added in version `0.85`.
+  * Enable new cop `Style/RedundantFetchBlock`. Cop was added in version `0.86`.
+  * Enable new cops `Style/AccessorGrouping`, `Style/BisectedAttrAccessor` and `Style/RedundantAssignment`. Cops were added in version `0.87`.
+  * Enable new cops `Lint/DuplicateElsifCondition`, `Style/ArrayCoercion`, `Style/CaseLikeIf`, `Style/HashAsLastArrayItem`, `Style/HashLikeCase` and `Style/RedundantFileExtensionInRequire`. Cops were added in version `0.88`.
+* Update rubocop-rails to `2.6.0`.
+* Update rubocop-rspec to `1.42.0`.
+
+* Setup `no_braces` rule for `Style/HashAsLastArrayItem` cop which added in the rubocop version `0.88`.
+
+## 0.9.0 (2020-05-27)
+
+### Changed
+
+* **(Breaking)** Drop support for Ruby 2.3. ([@r.dubrovsky][])
+
+* Update rubocop to `0.84.0`. ([@r.dubrovsky][])
+  * Enable new cops `Lint/RaiseException` and `Lint/StructNewOverride`. Cops were added in version `0.81`.
+  * Enable new cops `Layout/SpaceAroundMethodCallOperator` and `Style/ExponentialNotation`. Cops were added in version `0.82`.
+  * Enable new cops `Layout/EmptyLinesAroundAttributeAccessor` and `Style/SlicingWithRange`. Cops were added in version `0.83`.
+  * Enable new cop `Lint/DeprecatedOpenSSLConstant`. Cop was added in version `0.84`.
+
+* Update rubocop-rails to `2.5.2`.
+* Update rubocop-rspec to `1.39`.
+* Update `activesupport` for fixing security issues.
 
 ## 0.8.1 (2020-03-02)
 
@@ -155,4 +221,4 @@ The format is described in [Contributing notes](CONTRIBUTING.md#changelog-entry-
 [@v.kuzmik]: https://github.com/TheBlackArroVV/
 [@a.branzeanu]: https://github.com/texpert
 [@nikitasakov]: https://github.com/nikitasakov
-
+[@paydaylight]: https://github.com/paydaylight

data/Gemfile

@@ -6,4 +6,4 @@ gemspec
 
 gem "pry-byebug"
 gem "rake", "~> 13.0"
-gem "rspec", "~> 3.7"
+gem "rspec", "~> 3.10"

data/Gemfile.lock

@@ -1,61 +1,79 @@
 PATH
   remote: .
   specs:
-    datarockets-style (0.8.1)
-      rubocop (~> 0.80)
-      rubocop-rails (~> 2.4)
-      rubocop-rspec (~> 1.38)
+    datarockets-style (1.1.0)
+      rubocop (>= 1.2, < 2.0)
+      rubocop-rails (>= 2.8.0, < 2.10.0)
+      rubocop-rspec (~> 2.0)
 
 GEM
   remote: https://rubygems.org/
   specs:
-    ast (2.4.0)
-    byebug (11.0.1)
+    activesupport (6.1.1)
+      concurrent-ruby (~> 1.0, >= 1.0.2)
+      i18n (>= 1.6, < 2)
+      minitest (>= 5.1)
+      tzinfo (~> 2.0)
+      zeitwerk (~> 2.3)
+    ast (2.4.2)
+    byebug (11.1.3)
     coderay (1.1.2)
-    diff-lcs (1.3)
-    jaro_winkler (1.5.4)
-    method_source (0.9.2)
-    parallel (1.19.1)
-    parser (2.7.0.4)
-      ast (~> 2.4.0)
-    pry (0.12.2)
-      coderay (~> 1.1.0)
-      method_source (~> 0.9.0)
-    pry-byebug (3.7.0)
+    concurrent-ruby (1.1.7)
+    diff-lcs (1.4.4)
+    i18n (1.8.7)
+      concurrent-ruby (~> 1.0)
+    method_source (1.0.0)
+    minitest (5.14.3)
+    parallel (1.20.1)
+    parser (3.0.0.0)
+      ast (~> 2.4.1)
+    pry (0.13.1)
+      coderay (~> 1.1)
+      method_source (~> 1.0)
+    pry-byebug (3.9.0)
       byebug (~> 11.0)
-      pry (~> 0.10)
-    rack (2.2.2)
+      pry (~> 0.13.0)
+    rack (2.2.3)
     rainbow (3.0.0)
-    rake (13.0.1)
+    rake (13.0.3)
+    regexp_parser (2.0.3)
     rexml (3.2.4)
-    rspec (3.9.0)
-      rspec-core (~> 3.9.0)
-      rspec-expectations (~> 3.9.0)
-      rspec-mocks (~> 3.9.0)
-    rspec-core (3.9.0)
-      rspec-support (~> 3.9.0)
-    rspec-expectations (3.9.0)
+    rspec (3.10.0)
+      rspec-core (~> 3.10.0)
+      rspec-expectations (~> 3.10.0)
+      rspec-mocks (~> 3.10.0)
+    rspec-core (3.10.0)
+      rspec-support (~> 3.10.0)
+    rspec-expectations (3.10.0)
       diff-lcs (>= 1.2.0, < 2.0)
-      rspec-support (~> 3.9.0)
-    rspec-mocks (3.9.0)
+      rspec-support (~> 3.10.0)
+    rspec-mocks (3.10.0)
       diff-lcs (>= 1.2.0, < 2.0)
-      rspec-support (~> 3.9.0)
-    rspec-support (3.9.0)
-    rubocop (0.80.1)
-      jaro_winkler (~> 1.5.1)
+      rspec-support (~> 3.10.0)
+    rspec-support (3.10.0)
+    rubocop (1.9.1)
       parallel (~> 1.10)
-      parser (>= 2.7.0.1)
+      parser (>= 3.0.0.0)
       rainbow (>= 2.2.2, < 4.0)
+      regexp_parser (>= 1.8, < 3.0)
       rexml
+      rubocop-ast (>= 1.2.0, < 2.0)
       ruby-progressbar (~> 1.7)
-      unicode-display_width (>= 1.4.0, < 1.7)
-    rubocop-rails (2.4.2)
+      unicode-display_width (>= 1.4.0, < 3.0)
+    rubocop-ast (1.4.1)
+      parser (>= 2.7.1.5)
+    rubocop-rails (2.9.1)
+      activesupport (>= 4.2.0)
       rack (>= 1.1)
-      rubocop (>= 0.72.0)
-    rubocop-rspec (1.38.1)
-      rubocop (>= 0.68.1)
-    ruby-progressbar (1.10.1)
-    unicode-display_width (1.6.1)
+      rubocop (>= 0.90.0, < 2.0)
+    rubocop-rspec (2.1.0)
+      rubocop (~> 1.0)
+      rubocop-ast (>= 1.1.0)
+    ruby-progressbar (1.11.0)
+    tzinfo (2.0.4)
+      concurrent-ruby (~> 1.0)
+    unicode-display_width (2.0.0)
+    zeitwerk (2.4.2)
 
 PLATFORMS
   ruby
@@ -64,7 +82,7 @@ DEPENDENCIES
   datarockets-style!
   pry-byebug
   rake (~> 13.0)
-  rspec (~> 3.7)
+  rspec (~> 3.10)
 
 BUNDLED WITH
-   1.16.3
+   2.1.4

data/README.md

@@ -10,14 +10,14 @@ Add this line to your application's Gemfile:
 
 ```ruby
 group :test, :development do
-  gem 'datarockets-style'
+  gem 'datarockets-style', '~> 1.0.0'
 end
 ```
 
 Or, for a Ruby library, add this to your gemspec:
 
 ```ruby
-spec.add_development_dependency 'datarockets-style'
+spec.add_development_dependency 'datarockets-style', '~> 1.0.0'
 ```
 
 And then execute:

data/RELEASING.md

@@ -15,3 +15,19 @@ gem push *built_file*
 * Announce the new release,
    making sure to say "thank you" to the contributors
    who helped shape this version!
+
+## Versioning stragety
+
+As versioning strategy, we're using SemVer: `MAJOR.MINOR.PATCH`
+
+A `MAJOR` version must be incremented if there are any backward-incompatible breaking changes included in a release. This has the benefit of making it easy for anyone to quickly identify if a new version will work differently than a previous one.
+
+Usually, we'll increment it on updating major versions of rubocop dependencies. Ideally, it should be synchronized with the major verion of `rubocop`.
+
+The `MINOR` version must be incremented if backward-compatible functionality is introduced. In the strictest sense, this means you should be able to upgrade to a new minor version without experiencing any breaking changes.
+
+Usually, we'll increment it on changing the list of cops, updating minor versions of dependencies.
+
+The `PATCH` version is meant for backward-compatible bug fixes. You should expect no new functionality with a new patch version, only improvements.
+
+Usually, we'll use it for updating patch versions of dependencies or fixing bugs of the last version. Changed rules of cops in most cases should be added to minor version.

data/bin/console

@@ -1,7 +1,7 @@
 #!/usr/bin/env ruby
 
 require "bundler/setup"
-require "datarockets/style"
+require "datarockets_style"
 
 # You can add fixtures and/or initialization code here to make experimenting
 # with your gem easier. You can also use a different console, if you like.

data/config/base.yml

@@ -1,4 +1,7 @@
-require: datarockets/style
+require: datarockets_style
+
+AllCops:
+  NewCops: enable
 
 Bundler/DuplicatedGem:
   Enabled: true
@@ -29,6 +32,9 @@ Layout/ParameterAlignment:
   EnforcedStyle: with_fixed_indentation
   IndentationWidth: 2
 
+Layout/EmptyLinesAroundAttributeAccessor:
+  Enabled: true
+
 Layout/EndAlignment:
   EnforcedStyleAlignWith: variable
 
@@ -44,25 +50,101 @@ Layout/FirstHashElementIndentation:
 Layout/MultilineMethodCallIndentation:
   EnforcedStyle: indented
 
+Layout/MultilineOperationIndentation:
+  EnforcedStyle: indented
+
+Layout/SpaceAroundMethodCallOperator:
+  Enabled: true
+
 Layout/SpaceInsideHashLiteralBraces:
   EnforcedStyle: no_space
   EnforcedStyleForEmptyBraces: no_space
 
+Lint/BinaryOperatorWithIdenticalOperands:
+  Enabled: true
+
+Lint/DeprecatedOpenSSLConstant:
+  Enabled: true
+
+Lint/DuplicateElsifCondition:
+  Enabled: true
+
+Lint/DuplicateRescueException:
+  Enabled: true
+
+Lint/EmptyConditionalBody:
+  Enabled: true
+
+Lint/FloatComparison:
+  Enabled: true
+
+Lint/MissingSuper:
+  Enabled: true
+
+Lint/MixedRegexpCaptureTypes:
+  Enabled: true
+
+Lint/OutOfRangeRegexpRef:
+  Enabled: true
+
+Lint/RaiseException:
+  Enabled: true
+
+Lint/SelfAssignment:
+  Enabled: true
+
+Lint/StructNewOverride:
+  Enabled: true
+
+Lint/TopLevelReturnWithArgument:
+  Enabled: true
+
+Lint/UnreachableLoop:
+  Enabled: true
+
 Naming/MemoizedInstanceVariableName:
   EnforcedStyleForLeadingUnderscores: required
 
 Naming/RescuedExceptionsVariableName:
   PreferredName: error
 
+Style/AccessorGrouping:
+  Enabled: true
+
+Style/ArrayCoercion:
+  Enabled: true
+
+Style/BisectedAttrAccessor:
+  Enabled: true
+
+Style/CaseLikeIf:
+  Enabled: true
+
 Style/ClassAndModuleChildren:
   EnforcedStyle: nested
 
 Style/EmptyMethod:
   EnforcedStyle: expanded
 
+Style/ExplicitBlockArgument:
+  Enabled: true
+
+Style/ExponentialNotation:
+  Enabled: true
+
+Style/GlobalStdStream:
+  Enabled: true
+
+Style/HashAsLastArrayItem:
+  Enabled: true
+  EnforcedStyle: no_braces
+
 Style/HashEachMethods:
   Enabled: true
 
+Style/HashLikeCase:
+   Enabled: true
+
 Style/HashTransformKeys:
   Enabled: true
 
@@ -75,5 +157,38 @@ Style/FrozenStringLiteralComment:
 Style/NestedInterpolation:
   Enabled: true
 
+Style/OptionalBooleanParameter:
+  Enabled: true
+
+Style/RedundantAssignment:
+  Enabled: true
+
+Style/RedundantFetchBlock:
+  Enabled: true
+
+Style/RedundantFileExtensionInRequire:
+  Enabled: true
+
+Style/RedundantRegexpCharacterClass:
+  Enabled: true
+
+Style/RedundantRegexpEscape:
+  Enabled: true
+
+Style/SingleArgumentDig:
+  Enabled: true
+
+Style/SlicingWithRange:
+  Enabled: true
+
+Style/SoleNestedConditional:
+  Enabled: true
+
+Style/StringConcatenation:
+  Enabled: true
+
 Style/StringLiterals:
   EnforcedStyle: double_quotes
+
+Style/TrailingCommaInHashLiteral:
+  EnforcedStyleForMultiline: comma

data/config/rails.yml

@@ -1,5 +1,7 @@
 require: rubocop-rails
 
+inherit_from: base.yml
+
 AllCops:
   Exclude:
     - "db/**/*"
@@ -12,8 +14,6 @@ AllCops:
     - "config.ru"
     - "Rakefile"
 
-inherit_from: base.yml
-
 Style/Documentation:
   Enabled: true
   Exclude:

data/config/rspec.yml

@@ -31,5 +31,8 @@ RSpec/LetSetup:
 RSpec/MessageSpies:
   EnforcedStyle: have_received
 
+RSpec/MultipleMemoizedHelpers:
+  Enabled: false
+
 RSpec/NestedGroups:
   Enabled: false

data/datarockets-style.gemspec

@@ -1,10 +1,10 @@
 lib = File.expand_path("lib", __dir__)
 $LOAD_PATH.unshift(lib) unless $LOAD_PATH.include?(lib)
-require "datarockets/style/version"
+require "datarockets_style/version"
 
 Gem::Specification.new do |spec|
   spec.name          = "datarockets-style"
-  spec.version       = Datarockets::Style::VERSION
+  spec.version       = DatarocketsStyle::VERSION
   spec.authors       = ["Roman Dubrovsky"]
   spec.email         = ["r.dubrovsky@datarockets.com"]
 
@@ -12,7 +12,7 @@ Gem::Specification.new do |spec|
   spec.homepage      = "https://github.com/datarockets/datarockets-style"
   spec.license       = "MIT"
 
-  spec.required_ruby_version = ">= 2.3.0"
+  spec.required_ruby_version = ">= 2.4.0"
 
   # Prevent pushing this gem to RubyGems.org. To allow pushes either set the 'allowed_push_host'
   # to allow pushing to a single host or delete this section to allow pushing to any host.
@@ -30,7 +30,7 @@ Gem::Specification.new do |spec|
   spec.executables   = spec.files.grep(%r{^exe/}) { |f| File.basename(f) }
   spec.require_paths = ["lib"]
 
-  spec.add_dependency "rubocop", "~> 0.80"
-  spec.add_dependency "rubocop-rails", "~> 2.4"
-  spec.add_dependency "rubocop-rspec", "~> 1.38"
+  spec.add_dependency "rubocop", ">= 1.2  ", "< 2.0"
+  spec.add_dependency "rubocop-rails", ">= 2.8.0", "< 2.10.0"
+  spec.add_dependency "rubocop-rspec", "~> 2.0"
 end

data/doc/STYLE_GUIDE.md

@@ -392,6 +392,46 @@ def foo
 end
 ```
 
+* <a name="style-trailing-comma-in-hash-literal"></a>
+  Requires a comma after the last item in a hash.
+  <sup>[[link](#style-trailing-comma-in-hash-literal)]</sup>
+
+```ruby
+# bad
+a = { foo: 1, bar: 2, }
+
+# good
+a = { foo: 1, bar: 2 }
+
+# bad
+a = {
+  foo: 1, bar: 2,
+  qux: 3,
+}
+
+# good
+a = {
+  foo: 1, bar: 2,
+  qux: 3
+}
+
+# bad
+a = {
+  foo: 1, bar: 2, qux: 3,
+}
+
+# good
+a = {
+  foo: 1, bar: 2, qux: 3
+}
+
+# good
+a = {
+  foo: 1,
+  bar: 2,
+}
+```
+
 * <a name="style-magic-link"></a>
   There are not any required rules for `frozen_string_literal` magic url.
   Set up [this cop](https://rubocop.readthedocs.io/en/latest/cops_style/#stylefrozenstringliteralcomment) depends on the project.

data/doc/STYLE_GUIDE_RSPEC.md

@@ -15,6 +15,10 @@ This style guide recommends best practices for writing a clear Rspec tests and e
   So there are not any limits for deep or nested groups.
   <sup>[[link](#rspec-nested-groups)]</sup>
 
+* <a name="rspec-let-count"></a>
+  We're not limiting a number of `let` blocks in describe and context blocks.
+  <sup>[[link](#rspec-let-count)]</sup>
+
 * <a name="rspec-subject"></a>
   Each subject should be named, and we should not use `subject` in our test cases.
   Prefer to use `is_expected` that `expect(subject_name)` for small tests.

data/lib/datarockets_style.rb

@@ -0,0 +1,12 @@
+require "rubocop"
+require "datarockets_style/formatter/todo_list_formatter"
+
+require "datarockets_style/version"
+
+require "datarockets_style/cop/layout/array_alignment_extended"
+require "datarockets_style/cop/style/nested_interpolation"
+
+# Top level module for datarockets-style
+module DatarocketsStyle
+  # Datarickors sharable config
+end

data/lib/datarockets_style/cop/layout/array_alignment_extended.rb

@@ -0,0 +1,81 @@
+# frozen_string_literal: true
+
+module DatarocketsStyle
+  module Cop
+    module Layout
+      # Here we check if the elements of a multi-line array literal are
+      # aligned.
+      #
+      # @example EnforcedStyle: with_first_argument (default)
+      #   # good
+      #
+      #   array = [1, 2, 3,
+      #            4, 5, 6]
+      #   array = ['run',
+      #            'forrest',
+      #            'run']
+      #
+      #   # bad
+      #
+      #   array = [1, 2, 3,
+      #     4, 5, 6]
+      #   array = ['run',
+      #        'forrest',
+      #        'run']
+      #
+      # @example EnforcedStyle: with_fixed_indentation
+      #   # good
+      #
+      #   array = [1, 2, 3,
+      #     4, 5, 6]
+      #
+      #   # bad
+      #
+      #   array = [1, 2, 3,
+      #            4, 5, 6]
+      class ArrayAlignmentExtended < RuboCop::Cop::Cop
+        include RuboCop::Cop::Alignment
+
+        ALIGN_PARAMS_MSG = "Align the elements of an array literal if they span more than one line."
+
+        FIXED_INDENT_MSG = "Use one level of indentation for elements " \
+            "following the first line of a multi-line array."
+
+        def on_array(node)
+          return if node.children.size < 2
+
+          check_alignment(node.children, base_column(node, node.children))
+        end
+
+        def autocorrect(node)
+          RuboCop::Cop::AlignmentCorrector.correct(processed_source, node, column_delta)
+        end
+
+        private
+
+        def message(_node)
+          fixed_indentation? ? FIXED_INDENT_MSG : ALIGN_PARAMS_MSG
+        end
+
+        def fixed_indentation?
+          cop_config["EnforcedStyle"] == "with_fixed_indentation"
+        end
+
+        def base_column(node, args)
+          fixed_indentation? ? line_indentation(node) : display_column(args.first.source_range)
+        end
+
+        def line_indentation(node)
+          lineno = target_method_lineno(node)
+          line = node.source_range.source_buffer.source_line(lineno)
+          line_indentation = /\S.*/.match(line).begin(0)
+          line_indentation + configured_indentation_width
+        end
+
+        def target_method_lineno(node)
+          node.loc.line
+        end
+      end
+    end
+  end
+end

data/lib/datarockets_style/cop/style/nested_interpolation.rb

@@ -0,0 +1,35 @@
+module DatarocketsStyle
+  module Cop
+    module Style
+      # This cop checks nested interpolations
+      #
+      # @example
+      #
+      # # bad
+      # "Hello, #{user.blank? ? 'guest' : "dear #{user.name}"}"
+      #
+      # # good
+      # user_name = user.blank? ? 'guest' : "dear #{user.name}"
+      # "Hello, #{user_name}"
+      class NestedInterpolation < RuboCop::Cop::Cop
+        include RuboCop::Cop::Interpolation
+
+        MSG = "Redundant nested interpolation.".freeze
+
+        def on_interpolation(node)
+          node.each_descendant(:dstr) do |descendant_node|
+            detect_double_interpolation(descendant_node)
+          end
+        end
+
+        private
+
+        def detect_double_interpolation(node)
+          node.each_child_node(:begin) do |begin_node|
+            add_offense(begin_node)
+          end
+        end
+      end
+    end
+  end
+end

data/lib/{datarockets/style → datarockets_style}/formatter/todo_list_formatter.rb

@@ -42,9 +42,10 @@ class TodoListFormatter < RuboCop::Formatter::ProgressFormatter
   def finished(inspected_files)
     report_summary(inspected_files.length,
       @total_offense_count,
-      @total_correction_count)
+      @total_correction_count,
+      @total_correctable_count)
     output.puts
 
-    Datarockets::Style::Formatter::TodoListFormatter::ReportSummary.new(offense_list).call(output)
+    DatarocketsStyle::Formatter::TodoListFormatter::ReportSummary.new(offense_list).call(output)
   end
 end

data/lib/datarockets_style/formatter/todo_list_formatter/report_summary.rb

@@ -0,0 +1,55 @@
+module DatarocketsStyle
+  module Formatter
+    module TodoListFormatter
+      # Get file of pairs: file path and cop name - and prepare report for ToDo list formatter.
+      #
+      # Example of result:
+      #
+      # LineLength
+      #   Exclude:
+      #     - "really/bad/file.rb" # 100500
+      #     - "almost/ok.rb" # 1
+      class ReportSummary
+        attr_reader :offense_list
+
+        FileGroup = Struct.new(:file, :offenses_count) do
+          def print(output)
+            output.puts "    - '#{file}' # #{offenses_count}"
+          end
+        end
+
+        OffenseGroup = Struct.new(:cop_name, :offenses) do
+          def file_groups
+            @_file_groups ||= offenses.group_by(&:file_path).map do |file, offenses|
+              FileGroup.new(file, offenses.length)
+            end
+          end
+
+          def print(output)
+            output.puts("#{cop_name}:")
+            output.puts("  Exclude:")
+            file_groups.sort_by(&:file).each do |file_group|
+              file_group.print(output)
+            end
+            output.puts
+          end
+        end
+
+        def initialize(offense_list)
+          @offense_list = offense_list
+        end
+
+        def call(output)
+          offense_groups.sort_by(&:cop_name).each { |group| group.print(output) }
+        end
+
+        private
+
+        def offense_groups
+          @_offense_groups ||= offense_list.group_by(&:cop_name)
+            .map { |cop_name, offenses| OffenseGroup.new(cop_name, offenses) }
+        end
+      end
+    end
+  end
+end

data/lib/datarockets_style/version.rb

@@ -0,0 +1,3 @@
+module DatarocketsStyle
+  VERSION = "1.1.0".freeze
+end

metadata

@@ -1,57 +1,69 @@
 --- !ruby/object:Gem::Specification
 name: datarockets-style
 version: !ruby/object:Gem::Version
-  version: 0.8.1
+  version: 1.1.0
 platform: ruby
 authors:
 - Roman Dubrovsky
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2020-03-02 00:00:00.000000000 Z
+date: 2021-02-09 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rubocop
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '1.2'
+    - - "<"
       - !ruby/object:Gem::Version
-        version: '0.80'
+        version: '2.0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '1.2'
+    - - "<"
       - !ruby/object:Gem::Version
-        version: '0.80'
+        version: '2.0'
 - !ruby/object:Gem::Dependency
   name: rubocop-rails
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 2.8.0
+    - - "<"
       - !ruby/object:Gem::Version
-        version: '2.4'
+        version: 2.10.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 2.8.0
+    - - "<"
       - !ruby/object:Gem::Version
-        version: '2.4'
+        version: 2.10.0
 - !ruby/object:Gem::Dependency
   name: rubocop-rspec
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.38'
+        version: '2.0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.38'
+        version: '2.0'
 description: 
 email:
 - r.dubrovsky@datarockets.com
@@ -86,12 +98,12 @@ files:
 - doc/STYLE_GUIDE.md
 - doc/STYLE_GUIDE_RAILS.md
 - doc/STYLE_GUIDE_RSPEC.md
-- lib/datarockets/style.rb
-- lib/datarockets/style/cop/layout/array_alignment_extended.rb
-- lib/datarockets/style/cop/style/nested_interpolation.rb
-- lib/datarockets/style/formatter/todo_list_formatter.rb
-- lib/datarockets/style/formatter/todo_list_formatter/report_summary.rb
-- lib/datarockets/style/version.rb
+- lib/datarockets_style.rb
+- lib/datarockets_style/cop/layout/array_alignment_extended.rb
+- lib/datarockets_style/cop/style/nested_interpolation.rb
+- lib/datarockets_style/formatter/todo_list_formatter.rb
+- lib/datarockets_style/formatter/todo_list_formatter/report_summary.rb
+- lib/datarockets_style/version.rb
 - manual/cops_layout.md
 homepage: https://github.com/datarockets/datarockets-style
 licenses:
@@ -106,15 +118,14 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: 2.3.0
+      version: 2.4.0
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubyforge_project: 
-rubygems_version: 2.6.14
+rubygems_version: 3.2.5
 signing_key: 
 specification_version: 4
 summary: Datarockets style guides and shared style configs

data/lib/datarockets/style.rb

@@ -1,14 +0,0 @@
-require "rubocop"
-require "datarockets/style/formatter/todo_list_formatter"
-
-require "datarockets/style/version"
-
-require "datarockets/style/cop/layout/array_alignment_extended"
-require "datarockets/style/cop/style/nested_interpolation"
-
-module Datarockets
-  # Datarickors sharable config
-  module Style
-    # Your code goes here...
-  end
-end

data/lib/datarockets/style/cop/layout/array_alignment_extended.rb

@@ -1,83 +0,0 @@
-# frozen_string_literal: true
-
-module Datarockets
-  module Style
-    module Cop
-      module Layout
-        # Here we check if the elements of a multi-line array literal are
-        # aligned.
-        #
-        # @example EnforcedStyle: with_first_argument (default)
-        #   # good
-        #
-        #   array = [1, 2, 3,
-        #            4, 5, 6]
-        #   array = ['run',
-        #            'forrest',
-        #            'run']
-        #
-        #   # bad
-        #
-        #   array = [1, 2, 3,
-        #     4, 5, 6]
-        #   array = ['run',
-        #        'forrest',
-        #        'run']
-        #
-        # @example EnforcedStyle: with_fixed_indentation
-        #   # good
-        #
-        #   array = [1, 2, 3,
-        #     4, 5, 6]
-        #
-        #   # bad
-        #
-        #   array = [1, 2, 3,
-        #            4, 5, 6]
-        class ArrayAlignmentExtended < RuboCop::Cop::Cop
-          include RuboCop::Cop::Alignment
-
-          ALIGN_PARAMS_MSG = "Align the elements of an array literal if they span more than one line."
-
-          FIXED_INDENT_MSG = "Use one level of indentation for elements " \
-            "following the first line of a multi-line array."
-
-          def on_array(node)
-            return if node.children.size < 2
-
-            check_alignment(node.children, base_column(node, node.children))
-          end
-
-          def autocorrect(node)
-            RuboCop::Cop::AlignmentCorrector.correct(processed_source, node, column_delta)
-          end
-
-          private
-
-          def message(_node)
-            fixed_indentation? ? FIXED_INDENT_MSG : ALIGN_PARAMS_MSG
-          end
-
-          def fixed_indentation?
-            cop_config["EnforcedStyle"] == "with_fixed_indentation"
-          end
-
-          def base_column(node, args)
-            fixed_indentation? ? line_indentation(node) : display_column(args.first.source_range)
-          end
-
-          def line_indentation(node)
-            lineno = target_method_lineno(node)
-            line = node.source_range.source_buffer.source_line(lineno)
-            line_indentation = /\S.*/.match(line).begin(0)
-            line_indentation + configured_indentation_width
-          end
-
-          def target_method_lineno(node)
-            node.loc.line
-          end
-        end
-      end
-    end
-  end
-end

data/lib/datarockets/style/cop/style/nested_interpolation.rb

@@ -1,37 +0,0 @@
-module Datarockets
-  module Style
-    module Cop
-      module Style
-        # This cop checks nested interpolations
-        #
-        # @example
-        #
-        # # bad
-        # "Hello, #{user.blank? ? 'guest' : "dear #{user.name}"}"
-        #
-        # # good
-        # user_name = user.blank? ? 'guest' : "dear #{user.name}"
-        # "Hello, #{user_name}"
-        class NestedInterpolation < RuboCop::Cop::Cop
-          include RuboCop::Cop::Interpolation
-
-          MSG = "Redundant nested interpolation.".freeze
-
-          def on_interpolation(node)
-            node.each_descendant(:dstr) do |descendant_node|
-              detect_double_interpolation(descendant_node)
-            end
-          end
-
-          private
-
-          def detect_double_interpolation(node)
-            node.each_child_node(:begin) do |begin_node|
-              add_offense(begin_node)
-            end
-          end
-        end
-      end
-    end
-  end
-end

data/lib/datarockets/style/formatter/todo_list_formatter/report_summary.rb

@@ -1,57 +0,0 @@
-module Datarockets
-  module Style
-    module Formatter
-      module TodoListFormatter
-        # Get file of pairs: file path and cop name - and prepare report for ToDo list formatter.
-        #
-        # Example of result:
-        #
-        # LineLength
-        #   Exclude:
-        #     - "really/bad/file.rb" # 100500
-        #     - "almost/ok.rb" # 1
-        class ReportSummary
-          attr_reader :offense_list
-
-          FileGroup = Struct.new(:file, :count) do
-            def print(output)
-              output.puts "    - '#{file}' # #{count}"
-            end
-          end
-
-          OffenseGroup = Struct.new(:cop_name, :offenses) do
-            def file_groups
-              @_file_groups ||= offenses.group_by(&:file_path).map do |file, offenses|
-                FileGroup.new(file, offenses.length)
-              end
-            end
-
-            def print(output)
-              output.puts(cop_name + ":")
-              output.puts("  Exclude:")
-              file_groups.sort_by(&:file).each do |file_group|
-                file_group.print(output)
-              end
-              output.puts
-            end
-          end
-
-          def initialize(offense_list)
-            @offense_list = offense_list
-          end
-
-          def call(output)
-            offense_groups.sort_by(&:cop_name).each { |group| group.print(output) }
-          end
-
-          private
-
-          def offense_groups
-            @_offense_groups ||= offense_list.group_by(&:cop_name)
-              .map { |cop_name, offenses| OffenseGroup.new(cop_name, offenses) }
-          end
-        end
-      end
-    end
-  end
-end

data/lib/datarockets/style/version.rb

@@ -1,5 +0,0 @@
-module Datarockets
-  module Style
-    VERSION = "0.8.1".freeze
-  end
-end