retrieva-cop 0.1.7 → 0.1.8

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 46b0e1a36fbea77894bfffec9c5d24623e8d0dc5e1d3282305922ebde4405db9
-  data.tar.gz: 26db029124b6a7f9e7cb6d69cd2ea833ef3019099f9b4d6f2925a8e39af25df2
+  metadata.gz: 0bcffc6358e51c9241235d8fadb4252d497aa45b5b8c29446d7788b3ee9b8f10
+  data.tar.gz: 4b78d137c179cba050ea06e4c62938afd8f435b11fae06a7585d187ea489c0b2
 SHA512:
-  metadata.gz: 8d98202b84bbb13949b70d4216283d1b83363c41d933f5aefdfc4e2486ba8047e003d2f0c249d91fb816e9973817beacd8e7ecd0ee508f4cc5505f8b9aef1cb5
-  data.tar.gz: bc23d925ce2955db809e08e1d3a08550da14bb339d7649b166ae5351147e94020f1c9e53dd1ada83ae9d961cf809aa3208950c69a1a8c2ab9d4d5f684a97c691
+  metadata.gz: 1a2f72ece0ade1249df11cdc9565605fb94642638c4134108c0069ab1dc0d63a2435bb9547244af81b09708552d41e6e9573018c8e9ddeec39c0ecab1010ee31
+  data.tar.gz: 6db6e55b0b32b93a5b8ed8f3256c1d99e27fcc1dc6a0d0f2de65bba4ab79a97cc2f52a47e5ff03f6e5187dad6fdea52f7569cb404b139012a2cff0577042f021

data/config/rails.yml

@@ -8,3 +8,7 @@ Rails:
 Rails/NotNullColumn:
   Enabled: false
 
+# bin/ 以下の実行ファイルではトップレベルで include できて良い
+Style/MixinUsage:
+  Exclude:
+    - 'bin/*'

data/config/rspec.yml

@@ -1,9 +1,13 @@
 require: "rubocop-rspec"
 
-# feature spec で scenario を使っても良い
-Capybara/FeatureMethods:
+# 日本語だと「〜の場合」になるので suffix でないと対応できない
+RSpec/ContextWording:
   Enabled: false
 
+# subject はコピペ可搬性よりもそのまま USAGE であって欲しい
+RSpec/DescribedClass:
+  EnforcedStyle: explicit
+
 # 短い subject は一行で書いて良い
 RSpec/EmptyLineAfterSubject:
   Enabled: false
@@ -22,6 +26,12 @@ RSpec/EmptyLineAfterFinalLet:
 RSpec/ExampleLength:
   Max: 8
 
+# block の方がテスト対象が
+# * `{}` の前後のスペースと相まって目立つ
+# * 普段書く形と同じなので自然に脳内に入ってくる
+RSpec/ExpectChange:
+  EnforcedStyle: block
+
 # 変に名前つけて呼ぶ方が分かりづらい。
 # テスト対象メソッドを呼ぶだけの subject 以外を書かないようにする方が効く。
 RSpec/NamedSubject:

data/config/rubocop.yml

@@ -98,6 +98,19 @@ Metrics/MethodLength:
   Exclude:
     - "db/migrate/*.rb"
 
+# 分岐の数。ガード句を多用しているとデフォルト 7 だと厳しい
+Metrics/PerceivedComplexity:
+  Max: 8
+
+#################### Naming ################################
+
+# 3 文字未満だと指摘されるが、未使用を示す _ や e(rror), b(lock),
+# n(umber) といった 1 文字変数は頻出するし、前置詞(by, to, ...)や
+# よく知られた省略語 (op: operator とか pk: primary key とか) も妥当。
+# 変数 s にどんな文字列かを形容したい場合と、不要な場合とがある＝無効
+Naming/MethodParameterName:
+  Enabled: false
+
 #################### Style #################################
 
 # redirect_to xxx and return のイディオムを維持したい
@@ -153,6 +166,10 @@ Style/HashSyntax:
 Style/IfUnlessModifier:
   Enabled: false
 
+# scope 等は複数行でも lambda ではなく ->{} で揃えた方が見た目が綺麗
+Style/Lambda:
+  EnforcedStyle: literal
+
 # 特に model 内において、ローカル変数とメソッド呼び出しの区別をつけた方が分かりやすい場合が多い
 Style/RedundantSelf:
   Enabled: false
@@ -162,6 +179,10 @@ Style/RedundantSelf:
 Style/RescueModifier:
   AutoCorrect: false
 
+# 無指定だと StandardError を rescue するのは常識の範疇なので。
+Style/RescueStandardError:
+  EnforcedStyle: implicit
+
 # Rubocop は %w[] を押すが、%w() で問題ないとする
 Style/PercentLiteralDelimiters:
   PreferredDelimiters:

data/lib/retrieva/cop/version.rb

@@ -1,5 +1,5 @@
 module Retrieva
   module Cop
-    VERSION = "0.1.7"
+    VERSION = "0.1.8"
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: retrieva-cop
 version: !ruby/object:Gem::Version
-  version: 0.1.7
+  version: 0.1.8
 platform: ruby
 authors:
 - Kei Shiratsuchi
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2019-12-02 00:00:00.000000000 Z
+date: 2019-12-05 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rubocop