nxt_cop 1.3.1 → 2.0.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: f3ced16c64af5bb68e2c10850dd9277e29f3a39de3d9cc32bdde3296530f55cf
-  data.tar.gz: 6fa32563d60a5fde5b6c4734617913d69cf6d6cd1441096fae6377837c75993a
+  metadata.gz: 530b577b485dd169ba5ef2bf3820e7a3fa6e8ec4a6604db775c755d5019a3dbc
+  data.tar.gz: cddfba144d854181aecd07f7e3ea98a24d83a281575ec7cece77543f7d9fd448
 SHA512:
-  metadata.gz: 1d43e0c5e70c4b7241c58784717a3b5218e0267dcef358a00308f1689bc17d3f2067eeddc428769e41342c612ce144a137cd39ffc1f8b3ae68f92966c1d4fef2
-  data.tar.gz: c8df079c6ffaac61b89fb07c04fdb38d466cb01ccd5be1c004cb9f73a104fa3647cd7b790dabeb02cdcf668fdf3718300f353a4fd747991e3fefe340d8ba0b22
+  metadata.gz: 04b24945b0ca039e944b465e822b9be9882672db7a2d9dca15528ae6be45945a0c2a3ec2ebd90a7ce7c48a6a10703ce183ffedb2c5b2e2f189ea79b7f3f07beb
+  data.tar.gz: 5c535e553548102f708b5c981e255af718bcdde9580cdda845a06ba7ae8f9cdd162139d2b0f7e4cc7873df53094677d0c21aac0f97848e5be69fc6b42cbc83d5

data/CHANGELOG.md

@@ -1,3 +1,15 @@
+# v2.0.1 2024-01-10
+## What's Changed
+* Disable more cops depending on the code style in our backend-services
+
+* **Full Changelog**: https://github.com/nxt-insurance/nxt_cop/compare/v2.0.0...v2.0.1
+
+# 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

data/Gemfile.lock

@@ -1,15 +1,15 @@
 PATH
   remote: .
   specs:
-    nxt_cop (1.3.1)
-      rubocop (~> 1.57.2)
+    nxt_cop (2.0.1)
+      rubocop (~> 1.60.0)
       rubocop-rails (~> 2.8)
       rubocop-rspec (~> 2.12)
 
 GEM
   remote: https://rubygems.org/
   specs:
-    activesupport (7.1.1)
+    activesupport (7.1.3)
       base64
       bigdecimal
       concurrent-ruby (~> 1.0, >= 1.0.2)
@@ -21,49 +21,50 @@ GEM
       tzinfo (~> 2.0)
     ast (2.4.2)
     base64 (0.2.0)
-    bigdecimal (3.1.4)
-    concurrent-ruby (1.2.2)
+    bigdecimal (3.1.5)
+    concurrent-ruby (1.2.3)
     connection_pool (2.4.1)
     drb (2.2.0)
       ruby2_keywords
     i18n (1.14.1)
       concurrent-ruby (~> 1.0)
-    json (2.6.3)
+    json (2.7.1)
     language_server-protocol (3.17.0.3)
-    minitest (5.20.0)
+    minitest (5.21.1)
     mutex_m (0.2.0)
-    parallel (1.23.0)
-    parser (3.2.2.4)
+    parallel (1.24.0)
+    parser (3.3.0.4)
       ast (~> 2.4.1)
       racc
     racc (1.7.3)
     rack (3.0.8)
     rainbow (3.1.1)
     rake (13.1.0)
-    regexp_parser (2.8.2)
+    regexp_parser (2.9.0)
     rexml (3.2.6)
-    rubocop (1.57.2)
+    rubocop (1.60.0)
       json (~> 2.3)
       language_server-protocol (>= 3.17.0)
       parallel (~> 1.10)
-      parser (>= 3.2.2.4)
+      parser (>= 3.3.0.2)
       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)
+      rubocop-ast (>= 1.30.0, < 2.0)
       ruby-progressbar (~> 1.7)
       unicode-display_width (>= 2.4.0, < 3.0)
     rubocop-ast (1.30.0)
       parser (>= 3.2.1.0)
-    rubocop-capybara (2.19.0)
+    rubocop-capybara (2.20.0)
       rubocop (~> 1.41)
-    rubocop-factory_bot (2.24.0)
-      rubocop (~> 1.33)
-    rubocop-rails (2.22.1)
+    rubocop-factory_bot (2.25.1)
+      rubocop (~> 1.41)
+    rubocop-rails (2.23.1)
       activesupport (>= 4.2.0)
       rack (>= 1.1)
       rubocop (>= 1.33.0, < 2.0)
-    rubocop-rspec (2.25.0)
+      rubocop-ast (>= 1.30.0, < 2.0)
+    rubocop-rspec (2.26.1)
       rubocop (~> 1.40)
       rubocop-capybara (~> 2.17)
       rubocop-factory_bot (~> 2.22)

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,392 @@ 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
+Style/MultilineTernaryOperator:
+  Enabled: false
+Style/RedundantReturn:
+  Enabled: false
+Style/ClassCheck:
+  Enabled: false
+Style/LineEndConcatenation:
+  Enabled: false
+Style/RedundantFetchBlock:
+  Enabled: false
+Style/FloatDivision:
+  Enabled: false
+Layout/MultilineOperationIndentation:
+  Enabled: false
+Style/SoleNestedConditional:
+  Enabled: false
+Rails/FindEach:
+  Enabled: false
+Style/RedundantParentheses:
+  Enabled: false
+Style/IfInsideElse:
+  Enabled: false
+Style/EmptyCaseCondition:
+  Enabled: false
+Style/TernaryParentheses:
+  EnforcedStyle: require_parentheses_when_complex
+Style/HashLikeCase:
+  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.1'.freeze
+  VERSION = '2.0.1'.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.57.2'
+  spec.required_ruby_version = '>= 3.2'
+
+  spec.add_dependency 'rubocop', '~> 1.60.0'
   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.1
+  version: 2.0.1
 platform: ruby
 authors:
 - Scott Livingstone
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2023-11-10 00:00:00.000000000 Z
+date: 2024-01-17 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rubocop
@@ -16,14 +16,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 1.57.2
+        version: 1.60.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 1.57.2
+        version: 1.60.0
 - !ruby/object:Gem::Dependency
   name: rubocop-rails
   requirement: !ruby/object:Gem::Requirement
@@ -117,7 +117,7 @@ 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:
   - - ">="