nxt_cop 1.3.0 → 2.0.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: c9cbb2bc281782ecc8344596723250c60a347e54756fb8ab3e901903fce85437
-  data.tar.gz: 1ed23e495e6d4eb4287e58115c442c62a497d8b300db00e41b8d2b5633e7e9cb
+  metadata.gz: 1eb597d25eddcad36a8b678d16aec8012a29e0b9c33a145fc7e528303cf1ea27
+  data.tar.gz: dfa1494d7e788cf1500f030938995d8e5a53a74cee29d3b1fc726687dc7cab85
 SHA512:
-  metadata.gz: 36b72e877b2d70bc2a51075442e269801cc059e19113948b592e7d6a94c95df5306963175bb06257e7b79843bc15367b53b2d001204178587a93adce73b953d6
-  data.tar.gz: aceeda13120fa2354923c432daa796c4eb4ed9675a270a150d984d76408a41c8aa6bce21e23ef881032643a13c4816a65377aa0e8caa543e14a0bb54c571c999
+  metadata.gz: 1d2923f5964870f8e7262c19476874f01e1785ddbba09ce158909f3cbf0758f7857de4df85586372abb669686d79cbf74f49d74500a4566e457b4eecca830583
+  data.tar.gz: 5e7efb90fc058ffc9b7c02a6aa948fcd0b20cf50c3b274998dbcc0ca5acc7c268c950afa97031073202747b009c4e520a1b400a1f60d2110f9479c340298acba

data/CHANGELOG.md

@@ -1,3 +1,15 @@
+# v2.0.0 2023-11-28
+## What's Changed
+* Enabled all cops by default
+
+**Full Changelog**: https://github.com/nxt-insurance/nxt_cop/compare/v1.3.1...v2.0.0
+
+# v1.3.1 2023-10-10
+## What's Changed
+* Bumped dependencies version
+
+**Full Changelog**: https://github.com/nxt-insurance/nxt_cop/compare/v1.3.0...v1.3.1
+
 # v1.3.0 2023-09-22
 ## What's Changed
 * `inherit_mode` was added to `default.yml`

data/Gemfile.lock

@@ -1,64 +1,78 @@
 PATH
   remote: .
   specs:
-    nxt_cop (1.3.0)
-      rubocop (~> 1.56.1)
+    nxt_cop (2.0.0)
+      rubocop (~> 1.57.2)
       rubocop-rails (~> 2.8)
       rubocop-rspec (~> 2.12)
 
 GEM
   remote: https://rubygems.org/
   specs:
-    activesupport (7.0.7.2)
+    activesupport (7.1.2)
+      base64
+      bigdecimal
       concurrent-ruby (~> 1.0, >= 1.0.2)
+      connection_pool (>= 2.2.5)
+      drb
       i18n (>= 1.6, < 2)
       minitest (>= 5.1)
+      mutex_m
       tzinfo (~> 2.0)
     ast (2.4.2)
-    base64 (0.1.1)
+    base64 (0.2.0)
+    bigdecimal (3.1.4)
     concurrent-ruby (1.2.2)
+    connection_pool (2.4.1)
+    drb (2.2.0)
+      ruby2_keywords
     i18n (1.14.1)
       concurrent-ruby (~> 1.0)
     json (2.6.3)
     language_server-protocol (3.17.0.3)
     minitest (5.20.0)
+    mutex_m (0.2.0)
     parallel (1.23.0)
-    parser (3.2.2.3)
+    parser (3.2.2.4)
       ast (~> 2.4.1)
       racc
-    racc (1.7.1)
+    racc (1.7.3)
     rack (3.0.8)
     rainbow (3.1.1)
-    rake (13.0.6)
-    regexp_parser (2.8.1)
+    rake (13.1.0)
+    regexp_parser (2.8.2)
     rexml (3.2.6)
-    rubocop (1.56.3)
-      base64 (~> 0.1.1)
+    rubocop (1.57.2)
       json (~> 2.3)
       language_server-protocol (>= 3.17.0)
       parallel (~> 1.10)
