rubocop-iknow 0.0.7 → 0.0.10

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: cab74c8ba8cf2dc6274a65203c0e27ab58e687f038c5a8cdd1b2e1b692fd2a94
-  data.tar.gz: 119c8bc5fa399b06873e550e124f8c5ae5c5043483f8f85297f459fb17dcbbd5
+  metadata.gz: eb6f366b0e1a2f83c5ef514743ea9c10f459747f726637e52147ca409d37b427
+  data.tar.gz: 22fdcef2d05cc5c9e7831b6cfff7320138081b9eeab7381729da92f0314f8fee
 SHA512:
-  metadata.gz: 6baa63db33e31ee914f8f03df5e5a75867ca0b19dd8bdbb2de79dc94bbb1b81f1c7350aa2c016c7e6d709ff0fa641361be50f64cc9dc2f31b0b3151f92051eaf
-  data.tar.gz: 9bb7effb6f4176f85e285de8ee675ae9ef2bf3d274e4cc7b7f11f0e6c6954ded24f98d2abfcbc1700ad0b6a486fe8e05e76dd83a1631e78c5b2cb841c2fd927b
+  metadata.gz: eaf65f18cb6bb6f87c4da15e51c095b07b60bb709628af78d11a2ca727559574358d3d5df045c71f714bd1693e7efad0598d4c42a731222f349663e5ac4d017e
+  data.tar.gz: a7d622e02ccc79467c17c8a71c8a0f438edad535ca8a78c1567d365fea1a1141acac7f0bd97094df2ff7155009fb51155efcabfc4c6e624460727707a9550159

data/rubocop.yml

@@ -1,190 +1,285 @@
----
 AllCops:
   TargetRubyVersion: 2.5
   NewCops: enable
+
+Layout/EmptyLineAfterMagicComment:
+  Severity: error
+
+Layout/EmptyLinesAroundExceptionHandlingKeywords:
+  Enabled: false
+
+Layout/FirstArgumentIndentation:
+  # This forbids indentation relative to the expression
+  Enabled: false
+
+Layout/FirstHashElementIndentation:
+  # This one in particular forbids what Emacs does by default for multi-line
+  # hash entries
+  Enabled: false
+
+Layout/HashAlignment:
+  EnforcedHashRocketStyle:
+  - table
+  - key
+  EnforcedColonStyle:
+  - table
+  - key
+
 Layout/LineLength:
   # Max: 100
   Enabled: false # Potentially bring this back later
-Metrics/BlockLength:
-  Enabled: false
-Style/AsciiComments:
-  Enabled: false
-Style/Documentation:
+
+Layout/MultilineHashBraceLayout:
   Enabled: false
-Style/BlockDelimiters:
+
+Layout/MultilineMethodCallBraceLayout:
+  # Rules for formatting hash and method call braces: let's avoid these unless we
+  # can find something unambiguously positive.
   Enabled: false
-Style/TernaryParentheses:
+
+Layout/MultilineMethodCallIndentation:
+  # Cannot be configured to indent relative to expression.
   Enabled: false
-Style/RedundantReturn:
+
+Layout/MultilineMethodDefinitionBraceLayout:
+  # None of their options are lovely
   Enabled: false
-Style/RedundantSelf:
+
+Layout/SpaceInsideRangeLiteral:
   Enabled: false
-Style/GuardClause:
+
+Layout/TrailingWhitespace:
+  Severity: error
+
+Lint/AmbiguousBlockAssociation:
+  # In practise this rule seems overly restrictive for
+  # DSLs like RSpec that avoid parens.
   Enabled: false
-Style/IfUnlessModifier:
+
+Lint/EmptyWhen:
   Enabled: false
-Style/WhileUntilModifier:
+
+Lint/MissingSuper:
+  Severity: error
+
+Lint/ParenthesesAsGroupedExpression:
   Enabled: false
-Style/RaiseArgs:
-  EnforcedStyle: compact
-Style/RedundantException:
+
+Lint/RaiseException:
+  Enabled: true
+
+Lint/ShadowingOuterLocalVariable:
+  # I like this rule, but can see arguments against it
+  Enabled: true
+
+Lint/StructNewOverride:
   Enabled: false
-Style/SpecialGlobalVars:
+
+Lint/UnusedBlockArgument:
+  AllowUnusedKeywordArguments: true
+
+Lint/UnusedMethodArgument:
+  AllowUnusedKeywordArguments: true
+  IgnoreEmptyMethods: true
+
+Metrics/AbcSize:
   Enabled: false
