ws-style 6.0.3 → 6.3.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 1407de5de16f267a0ddeaf3e2b14dac71c8946042f2765b80c8ad58a9ff24ce6
-  data.tar.gz: cdc35d35136a1bece890de569aa4faeac65c5ff1223aa19a75eb364c198377c1
+  metadata.gz: 574334100ed40e46d82434550ca90e558cd9eb2eed2ba51f860496ba5ae713e2
+  data.tar.gz: 87558c16e965e50133abd8adaa0938ca4eff3d43f08b1a0357c2d6da743b090c
 SHA512:
-  metadata.gz: 6713f23de1b5851821ea3cb380c2c3a872a56129454154b343514481cccfe48ed6db3bc539ce7b776b8eec1b7b147f2fd17c6628a23bf6f709f497371d72f493
-  data.tar.gz: 82b2d9042b9090e648e19eaa132f0954f9d902e7d6434477a2ea173f585e1f3f880f9589e2a9a353183cc1b5a84813d6f9f665e82d582155cde216b9fd1e12ef
+  metadata.gz: fa1ed665e440cd2dbefcaafef11eb4f4a4d87e778f4ae5e491c122f8d679f8bea0afc23c9865e0c89634c51101caa1138c027b63f792b8ae11f0ded3a0de5e5b
+  data.tar.gz: d63d70d6420e098df37293502ccbc60b1e3e23dc646eb5265c18c758576d01ea1e7e6abc90ffa3cf4ed2a1da6c6b76968b39cb08c4b6d684885b016fd28328ab

data/.circleci/config.yml

@@ -3,7 +3,7 @@ version: 2
 defaults: &defaults
   working_directory: ~/wealthsimple
   docker:
-    - image: circleci/ruby:2.5.3
+    - image: circleci/ruby:2.5.7
 
 references:
   attach_code_workspace: &attach_code_workspace

data/.ruby-version

@@ -1 +1 @@
-2.5.3
+2.5.7

data/CHANGELOG.md

@@ -4,6 +4,22 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## 6.2.0 - 2020-09-04
+- Disable Cops enabled in 6.1.0 until we can confirm we want them
+
+## 6.1.1 - 2020-09-04
+- Disable RSpec/MultipleMemoizedHelpers cop
+
+## 6.1.0 - 2020-08-27
+- Upgraded rubocop to 0.89
+- Upgraded rubocop-performance to 1.7
+- Upgraded rubocop-rails to 2.7
+- Split out Rails cops into their own configuration to improve installation for non-Rails projects
+
+## 6.0.4 - 2020-07-06
+### Changed
+- Upgraded rubocop to >= 0.86 and opted into the new cops.
+
 ## 6.0.1 - 2020-05-06
 ### Changed
 - Don't enforce Style/FormatStringToken. We use template style strings heavily, and don't feel that annotated strings are a necessary improvement.

data/README.md