-      parser (>= 3.2.2.3)
+      parser (>= 3.2.2.4)
       rainbow (>= 2.2.2, < 4.0)
       regexp_parser (>= 1.8, < 3.0)
       rexml (>= 3.2.5, < 4.0)
       rubocop-ast (>= 1.28.1, < 2.0)
       ruby-progressbar (~> 1.7)
       unicode-display_width (>= 2.4.0, < 3.0)
-    rubocop-ast (1.29.0)
+    rubocop-ast (1.30.0)
       parser (>= 3.2.1.0)
-    rubocop-capybara (2.17.1)
+    rubocop-capybara (2.19.0)
       rubocop (~> 1.41)
-    rubocop-rails (2.18.0)
+    rubocop-factory_bot (2.24.0)
+      rubocop (~> 1.33)
+    rubocop-rails (2.22.2)
       activesupport (>= 4.2.0)
       rack (>= 1.1)
       rubocop (>= 1.33.0, < 2.0)
-    rubocop-rspec (2.19.0)
-      rubocop (~> 1.33)
+      rubocop-ast (>= 1.30.0, < 2.0)
+    rubocop-rspec (2.25.0)
+      rubocop (~> 1.40)
       rubocop-capybara (~> 2.17)
+      rubocop-factory_bot (~> 2.22)
     ruby-progressbar (1.13.0)
+    ruby2_keywords (0.0.5)
     tzinfo (2.0.6)
       concurrent-ruby (~> 1.0)
-    unicode-display_width (2.4.2)
+    unicode-display_width (2.5.0)
 
 PLATFORMS
   ruby

data/README.md

