config/cookstyle.yml in cookstyle-1.4.0 vs config/cookstyle.yml in cookstyle-2.0.0
- old
+ new
@@ -12,11 +12,11 @@
Enabled: false
Style/FileName:
Enabled: false
Style/Alias:
Enabled: false
-Style/SpaceBeforeFirstArg:
+Layout/SpaceBeforeFirstArg:
Enabled: false
Style/ClassAndModuleChildren:
Enabled: false
Style/DoubleNegation:
Enabled: false
@@ -49,11 +49,11 @@
Enabled: false
# This often triggers no matter what you do with inspec/serverspec matchers
Lint/AmbiguousRegexpLiteral:
Exclude:
- - '**/*/test/**/*'
+ - '**/test/**/*'
# some names are not ascii and this prevents copyright comments
Style/AsciiComments:
Enabled: false
@@ -101,11 +101,11 @@
Style/NumericPredicate:
Enabled: true
EnforcedStyle: comparison
# this is bad %w(something another_thing one_more)
-Style/SpaceInsideArrayPercentLiteral:
+Layout/SpaceInsideArrayPercentLiteral:
Enabled: true
# disable this until there's an autocorrect (0.47 has one)
Bundler/OrderedGems:
Enabled: false
@@ -115,5 +115,26 @@
Enabled: true
# This results in very confusing code to read with little perf benefit
Performance/Casecmp:
Enabled: false
+
+# maintain the previous array behavior in previous cookstyle releases
+Style/PercentLiteralDelimiters:
+ PreferredDelimiters:
+ '%': ()
+ '%i': ()
+ '%I': ()
+ '%q': ()
+ '%Q': ()
+ '%r': '{}'
+ '%s': ()
+ '%w': ()
+ '%W': ()
+ '%x': ()
+
+# backwards conditions are hard to read
+Style/YodaCondition:
+ Enabled: true
+
+Layout/EmptyLinesAroundExceptionHandlingKeywords:
+ Enabled: true