@@ -59,6 +59,14 @@ Style/HashSyntax:
   EnforcedStyle: hash_rockets
 ```
 
+If you're using `ws-style` in a non-Rails project, inherit from the core configuration file instead:
+
+```yaml
+inherit_gem:
+  ws-style:
+    - core.yml
+```
+
 ## Development
 
 After checking out the repo, run `bin/setup` to install dependencies. Then, run `rake spec` to run the tests. You can also run `bin/console` for an interactive prompt that will allow you to experiment.

data/Rakefile

@@ -1,6 +1,30 @@
 require "bundler/gem_tasks"
 require "rspec/core/rake_task"
+require "rubocop"
+require "rubocop-performance"
+require "rubocop-rails"
+require "rubocop-rspec"
 
 RSpec::Core::RakeTask.new(:spec)
 
 task default: :spec
+
+desc 'Print out comments that can be used for a GitHub cop election'
+task :election do # rubocop:disable Rails/RakeEnvironment
+  RuboCop::ConfigLoader.default_configuration.pending_cops.each do |pending_cop|
+    base_urls = {
+      'layout' => 'https://docs.rubocop.org/rubocop/cops_layout.html#layout',
+      'lint' => 'https://docs.rubocop.org/rubocop/cops_lint.html#lint',
+      'style' => 'https://docs.rubocop.org/rubocop/cops_style.html#style',
+      'performance' => 'https://docs.rubocop.org/rubocop-performance/cops_performance.html#performance',
+      'rails' => 'https://docs.rubocop.org/rubocop-rails/cops_rails.html#rails',
+      'rspec' => 'https://docs.rubocop.org/rubocop-rspec/cops_rspec.html#rspec',
+    }
+    department, anchor = pending_cop.name.downcase.split('/')
+    puts <<~COMMENT
+      [**#{pending_cop.name}**](#{base_urls.fetch(department)}#{anchor})
+      #{pending_cop.metadata.fetch('Description')}
+
+    COMMENT
+  end
+end

data/core.yml

@@ -0,0 +1,345 @@
+# See documentation for details on definitions:
+# https://rubocop.readthedocs.io
+
+require:
+  - rubocop-performance
+  - rubocop-rspec
+  - rubocop-vendor
+
+inherit_mode:
+  merge:
+    - AllowedNames
+    - Exclude
+
+AllCops:
+  Exclude:
+    - "bin/**/*"
+    - "data/**/*"
+    - "db/data/**/*"
+    - "db/migrate/**/*"
+    - "db/schema.rb"
+    - "gemfiles/**/*"
+    - "lib/tasks/circle_0.rake"
+    - "log/**/*"
+    - "node_modules/**/*"
+    - "public/**/*"
+    - "tmp/**/*"
+    - "vendor/**/*"
+
+Layout/HashAlignment:
+  EnforcedLastArgumentHashStyle: ignore_implicit
+
+Layout/ParameterAlignment:
+  Enabled: false
+
+Layout/ArgumentAlignment:
+  Enabled: false
+
+Lint/AmbiguousBlockAssociation:
+  Exclude:
+    - "spec/**/*"
+
+Layout/DotPosition:
+  EnforcedStyle: trailing
+
+Layout/CaseIndentation:
+  Enabled: false
+
+Layout/ElseAlignment:
+  Enabled: false
+
+Layout/EndAlignment:
+  Enabled: false
+
+Layout/FirstArrayElementIndentation:
+  EnforcedStyle: consistent
+
+Layout/IndentationWidth:
+  Enabled: true
+  Width: 2
+
+Layout/LineLength:
+  IgnoredPatterns: ['(\A|\s)#']
+  Max: 100
+
+Layout/FirstHashElementIndentation:
+  EnforcedStyle: consistent
+
+Layout/MultilineMethodCallIndentation:
+  EnforcedStyle: indented
+
+Metrics/AbcSize:
+  Max: 18
+
+Metrics/BlockLength:
+  Enabled: false
+
+Metrics/ClassLength:
+  Max: 150
+
+Metrics/CyclomaticComplexity:
+  Max: 7
+
+Metrics/MethodLength:
+  ExcludedMethods: ["extended"]
+  Max: 15
+
+Metrics/ModuleLength:
+  Exclude:
+    - "spec/**/*"
+
+Metrics/ParameterLists:
+  Enabled: false
+
+Metrics/PerceivedComplexity:
+  Max: 8
+
+Style/AndOr:
+  EnforcedStyle: conditionals
+
+Style/BlockDelimiters:
+  Enabled: false
+
+Style/ClassAndModuleChildren:
+  Enabled: false
+
+Style/Documentation:
+  Enabled: false
+
+Style/FormatStringToken:
+  Enabled: false
+
+Style/FrozenStringLiteralComment:
+  Enabled: false
+
+Style/NumericPredicate:
+  Enabled: false
+
+Style/StringLiterals:
+  Enabled: false
+
+Style/SymbolArray:
+  Enabled: false
+
+Style/TrailingCommaInArguments:
+  EnforcedStyleForMultiline: comma
+
+Style/TrailingCommaInArrayLiteral:
+  EnforcedStyleForMultiline: comma
+
+Style/TrailingCommaInHashLiteral:
+  EnforcedStyleForMultiline: comma
+
+Style/WordArray:
+  Enabled: false
+
+Style/NegatedIf:
+  Enabled: false
+
+RSpec/AnyInstance:
+  Enabled: false
+
+RSpec/ContextWording:
+  Enabled: false
+
+RSpec/EmptyExampleGroup:
+  Enabled: false
+
+RSpec/ExampleLength:
+  Max: 15
+
+RSpec/LetSetup:
+  Enabled: false
+
+RSpec/MessageChain:
+  Enabled: false
+
+RSpec/MultipleExpectations:
+  Enabled: false
+
+RSpec/MultipleMemoizedHelpers:
+  Enabled: false
+
+RSpec/NamedSubject:
+  Enabled: false
+
+RSpec/NestedGroups:
+  Max: 6
+
+# Cops added in Rubocop 0.80 or later need to be enabled one-by-one below.
+# For more information: https://docs.rubocop.org/en/latest/versioning/
+
+# 0.80
+Style/HashEachMethods:
+  Enabled: true
+
+Style/HashTransformKeys:
+  Enabled: true
+
+Style/HashTransformValues:
+  Enabled: true
+
+# 0.81
+Lint/RaiseException:
+  Enabled: true
+
+Lint/StructNewOverride:
+  Enabled: true
+
+# 0.82
+Layout/SpaceAroundMethodCallOperator:
+  Enabled: true
+
+Style/ExponentialNotation:
+  Enabled: true
+
+# 0.83
+Layout/EmptyLinesAroundAttributeAccessor:
+  Enabled: True
+
+Style/SlicingWithRange:
+  Enabled: True
+
+# 0.84
+Lint/DeprecatedOpenSSLConstant:
+  Enabled: True
+
+# 0.85
+Lint/MixedRegexpCaptureTypes:
+  Enabled: True
+
+Style/RedundantRegexpCharacterClass:
+  Enabled: True
+
+Style/RedundantRegexpEscape:
+  Enabled: True
+
+# 0.86
+Style/RedundantFetchBlock:
+  Enabled: True
+
+# 0.87
+Style/AccessorGrouping:
+  Enabled: True
+
+Style/BisectedAttrAccessor:
+  Enabled: True
+
+Style/RedundantAssignment:
+  Enabled: True
+
+# 0.88
+Lint/DuplicateElsifCondition:
+  Enabled: True
+
+Style/ArrayCoercion:
+  Enabled: True
+
+Style/CaseLikeIf:
+  Enabled: True
+
+Style/HashAsLastArrayItem:
+  Enabled: True
+
+Style/HashLikeCase:
+  Enabled: True
+
+Style/RedundantFileExtensionInRequire:
+  Enabled: True
+
+# 0.89
+Lint/BinaryOperatorWithIdenticalOperands:
+  Enabled: True
+
+Lint/DuplicateRescueException:
+  Enabled: True
+
+Lint/EmptyConditionalBody:
+  Enabled: True
+
+Lint/FloatComparison:
+  Enabled: True
+
+Lint/MissingSuper:
+  Enabled: True
+
+Lint/OutOfRangeRegexpRef:
+  Enabled: True
+
+Lint/SelfAssignment:
+  Enabled: True
+
+Lint/TopLevelReturnWithArgument:
+  Enabled: True
+
+Lint/UnreachableLoop:
+  Enabled: True
+
+Style/ExplicitBlockArgument:
+  Enabled: True
+
+Style/GlobalStdStream:
+  Enabled: False
+
+Style/OptionalBooleanParameter:
+  Enabled: True
+
+Style/SingleArgumentDig:
+  Enabled: True
+
+Style/StringConcatenation:
+  Enabled: True
+
+# rubocop-performance 1.7
+Performance/AncestorsInclude:
+  Enabled: True
+
+Performance/BigDecimalWithNumericArgument:
+  Enabled: True
+
+Performance/RedundantSortBlock:
+  Enabled: True
+
+Performance/RedundantStringChars:
+  Enabled: True
+
+Performance/ReverseFirst:
+  Enabled: True
+
+Performance/SortReverse:
+  Enabled: True
+
+Performance/Squeeze:
+  Enabled: True
+
+Performance/StringInclude:
+  Enabled: True
+
+# 0.90
+Lint/DuplicateRequire:
+  Enabled: True
+
+Lint/EmptyFile:
+  Enabled: True
+
+Lint/TrailingCommaInAttributeDeclaration:
+  Enabled: True
+
+Lint/UselessMethodDefinition:
+  Enabled: True
+
+Style/CombinableLoops:
+  Enabled: True
+
+Style/KeywordParametersOrder:
+  Enabled: True
+
+Style/RedundantSelfAssignment:
+  Enabled: True
+
+Style/SoleNestedConditional:
+  Enabled: True
+
+# rubocop-performance 1.7
+Performance/Sum:
+  Enabled: True

data/default.yml

@@ -1,204 +1,3 @@
-# See documentation for details on definitions:
-# https://rubocop.readthedocs.io
-
-require:
-  - rubocop-performance
-  - rubocop-rails
-  - rubocop-rspec
-  - rubocop-vendor
-
-inherit_mode:
-  merge:
-    - AllowedNames
-    - Exclude
-
-AllCops:
-  Exclude:
-    - "bin/**/*"
-    - "data/**/*"
-    - "db/data/**/*"
-    - "db/migrate/**/*"
-    - "db/schema.rb"
-    - "gemfiles/**/*"
-    - "lib/tasks/circle_0.rake"
-    - "log/**/*"
-    - "node_modules/**/*"
-    - "public/**/*"
-    - "tmp/**/*"
-    - "vendor/**/*"
-
-Layout/HashAlignment:
-  EnforcedLastArgumentHashStyle: ignore_implicit
-
-Layout/ParameterAlignment:
-  Enabled: false
-
-Layout/ArgumentAlignment:
-  Enabled: false
-
-Lint/AmbiguousBlockAssociation:
-  Exclude:
-    - "spec/**/*"
-
-Layout/DotPosition:
-  EnforcedStyle: trailing
-
-Layout/CaseIndentation:
-  Enabled: false
-
-Layout/ElseAlignment:
-  Enabled: false
-
-Layout/EndAlignment:
-  Enabled: false
-
-Layout/FirstArrayElementIndentation:
-  EnforcedStyle: consistent
-
-Layout/IndentationWidth:
-  Enabled: true
-  Width: 2
-
-Layout/LineLength:
-  IgnoredPatterns: ['(\A|\s)#']
-  Max: 100
-
-Layout/FirstHashElementIndentation:
-  EnforcedStyle: consistent
-
-Layout/MultilineMethodCallIndentation:
-  EnforcedStyle: indented
-
-Metrics/AbcSize:
-  Max: 18
-
-Metrics/BlockLength:
-  Enabled: false
-
-Metrics/ClassLength:
-  Max: 150
-
-Metrics/CyclomaticComplexity:
-  Max: 7
-
-Metrics/MethodLength:
-  ExcludedMethods: ["extended"]
-  Max: 15
-
-Metrics/ModuleLength:
-  Exclude:
-    - "spec/**/*"
-
-Metrics/ParameterLists:
-  Enabled: false
-
-Metrics/PerceivedComplexity:
-  Max: 8
-
-Style/AndOr:
-  EnforcedStyle: conditionals
-
-Style/BlockDelimiters:
-  Enabled: false
-
-Style/ClassAndModuleChildren:
-  Enabled: false
-
-Style/Documentation:
-  Enabled: false
-
-Style/FormatStringToken:
-  Enabled: false
-
-Style/FrozenStringLiteralComment:
-  Enabled: false
-
-Style/NumericPredicate:
-  Enabled: false
-
-Style/StringLiterals:
-  Enabled: false
-
-Style/SymbolArray:
-  Enabled: false
-
-Style/TrailingCommaInArguments:
-  EnforcedStyleForMultiline: comma
-
-Style/TrailingCommaInArrayLiteral:
-  EnforcedStyleForMultiline: comma
-
-Style/TrailingCommaInHashLiteral:
-  EnforcedStyleForMultiline: comma
-
-Style/WordArray:
-  Enabled: false
-
-Style/NegatedIf:
-  Enabled: false
-
-Rails:
-  Enabled: true
-
-Rails/Delegate:
-  Enabled: false
-
-Rails/FilePath:
-  EnforcedStyle: arguments
-
-Rails/TimeZone:
-  Enabled: false
-
-RSpec/AnyInstance:
-  Enabled: false
-
-RSpec/ContextWording:
-  Enabled: false
-
-RSpec/EmptyExampleGroup:
-  Enabled: false
-
-RSpec/ExampleLength:
-  Max: 15
-
-RSpec/LetSetup:
-  Enabled: false
-
-RSpec/MessageChain:
-  Enabled: false
-
-RSpec/MultipleExpectations:
-  Enabled: false
-
-RSpec/NamedSubject:
-  Enabled: false
-
-RSpec/NestedGroups:
-  Max: 6
-
-# Cops added in Rubocop 0.80 or later need to be enabled one-by-one below.
-# For more information: https://docs.rubocop.org/en/latest/versioning/
-
-# 0.80
-Style/HashEachMethods:
-  Enabled: true
-
-Style/HashTransformKeys:
-  Enabled: true
-
-Style/HashTransformValues:
-  Enabled: true
-
-# 0.81
-Lint/RaiseException:
-  Enabled: true
-
-Lint/StructNewOverride:
-  Enabled: true
-
-# 0.82
-Layout/SpaceAroundMethodCallOperator:
-  Enabled: true
-
-Style/ExponentialNotation:
-  Enabled: true
+inherit_from:
+  - core.yml
+  - rails.yml

data/lib/ws/style/version.rb

@@ -1,5 +1,5 @@
 module Ws
   module Style
-    VERSION = '6.0.3'.freeze
+    VERSION = '6.3.0'.freeze
   end
 end

data/rails.yml

@@ -0,0 +1,61 @@
+require:
+  - rubocop-rails
+
+Rails:
+  Enabled: true
+
+Rails/Delegate:
+  Enabled: false
+
+Rails/FilePath:
+  EnforcedStyle: arguments
+
+Rails/TimeZone:
+  Enabled: false
+
+# rubocop-rails 2.7
+Rails/ActiveRecordCallbacksOrder:
+  Enabled: True
+
+Rails/FindById:
+  Enabled: True
+
+Rails/Inquiry:
+  Enabled: True
+
+Rails/MailerName:
+  Enabled: True
+
+Rails/MatchRoute:
+  Enabled: True
+
+Rails/NegateInclude:
+  Enabled: True
+
+Rails/Pluck:
+  Enabled: True
+
+Rails/PluckInWhere:
+  Enabled: True
+
+Rails/RenderInline:
+  Enabled: True
+
+Rails/RenderPlainText:
+  Enabled: True
+
+Rails/ShortI18n:
+  Enabled: False
+
+Rails/WhereExists:
+  Enabled: True
+
+# rubocop-rails 2.8
+Rails/AfterCommitOverride:
+  Enabled: True
+
+Rails/SquishedSQLHeredocs:
+  Enabled: False
+
+Rails/WhereNot:
+  Enabled: True

data/ws-style.gemspec

@@ -20,11 +20,12 @@ Gem::Specification.new do |s|
   s.bindir        = 'exe'
   s.executables   = s.files.grep(%r{^exe/}) { |f| File.basename(f) }
   s.require_paths = ['lib']
+  s.required_ruby_version = '>= 2.5.7'
 
-  s.add_dependency 'rubocop', '>= 0.82'
-  s.add_dependency 'rubocop-performance', '>= 1.1.0'
-  s.add_dependency 'rubocop-rails', '>= 2.4.2'
-  s.add_dependency 'rubocop-rspec', '~> 1.32'
+  s.add_dependency 'rubocop', '>= 0.90.0'
+  s.add_dependency 'rubocop-performance', '>= 1.8.1'
+  s.add_dependency 'rubocop-rails', '>= 2.8.1'
+  s.add_dependency 'rubocop-rspec', '>= 1.43.2'
   s.add_dependency 'rubocop-vendor', '~> 0'
 
   s.add_development_dependency 'bundler'

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: ws-style
 version: !ruby/object:Gem::Version
-  version: 6.0.3
+  version: 6.3.0
 platform: ruby
 authors:
 - Peter Graham
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2020-05-15 00:00:00.000000000 Z
+date: 2020-10-07 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rubocop
@@ -16,56 +16,56 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '0.82'
+        version: 0.90.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '0.82'
+        version: 0.90.0
 - !ruby/object:Gem::Dependency
   name: rubocop-performance
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 1.1.0
+        version: 1.8.1
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 1.1.0
+        version: 1.8.1
 - !ruby/object:Gem::Dependency
   name: rubocop-rails
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 2.4.2
+        version: 2.8.1
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 2.4.2
+        version: 2.8.1
 - !ruby/object:Gem::Dependency
   name: rubocop-rspec
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '1.32'
+        version: 1.43.2
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '1.32'
+        version: 1.43.2
 - !ruby/object:Gem::Dependency
   name: rubocop-vendor
   requirement: !ruby/object:Gem::Requirement
@@ -172,9 +172,11 @@ files:
 - Rakefile
 - bin/console
 - bin/setup
+- core.yml
 - default.yml
 - lib/ws/style.rb
 - lib/ws/style/version.rb
+- rails.yml
 - ws-style.gemspec
 homepage: https://github.com/wealthsimple/ws-style
 licenses: []
@@ -187,7 +189,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: '0'
+      version: 2.5.7
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="