@@ -81,6 +81,7 @@ Add to `~/.gem/credentials` (create if it doesn't exist):
 - Run `rubocop` to see if new rules were added
   - If yes, add them to `default.yml` to prevent "New rules" warning
 - Update the version number in `version.rb`
+- Run `bundle install` again to update the version in Gemfile.lock
 - Update the `CHANGELOG.md`
 - Open the PR, merge everything to `main`
 

data/default.yml

@@ -18,7 +18,8 @@ AllCops:
     - db/migrate/*
     - vendor/**/*
     - node_modules/**/*
-  DisabledByDefault: true
+  DisabledByDefault: false
+  NewCops: disable # to avoid warnings. TODO: remove this setting and add new cops from the warning.
 
 # A bunch of these layout cops are enabled by default,
 # but for some reason 'DisableByDefault' is set to 'true' above.
@@ -260,3 +261,364 @@ Lint/MixedCaseRange:
   Enabled: true
 Lint/RedundantRegexpQuantifiers:
   Enabled: true
+
+# This is default cops we want to keep disabled:
+Style/FrozenStringLiteralComment:
+  Enabled: false
+Layout/LineLength:
+  Enabled: false
+Style/ArgumentsForwarding:
+  Enabled: false
+Naming/BlockForwarding:
+  Enabled: false
+Rails/FreezeTime:
+  Enabled: false
+Style/Documentation:
+  Enabled: false
+Metrics/MethodLength:
+  Enabled: false
+Naming/VariableNumber:
+  Enabled: false
+Metrics/AbcSize:
+  Enabled: false
+Metrics/ClassLength:
+  Enabled: false
+Style/ClassAndModuleChildren:
+  Enabled: false
+Metrics/BlockLength:
+  Enabled: false
+Metrics/CyclomaticComplexity:
+  Enabled: false
+Style/SafeNavigation:
+  Enabled: false
+Rails/Output:
+  Enabled: false
+Style/Next: # this one looks dangerous
+  Enabled: false
+Bundler/OrderedGems:
+  Enabled: false
+Bundler/DuplicatedGroup:
+  Enabled: false
+Rails/HttpStatus:
+  Enabled: false
+Rails/SkipsModelValidations:
+  Enabled: false
+Style/InfiniteLoop:
+  Enabled: false
+Style/StderrPuts:
+  Enabled: false
+Style/NumericPredicate:
+  Enabled: false
+Metrics/PerceivedComplexity:
+  Enabled: false
+Style/RegexpLiteral:
+  Enabled: false
+Rails/Delegate:
+  Enabled: false
+Style/RaiseArgs:
+  Enabled: false
+Rails/HasManyOrHasOneDependent: # Nice to have but challenging to fix
+  Enabled: false
+Rails/InverseOf:
+  Enabled: false
+Style/MultilineBlockChain:
+  Enabled: false
+Naming/AccessorMethodName:
+  Enabled: false
+Naming/PredicateName:
+  Enabled: false
+Style/FormatStringToken:
+  Enabled: false
+Style/RedundantSort: # dangerous since autofix breaks the code
+  Enabled: false
+Style/SymbolProc:
+  Enabled: false
+Rails/ApplicationController:
+  Enabled: false
+Rails/ApplicationRecord:
+  Enabled: false
+Style/ExplicitBlockArgument:
+  Enabled: false
+Rails/DynamicFindBy:
+  Enabled: false
+Metrics/ParameterLists:
+  Enabled: false
+Rails/IndexWith:
+  Enabled: false
+Style/FormatString:
+  Enabled: false
+Rails/Blank:
+  Enabled: false
+Rails/Present:
+  Enabled: false
+Style/NumericLiterals:
+  Enabled: false
+Style/Lambda:
+  Enabled: false
+Style/Proc:
+  Enabled: false
+Style/ExponentialNotation:
+  Enabled: false
+Style/PreferredHashMethods: # can be changed via vote
+  Enabled: false
+Rails/Presence:
+  Enabled: false
+Metrics/ModuleLength:
+  Enabled: false
+Style/RedundantConditional:
+  Enabled: false
+Style/TrailingUnderscoreVariable:
+  Enabled: false
+Style/ParenthesesAroundCondition:
+  Enabled: false
+Style/HashEachMethods:
+  Enabled: false
+Style/RedundantAssignment:
+  Enabled: false
+Style/CaseLikeIf:
+  Enabled: false
+Style/RedundantBegin:
+  Enabled: false
+Rails/FilePath:
+  Enabled: false
+Style/Alias:
+  Enabled: false
+Style/SingleArgumentDig:
+  Enabled: false
+Style/RescueModifier: # because it's heavily used in application service
+  Enabled: false
+Style/IfUnlessModifier:
+  Enabled: false
+Naming/BinaryOperatorParameterName:
+  Enabled: false
+Style/SignalException:
+  Enabled: false
+Style/TrivialAccessors:
+  Enabled: false
+Rails/HasAndBelongsToMany:
+  Enabled: false
+Style/AccessModifierDeclarations:
+  Enabled: false
+Rails/ReadWriteAttribute:
+  Enabled: false
+Rails/RedundantReceiverInWithOptions:
+  Enabled: false
+Style/HashTransformValues:
+  Enabled: false
+Rails/IndexBy:
+  Enabled: false
+Rails/Pick:
+  Enabled: false
+Style/CombinableLoops:  # hard to autocorrect properly
+  Enabled: false
+FactoryBot/CreateList:
+  Enabled: false
+Rails/ReflectionClassName:
+  Enabled: false
+Style/RedundantSelf:
+  Enabled: false
+Style/GuardClause:
+  Enabled: false
+Style/PercentLiteralDelimiters:
+  Enabled: false
+Style/StringConcatenation:  # heavily used in insurance service
+  Enabled: false
+Style/InverseMethods:
+  Enabled: false
+Style/RedundantInterpolation:
+  Enabled: false
+Style/ZeroLengthPredicate:
+  Enabled: false
+
+# 🤔 Should we enable these cops or not? They look useful but require alignment within the team
+Style/OpenStructUse:
+  Enabled: false
+Style/RedundantConstantBase: # new in 1.40
+  Enabled: false
+Naming/MemoizedInstanceVariableName:
+  Enabled: false
+Style/ConditionalAssignment:
+  Enabled: false
+Style/BlockDelimiters:
+  Enabled: false
+Style/TrailingCommaInArrayLiteral:
+  Enabled: false
+FactoryBot/FactoryClassName:
+  Enabled: false
+Style/NumericLiteralPrefix:
+  Enabled: false
+Style/OptionalBooleanParameter:
+  Enabled: false
+Style/IdenticalConditionalBranches: # Is it safe to auto-fix?
+  Enabled: false
+Naming/RescuedExceptionsVariableName: # Would be nice to have, but should align on the name first
+  Enabled: false
+
+Naming/MethodParameterName: # should enable but we have to manually fix this
+  Enabled: false
+Rails/UniqueValidationWithoutIndex: # definitely enable!!!
+  Enabled: false
+Rails/Validation: # discuss with the team
+  Enabled: false
+Rails/LexicallyScopedActionFilter:  # enable and manually disable offences
+  Enabled: false
+Security/Open:
+  Enabled: false
+Naming/FileName:
+  Enabled: false
+Style/EmptyElse:
+  Enabled: false
+Style/MixinUsage: # Enable and then manually disable offenses?
+  Enabled: false
+
+
+
+# ✅ Enabled in 2.0 version (activated by "enable all rules by default')
+# Let's keep commented list here for visibility for a while
+#
+#Layout/SpaceInsideParens:
+#  Enabled: false
+#Layout/SpaceBeforeFirstArg:
+#  Enabled: false
+#Layout/ExtraSpacing:
+#  Enabled: false
+#Layout/SpaceAfterColon:
+#  Enabled: false
+#Rails/PluralizationGrammar:
+#  Enabled: false
+#Layout/SpaceAroundMethodCallOperator:
+#  Enabled: false
+#Style/RedundantFreeze:
+#  Enabled: false
+#Layout/SpaceInLambdaLiteral:
+#  Enabled: false
+#Layout/LeadingCommentSpace:
+#  Enabled: false
+#Layout/DefEndAlignment:
+#  Enabled: false
+#Style/RedundantRegexpEscape:
+#  Enabled: false
+#Layout/HeredocIndentation:
+#  Enabled: false
+#Layout/ClosingHeredocIndentation:
+#  Enabled: false
+#Layout/LeadingEmptyLines:
+#  Enabled: false
+#Layout/EmptyLinesAroundAccessModifier:
+#  Enabled: false
+#Style/NegatedIf:
+#  Enabled: false
+#Style/CommentAnnotation:
+#  Enabled: false
+#Layout/MultilineOperationIndentation:
+#  Enabled: false
+#Style/ExpandPathArguments:
+#  Enabled: false
+#Layout/EmptyLinesAroundMethodBody:
+#  Enabled: false
+#Style/RedundantReturn:
+#  Enabled: false
+#Style/GlobalStdStream:
+#  Enabled: false
+#Style/FloatDivision:
+#  Enabled: false
+#Style/RescueStandardError:
+#  Enabled: false
+#Style/NilComparison:
+#  Enabled: false
+#Style/RedundantFetchBlock:
+#  Enabled: false
+#Style/Encoding:
+#  Enabled: false
+#Style/EmptyLiteral:
+#  Enabled: false
+#Layout/SpaceInsideReferenceBrackets:
+#  Enabled: false
+#Style/EmptyLambdaParameter:
+#  Enabled: false
+#Style/TernaryParentheses:
+#  Enabled: false
+#Naming/MethodName:
+#  Enabled: false
+#Style/SlicingWithRange:
+#  Enabled: false
+#Style/EmptyCaseCondition:
+#  Enabled: false
+#Style/SoleNestedConditional:
+#  Enabled: false
+#Rails/FindEach:
+#  Enabled: false
+#Style/RedundantFileExtensionInRequire:
+#  Enabled: false
+#Layout/SpaceInsideArrayLiteralBrackets:
+#  Enabled: false
+#Style/HashAsLastArrayItem:
+#  Enabled: false
+#Lint/RedundantCopDisableDirective:
+#  Enabled: false
+#Layout/SpaceAroundKeyword:
+#  Enabled: false
+#Style/KeywordParametersOrder:
+#  Enabled: false
+#Style/BlockComments:
+#  Enabled: false
+#Style/AndOr:
+#  Enabled: false
+#Style/EachWithObject:
+#  Enabled: false
+#Layout/SpaceAroundBlockParameters:
+#  Enabled: false
+#Style/MultilineTernaryOperator:
+#  Enabled: false
+#Style/RedundantParentheses:
+#  Enabled: false
+#Security/JSONLoad:
+#  Enabled: false
+#Style/MethodCallWithoutArgsParentheses:
+#  Enabled: false
+#Style/CommentedKeyword:
+#  Enabled: false
+#Style/MultilineIfModifier:
+#  Enabled: false
+#Style/IfInsideElse:
+#  Enabled: false
+#Style/AccessorGrouping:
+#  Enabled: false
+#Layout/BlockEndNewline:
+#  Enabled: false
+#Rails/RedundantForeignKey:
+#  Enabled: false
+#Style/UnlessElse:
+#  Enabled: false
+#Layout/EmptyLinesAroundModuleBody:
+#  Enabled: false
+#Rails/RakeEnvironment:
+#  Enabled: false
+#Layout/EmptyLinesAroundArguments:
+#  Enabled: false
+#Style/EachForSimpleLoop:
+#  Enabled: false
+#Style/ClassCheck:
+#  Enabled: false
+#Style/NestedParenthesizedCalls:
+#  Enabled: false
+#Style/LineEndConcatenation:
+#  Enabled: false
+#Style/ParallelAssignment:
+#  Enabled: false
+#Layout/EmptyLinesAroundExceptionHandlingKeywords:
+#  Enabled: false
+#Layout/SpaceAroundEqualsInParameterDefault:
+#  Enabled: false
+#Style/YodaCondition:
+#  Enabled: false
+#Layout/SpaceInsideStringInterpolation:
+#  Enabled: false
+#Style/EmptyMethod:
+#  Enabled: false
+#Layout/EmptyLineBetweenDefs:
+#  Enabled: false
+#Naming/ConstantName:
+#  Enabled: false
+#Rails/RelativeDateConstant:
+#  Enabled: false

data/lib/nxt_cop/version.rb

@@ -1,3 +1,3 @@
 module NxtCop
-  VERSION = '1.3.0'.freeze
+  VERSION = '2.0.0'.freeze
 end

data/nxt_cop.gemspec

@@ -1,4 +1,4 @@
-lib = File.expand_path('../lib', __FILE__)
+lib = File.expand_path('lib', __dir__)
 $LOAD_PATH.unshift(lib) unless $LOAD_PATH.include?(lib)
 require 'nxt_cop/version'
 
@@ -24,14 +24,16 @@ Gem::Specification.new do |spec|
       'public gem pushes.'
   end
 
-  spec.files         = `git ls-files -z`.split("\x0").reject do |f|
+  spec.files = `git ls-files -z`.split("\x0").reject do |f|
     f.match(%r{^(test|spec|features)/})
   end
   spec.bindir        = 'exe'
   spec.executables   = spec.files.grep(%r{^exe/}) { |f| File.basename(f) }
   spec.require_paths = ['lib']
 
-  spec.add_dependency 'rubocop', '~> 1.56.1'
+  spec.required_ruby_version = '>= 3.2'
+
+  spec.add_dependency 'rubocop', '~> 1.57.2'
   spec.add_dependency 'rubocop-rails', '~> 2.8'
   spec.add_dependency 'rubocop-rspec', '~> 2.12'
   spec.add_development_dependency 'bundler', '~> 2.1'

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: nxt_cop
 version: !ruby/object:Gem::Version
-  version: 1.3.0
+  version: 2.0.0
 platform: ruby
 authors:
 - Scott Livingstone
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2023-09-25 00:00:00.000000000 Z
+date: 2024-01-09 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rubocop
@@ -16,14 +16,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 1.56.1
+        version: 1.57.2
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 1.56.1
+        version: 1.57.2
 - !ruby/object:Gem::Dependency
   name: rubocop-rails
   requirement: !ruby/object:Gem::Requirement
@@ -117,14 +117,14 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: '0'
+      version: '3.2'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.4.6
+rubygems_version: 3.4.10
 signing_key:
 specification_version: 4
 summary: Getsafe shared Rubocop.