-Style/EmptyCaseCondition:
+
+Metrics/BlockLength:
   Enabled: false
-Style/RedundantCondition:
+
+Metrics/ClassLength:
   Enabled: false
-Metrics/AbcSize:
+
+Metrics/CyclomaticComplexity:
   Enabled: false
+
 Metrics/MethodLength:
   Enabled: false
-Metrics/CyclomaticComplexity:
+
+Metrics/ModuleLength:
   Enabled: false
+
+Metrics/ParameterLists:
+  Enabled: false
+
 Metrics/PerceivedComplexity:
   Enabled: false
-Metrics/ClassLength:
+
+Naming/AccessorMethodName:
   Enabled: false
-Metrics/ModuleLength:
+
+Naming/MethodParameterName:
   Enabled: false
-Style/Lambda:
+
+Naming/PredicateName:
+  NamePrefix: [is_]
+
+Naming/RescuedExceptionsVariableName:
   Enabled: false
-Style/LambdaCall:
+
+Naming/VariableNumber:
   Enabled: false
-Style/MultilineBlockChain:
+
+Style/AccessorGrouping:
   Enabled: false
-Style/WordArray:
+
+Style/AsciiComments:
   Enabled: false
-Style/SymbolArray:
+
+Style/BlockDelimiters:
   Enabled: false
-Style/PreferredHashMethods:
-  EnforcedStyle: verbose
-Metrics/ParameterLists:
+
+Style/ClassAndModuleChildren:
   Enabled: false
-Layout/MultilineMethodCallIndentation:
-  # Cannot be configured to indent relative to expression.
+
+Style/ClassVars:
+  # Class vars are usually bad practice. Not universally.
   Enabled: false
-Style/RescueModifier:
-  # I can see valid uses for this but they're far and few between.
-  # That said, nobody in our team is going to use one of these unless
-  # it's a valid case
+
+Style/CommentedKeyword:
   Enabled: false
-Style/ExplicitBlockArgument:
-  # This goes too far by demanding expensive block allocation always
+
+Style/Documentation:
   Enabled: false
-Lint/UnusedBlockArgument:
-  AllowUnusedKeywordArguments: true
-Lint/UnusedMethodArgument:
-  AllowUnusedKeywordArguments: true
-  IgnoreEmptyMethods: true
-Lint/EmptyWhen:
+
+Style/EmptyCaseCondition:
   Enabled: false
-Lint/RaiseException:
-  Enabled: true
-Lint/StructNewOverride:
+
+Style/EmptyElse:
   Enabled: false
-Style/ClassAndModuleChildren:
+
+Style/ExplicitBlockArgument:
+  # This goes too far by demanding expensive block allocation always
   Enabled: false
-Style/NonNilCheck:
+
+Style/FrozenStringLiteralComment:
+  Severity: error
+
+Style/GuardClause:
+  # This is too prescriptive
   Enabled: false
+
+Style/HashEachMethods:
+  Enabled: true
+
 Style/HashSyntax:
   # We explicitly choose to use rocket style syntax when the Hash is being used
   # to represent some other sort of data structure (trees, pairs, etc).
   Enabled: false
-Style/HashEachMethods:
-  Enabled: true
+
 Style/HashTransformKeys:
   Enabled: true
+
 Style/HashTransformValues:
   Enabled: true
-Layout/FirstArgumentIndentation:
-  # This forbids indentation relative to the expression
-  Enabled: false
-Style/SafeNavigation:
-  Enabled: false
-Layout/EmptyLinesAroundExceptionHandlingKeywords:
+
+Style/IfUnlessModifier:
   Enabled: false
-Layout/SpaceInsideRangeLiteral:
+
+Style/KeywordParametersOrder:
   Enabled: false
-Style/SymbolProc:
+
+Style/Lambda:
+  Enabled: true
+  EnforcedStyle: literal
+
+Style/LambdaCall:
+  # restricts foo.() syntax, which we favour
   Enabled: false
-Lint/ParenthesesAsGroupedExpression:
+
+Style/MultilineBlockChain:
   Enabled: false
-Style/EmptyElse:
+
+Style/MutableConstant:
+  Severity: error
+
+Style/NonNilCheck:
   Enabled: false
+
 Style/NumericLiterals:
   Enabled: false
+
 Style/NumericPredicate:
   Enabled: false
