ws-style 6.0.2 → 6.2.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: ce620b6b24d00dace1fc64e0dd663ddffc76e4658c8779f70e105ced0ce70739
-  data.tar.gz: 332ff79e43e97640c0f8b79e5e7010a6c6eba50fe807437050a90b77cfefc4ba
+  metadata.gz: 04c36fcc000cf48774650d6edbf34c51959aa7d7f50cfe1e87892a4f88d49e94
+  data.tar.gz: 6e85312d35bafe7122ec4223791b1f5338fcd87b9f89d0285b039f465a0212ba
 SHA512:
-  metadata.gz: e16cf6b3de9ae4a31f21e0a8b187da918d2f873f3326d27e9a193681988e0d403caf103b95eec1381d22a3e457621b378cfebe1fb221314da7c52f310c9ca7fe
-  data.tar.gz: 7e8164cf1703a08a7a63b6f4c1facac9ceecaf61f4685d20c7c1c2156d3c39f8c586eea389bf914c0af26b41039913af2607bad8bc29b89e73251988a3cd7d62
+  metadata.gz: 8b84cc475ff90a16efab3f4a2f4829e1e8e3137d245675f6e8d88e1f16bd15876afcab7bff4fdfbb3a97090ae6a7be4abb6509d814c242d2c488db99c9c69e16
+  data.tar.gz: c8c2be2e97a14b0196ad7a73e8c4f0cd287a1c3202dde0a06771a4c91f79354ca889a85bac26a26bb73c9be9ba165bf56910116d342fa2b443fb618cf0654933

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/core.yml

@@ -0,0 +1,220 @@
+# 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
+

data/default.yml

@@ -1,203 +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/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.2'.freeze
+    VERSION = '6.2.0'.freeze
   end
 end

data/rails.yml

@@ -0,0 +1,52 @@
+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: True
+
+Rails/WhereExists:
+  Enabled: True
+

data/ws-style.gemspec

@@ -20,10 +20,11 @@ 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', '>= 0.89'
+  s.add_dependency 'rubocop-performance', '>= 1.7'
+  s.add_dependency 'rubocop-rails', '>= 2.7'
   s.add_dependency 'rubocop-rspec', '~> 1.32'
   s.add_dependency 'rubocop-vendor', '~> 0'
 

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: ws-style
 version: !ruby/object:Gem::Version
-  version: 6.0.2
+  version: 6.2.0
 platform: ruby
 authors:
 - Peter Graham
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2020-05-07 00:00:00.000000000 Z
+date: 2020-09-04 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rubocop
@@ -16,42 +16,42 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '0.82'
+        version: '0.89'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '0.82'
+        version: '0.89'
 - !ruby/object:Gem::Dependency
   name: rubocop-performance
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 1.1.0
+        version: '1.7'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 1.1.0
+        version: '1.7'
 - !ruby/object:Gem::Dependency
   name: rubocop-rails
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 2.4.2
+        version: '2.7'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 2.4.2
+        version: '2.7'
 - !ruby/object:Gem::Dependency
   name: rubocop-rspec
   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:
   - - ">="