Fixed previous commit always causing recompiling if shadow_pokemon.dat doesn't exist, also rubocopping

.rubocop.yml

@@ -21,6 +21,10 @@ Layout/HashAlignment:
   EnforcedHashRocketStyle: table
   EnforcedColonStyle: table
 
+# This interferes with the presentation of some code, notably registered procs.
+Layout/MultilineMethodCallBraceLayout:
+  Enabled: false
+
 # This means hashes and arrays are written the same way, rather than hashes
 # needing to be written like { foo => bar } while arrays are like [foo, bar].
 Layout/SpaceInsideHashLiteralBraces:
@@ -106,6 +110,11 @@ Style/GlobalVars:
 Style/HashSyntax:
   EnforcedStyle: no_mixed_keys
 
+# The alernative is ->(x) { x } which is less English than "lambda". This style
+# makes lambda definitions require the word "lambda".
+Style/Lambda:
+  EnforcedStyle: lambda
+
 # unless just adds mental gymnastics trying to figure out what it actually
 # means. I much prefer if !something.
 Style/NegatedIf:
@@ -139,6 +148,11 @@ Style/SingleLineBlockParams:
 Style/SingleLineMethods:
   Enabled: false
 
+# This requires writing array[n..] instead of array[n..-1], and I think endless
+# ranges look bad.
+Style/SlicingWithRange:
+  Enabled: false
+
 # Single quotes being faster is hardly measurable and only affects parse time.
 # Enforcing double quotes reduces the times where you need to change them
 # when introducing an interpolation or an apostrophe. Use single quotes only if
@@ -154,6 +168,10 @@ Style/SymbolArray:
 Style/WordArray:
   EnforcedStyle: brackets
 
+# Allows procs to be written like { |obj| obj.something } which is clearer.
+Style/SymbolProc:
+  AllowMethodsWithArguments: true
+
 # Parentheses around the condition in a ternary operator helps to differentiate
 # it from the true/false results.
 Style/TernaryParentheses: