rubocop-lint.yml in gitlab-styles-3.4.0 vs rubocop-lint.yml in gitlab-styles-4.0.0

- old
+ new

@@ -45,14 +45,10 @@ # Checks for the presence of `when` branches without a body. Lint/EmptyWhen: Enabled: true -# END blocks should not be placed inside method definitions. -Lint/EndInMethod: - Enabled: true - # Do not use return in an ensure block. Lint/EnsureReturn: Enabled: true # Checks for flip flops. @@ -65,14 +61,10 @@ # The number of parameters to format/sprint must match the fields. Lint/FormatParameterMismatch: Enabled: true -# This cop checks for *rescue* blocks with no body. -Lint/HandleExceptions: - Enabled: false - # Checks for adjacent string literals on the same line, which could better be # represented as a single string literal. Lint/ImplicitStringConcatenation: Enabled: true @@ -103,15 +95,27 @@ # Checks for method calls with a space before the opening parenthesis. Lint/ParenthesesAsGroupedExpression: Enabled: true +# Checks for raise or fail statements which are raising Exception class. +Lint/RaiseException: + Enabled: true + # Checks for `rand(1)` calls. Such calls always return `0` and most likely # a mistake. Lint/RandOne: Enabled: true +# This cop checks for unneeded usages of splat expansion +Lint/RedundantSplatExpansion: + Enabled: false + +# Checks for Object#to_s usage in string interpolation. +Lint/RedundantStringCoercion: + Enabled: true + # Use parentheses in the method call to avoid confusion about precedence. Lint/RequireParentheses: Enabled: true # Avoid rescuing the Exception class. @@ -129,24 +133,24 @@ # This cop looks for use of the same name as outer local variables # for block arguments or block local variables. Lint/ShadowingOuterLocalVariable: Enabled: false -# Checks for Object#to_s usage in string interpolation. -Lint/StringConversionInInterpolation: +# Checks unexpected overrides of the Struct built-in methods via Struct.new. +Lint/StructNewOverride: Enabled: true +# This cop checks for *rescue* blocks with no body. +Lint/SuppressedException: + Enabled: false + # Do not use prefix `_` for a variable that is used. Lint/UnderscorePrefixedVariableName: Enabled: true # This cop checks for using Fixnum or Bignum constant Lint/UnifiedInteger: Enabled: true - -# This cop checks for unneeded usages of splat expansion -Lint/UnneededSplatExpansion: - Enabled: false # Unreachable code. Lint/UnreachableCode: Enabled: true