fera-api 0.1.1 → 0.1.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: e29c0c0d0d005fc442519426dfbdf44d7f79188251f2b077a06b98b62d8108b7
-  data.tar.gz: 1cbf38f201a769c0f2148363034ab1d91affc2bcffc5778413eb2651276410fc
+  metadata.gz: 340f111ef9ae910374f8b8556b80a4c7cea4a982be22c807e1330b52f5a222a8
+  data.tar.gz: 758fe26ac14a9a4012039d14800e657690d821862eda47ea8e0866ea714a16d1
 SHA512:
-  metadata.gz: 02f31cc04ea2d869ba3e8531146a1c515469e9ca9178302bc699f43473bb75970a7d0ac92ad98293f298681edac44b546def1cfbd620d68cc05169997c60d31c
-  data.tar.gz: 4f3d7ea4e03bccbf0b30da1aa8fa2640de2b8b1fb3486f49b54683f5c291139e25e4350bbd6219a9f3d76630f14054dc1a924405ca4f8a978729524fca5461dc
+  metadata.gz: db7db0bd5a14e64dce22219b385d21db5675ddc5dc07196756c2a88dc20cb628786a48e911b5ad638b741c2124be96e7008dff341366d4a7ab83e827b767d5b5
+  data.tar.gz: 53ea2d445404901fc1377edd5720f186e3cccad97eafd420f4e13959fedb55d8778ca39847e3eb2622226b06b9261e5fbc38417f3d23a523acfe083701f84303

data/.rubocop.yml

@@ -1,56 +1,173 @@
-Style/StringLiterals:
-  Enabled: true
-  EnforcedStyle: double_quotes
-Style/StringLiteralsInInterpolation:
-  Enabled: true
-  EnforcedStyle: double_quotes
-Style/FrozenStringLiteralComment:
+# Base was copied from https://github.com/sider/meowcop/blob/master/config/rubocop.yml which is what Sider uses by default
+AllCops:
+  TargetRubyVersion: 2.3
+  Exclude:
+    - 'vendor/**/*'
+    - 'db/schema.rb'
+    - 'node_modules/**/*'
+    - 'tmp/**/*'
+    - 'lib/tasks/one_offs/**/*' # We don't really care about linting one-off tasks
+    - 'app/migrators/**/*' # Migrators are one-off tasks so there isn't much need to lint them
+  DisplayCopNames: true
+  DisplayStyleGuide: true
+  NewCops: enable
+
+Lint/NonLocalExitFromIterator:
+  Enabled: false # We actually believe that it is more readable with premature returns
+Layout/BeginEndAlignment:
+  Enabled: false # We believe it's more readable to have begin blocks aligned to the end blocks (and also helps with code smells)
+Lint/AmbiguousRegexpLiteral:
   Enabled: false
-Layout/LeadingCommentSpace:
+Lint/AssignmentInCondition:
   Enabled: false
-Layout/LineLength:
-  Max: 200
-# If AbcSize > 50, really really really really really really really complex
 Metrics/AbcSize:
-  Max: 50
+  Max: 60
 # Use the default setting
-Metrics/BlockNesting:
-  Enabled: true
 Metrics/BlockLength:
   Max: 200
   Exclude:
     # RuboCop's default
     - 'Rakefile'
     - '**/*.rake'
-    - 'spec/**/*.rb'
-    # In many cases, config/routes.rb has very long blocks
+    # In many cases, config/routes.rb has very long block
     - 'config/routes.rb'
 Metrics/ClassLength:
   Enabled: false
 Metrics/CyclomaticComplexity:
-  Max: 30
+  Max: 60
 Metrics/MethodLength:
   Enabled: false
 Metrics/ModuleLength:
   Enabled: false
 Metrics/ParameterLists:
-  Enabled: true
-  Max: 4
+  Max: 5
 Metrics/PerceivedComplexity:
   Enabled: false
-Layout/TrailingEmptyLines:
+Layout/ArgumentAlignment:
+  Enabled: false
+Layout/HashAlignment:
+  Enabled: false
+Layout/ParameterAlignment:
+  Enabled: false
+Layout/IndentationWidth:
+  Enabled: false
+Layout/FirstHashElementIndentation:
+  Enabled: false
+Layout/HeredocIndentation:
+  Enabled: false
+Layout/LineLength:
+  Max: 200
+  AllowedPatterns: ['^\s*#', '^\s*([a-zA-Z]*[lL]og[a-zA-Z]*)[ (]', '^\s*[''"]', '.*#.{30,300}', '^\s*[a-z_]:\s*".+",?[}\s]*$']
+Layout/MultilineArrayBraceLayout:
+  Enabled: false
+Layout/MultilineAssignmentLayout:
+  Enabled: false
+Layout/MultilineBlockLayout:
+  Enabled: false
+Layout/MultilineHashBraceLayout:
+  Enabled: false
+Layout/MultilineMethodCallBraceLayout:
+  Enabled: false
+Layout/MultilineMethodCallIndentation:
   Enabled: false
-Layout/TrailingWhitespace:
+Layout/MultilineMethodDefinitionBraceLayout:
   Enabled: false
-Layout/BlockAlignment:
+Layout/MultilineOperationIndentation:
   Enabled: false