-Style/TrailingCommaInArrayLiteral:
-  EnforcedStyleForMultiline: consistent_comma
-Style/TrailingCommaInHashLiteral:
-  EnforcedStyleForMultiline: comma
-Style/TrailingCommaInArguments:
-  EnforcedStyleForMultiline: comma
-Style/TrailingUnderscoreVariable:
-  # I think this does more harm than good by hiding the array destructuring
-  Enabled: false
-Style/SymbolLiteral:
-  Enabled: false
-Style/KeywordParametersOrder:
-  Enabled: false
-Style/StringConcatenation:
+
+Style/PreferredHashMethods:
+  EnforcedStyle: verbose
+
+Style/RaiseArgs:
+  EnforcedStyle: compact
+
+Style/RedundantCondition:
   Enabled: false
-Style/AccessorGrouping:
+
+Style/RedundantException:
   Enabled: false
-Naming/AccessorMethodName:
+
+Style/RedundantReturn:
+  # allow redundant return keyword where appropriate to convey meaning
   Enabled: false
-Naming/MethodParameterName:
+
+Style/RedundantSelf:
+  # allow redundant self keyword where appropriate to convey meaning
   Enabled: false
-Naming/VariableNumber:
+
+Style/RescueModifier:
+  # I can see valid uses for this but they're far and few between.
+  # That said, nobody in our team is going to use one of these unless
+  # it's a valid case
   Enabled: false
-Naming/PredicateName:
-  NamePrefix: ['is_']
-Naming/RescuedExceptionsVariableName:
+
+Style/SafeNavigation:
   Enabled: false
+
 Style/SingleLineMethods:
   Enabled: false
-Style/CommentedKeyword:
-  Enabled: false
 
-Layout/HashAlignment:
-  EnforcedHashRocketStyle:
-    - table
-    - key
-  EnforcedColonStyle:
-    - table
-    - key
-
-Style/ClassVars:
-  # Class vars are usually bad practice. Not universally.
+Style/SpecialGlobalVars:
   Enabled: false
 
-Layout/MultilineMethodDefinitionBraceLayout:
-  # None of their options are lovely
+Style/StringConcatenation:
   Enabled: false
 
 Style/StringLiterals:
   Enabled: true
+  Severity: error
+
 Style/StringLiteralsInInterpolation:
   Enabled: true
+  Severity: error
 
-Lint/AmbiguousBlockAssociation:
-  # In practise this rule seems overly restrictive for
-  # DSLs like RSpec that avoid parens.
+Style/SymbolArray:
   Enabled: false
 
-Lint/ShadowingOuterLocalVariable:
-  # I like this rule, but can see arguments against it
+Style/SymbolLiteral:
+  Enabled: false
+
+Style/SymbolProc:
+  Enabled: false
+
+Style/TernaryParentheses:
   Enabled: true
+  EnforcedStyle: require_parentheses_when_complex
 
-# Rules for formatting hash and method call braces: let's avoid these unless we
-# can find something unambiguously positive.
-Layout/MultilineMethodCallBraceLayout:
+Style/TrailingCommaInArguments:
+  Severity: error
+  EnforcedStyleForMultiline: comma
+
+Style/TrailingCommaInArrayLiteral:
+  Severity: error
+  EnforcedStyleForMultiline: consistent_comma
+
+Style/TrailingCommaInHashLiteral:
+  Severity: error
+  EnforcedStyleForMultiline: comma
+
+Style/TrailingUnderscoreVariable:
+  # I think this does more harm than good by hiding the array destructuring
   Enabled: false
-Layout/MultilineHashBraceLayout:
+
+Style/WhileUntilModifier:
   Enabled: false
-Layout/FirstHashElementIndentation:
-  # This one in particular forbids what Emacs does by default for multi-line
-  # hash entries
+
+Style/WordArray:
   Enabled: false

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: rubocop-iknow
 version: !ruby/object:Gem::Version
-  version: 0.0.7
+  version: 0.0.10
 platform: ruby
 authors:
 - iKnow Team
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2020-10-08 00:00:00.000000000 Z
+date: 2022-09-05 00:00:00.000000000 Z
 dependencies: []
 description: 
 email: edge@iknow.jp
@@ -36,7 +36,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.0.3
+rubygems_version: 3.0.3.1
 signing_key: 
 specification_version: 4
 summary: Rubocop Configuration used with iKnow Projects