-Layout/DefEndAlignment:
+Layout/RescueEnsureAlignment:
   Enabled: false
-Layout/EndAlignment:
+Layout/SpaceAfterNot:
   Enabled: false
-Layout/ClosingHeredocIndentation:
+Layout/SpaceBeforeBlockBraces:
   Enabled: false
-Layout/LeadingEmptyLines:
+Layout/SpaceInLambdaLiteral:
   Enabled: false
 Layout/SpaceInsideStringInterpolation:
   EnforcedStyle: space
+Naming/PredicateName:
+  Enabled: false # As of February 2022 we now want some methods to have is_ prefix so it is congruent with JavaScript
+Naming/VariableNumber:
+  Enabled: false
+Style/AsciiComments:
+  Enabled: false # It's 2022, emojis are everywhere
+Style/Alias:
+  EnforcedStyle: prefer_alias_method
+Style/CaseLikeIf:
+  Enabled: false
+Style/CommentAnnotation:
+  Enabled: false # Don't really care about being picky on comments like this. Not worth slowing down the developer.
+Style/CommentedKeyword:
+  Enabled: false # We want to allow comments on single-line method definitions like `# @alias`
+Style/ConstantVisibility:
+  Enabled: false
+Style/Copyright:
+  Enabled: false
+Style/DisableCopsWithinSourceCodeDirective:
+  Enabled: false # Allow disabling cops and we will use PRs to determine whether it should be enforced
+Style/Documentation:
+  Enabled: false # We will decide in PR whether public method should be documented or not.
+Style/DocumentationMethod:
+  Enabled: false # We will decide in PR whether public method should be documented or not.
+Style/EmptyElse:
+  AllowComments: true # When a comment is there then it means the programmer thought about the case so we can ignore it.
+  EnforcedStyle: empty  # When a comment or nil is there it means the programmer thought about the case, so there is no need for concern
+Style/FloatDivision:
+  Enabled: false # Sometimes we're trying to force the number to become a float and it could be nil
+Style/FrozenStringLiteralComment:
+  Enabled: false
+Style/GlobalVars:
+  Enabled: false # If we're using global variables, we have a darn good reason or it should be blocked in the PR
+Style/IfUnlessModifier:
+  Enabled: false # This isn't always easier to read. Sometimes you want to put the IF in front of the block of code to make it more clear.
+Style/ImplicitRuntimeError:
+  Enabled: false # These are actually readable as long as we all know that they represent a RuntimeError
+Style/InlineComment:
+  Enabled: false # Trailing inline comments are pretty compact and more readable
+Style/Lambda:
+  Enabled: false
+Style/LambdaCall:
+  Enabled: false
+Style/MethodCallWithArgsParentheses:
+  Enabled: false
+Style/MethodCallWithoutArgsParentheses:
+  Enabled: true # We use whatever is more readable
+Style/MethodCalledOnDoEndBlock:
+  Enabled: false # Actually don't mind these - they look more readable
+Style/MissingElse:
+  Enabled: false # Sometimes it's obvious we don't care about the else case (e.g. if we're prematurely returning from method)
+Style/MultilineBlockChain:
+  Enabled: false # Sometimes these are more readable
+Style/MutableConstant:
+  Enabled: false
+Style/MapToHash:
+  Enabled: false # We believe it's more readable to use map vs hash with block syntax
+Style/NumericPredicate:
+  Enabled: false # .zero? and .positive? are a bit less readable than == 0 IMO
+Style/OptionHash:
+  Enabled: false # It is more intuitive to pass option hashes to other objects in many times
+Style/OpenStructUse:
+  Enabled: false
+Style/RedundantRegexpCharacterClass:
+  Enabled: false # No need to make regexp less readable than they already are by reducing redundancy
+Style/RedundantRegexpEscape:
+  Enabled: false # No need to make regexp less readable than they already are by reducing redundancy
+Style/RedundantSelf:
+  Enabled: false # Sometimes it's actually more readable to explicitly reference self
+Style/RegexpLiteral:
+  Enabled: false
+Style/ReturnNil:
+  Enabled: false # Returning nil is more clear what is happening and can prevent issues with returning mixed types
+Style/Send:
+  Enabled: false
+Style/SingleLineMethods:
+  Enabled: false # Sometimes these are more compact and easier to read. No need to spam our file with simple method definitions.
+Style/StringLiterals:
+  Enabled: false
+Style/SymbolArray:
+  Enabled: false
+Style/TernaryParentheses:
+  Enabled: false
+Style/TrailingCommaInArrayLiteral:
+  EnforcedStyleForMultiline: consistent_comma # It makes managing arrays much easier when we have commas at the end
+Style/TrailingCommaInHashLiteral:
+  EnforcedStyleForMultiline: consistent_comma # It makes managing hashes much easier when we have commas at the end

data/.ruby-version

@@ -0,0 +1 @@
+2.7.6

data/.spellr.yml

@@ -0,0 +1,9 @@
+includes:
+  - lib/**/*
+excludes:
+  - node_modules/**/*
+  - app/**/*.min*
+  - "**/*.map"
+word_minimum_length: 4 # any words shorter than this will be ignored
+key_minimum_length: 6 # any strings shorter than this won't be considered non-word strings
+key_heuristic_weight: 5 # higher values mean strings are more likely to be considered words or non-words by the classifier.