retrieva-cop 0.1.3 → 0.1.9

checksums.yaml

@@ -1,7 +1,7 @@
 ---
-SHA1:
-  metadata.gz: 415c6332117a5a409ab4aec3d5a32a3b788d4729
-  data.tar.gz: b8e61590d43ded4f1c469724f5a0697450c852c4
+SHA256:
+  metadata.gz: 66598ae9bb80df6b3408ba76bfd59adb921a09216b89dfda3722c0112cdd002b
+  data.tar.gz: 9d95b709699357d50dcc48031fc1c8540f616d88b116a6e0a85e2e91b65d01b1
 SHA512:
-  metadata.gz: d454966509525576a4365d7830b4d3c47bb4693ba41e584a3e1eaeb66abbc91a66f15cb2d0aa682fc833894eddc1da59741f83849f6d67d656d63bf3a61d790c
-  data.tar.gz: 2c2f9bc5d957532785958c3ee0f0260304f30368dbda231d949b045f8d153814b5b782b36ea7f2834ee23aaba40c7252f6e51f166f205b1b320dd60c4fd61e2c
+  metadata.gz: fd17625a485b3c935ae3115f371b5ee2ccfd99d0cc27ef1b80330196c8d6fe0429231ba18e442bc2ed4faf454cdcc80372f4597961db1ed15df09000487d540d
+  data.tar.gz: 4dd0f73aff4a6c74353d7064f48e3df65a921034c120559a68a5669777e2ae5912c7e5d310069db5637f832059c7fe70b4414abeeeadbe97ccc6da881b5c631b

data/config/rails.yml

@@ -1,3 +1,5 @@
+require: rubocop-rails
+
 Rails:
   Enabled: true
 
@@ -6,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

@@ -12,6 +12,7 @@ AllCops:
     - "Capfile"
     - "**/*.spec"  # .spec file should be an RPM spec file not an RSpec file
   DisplayCopNames: true
+  NewCops: enable
 
 #################### Layout ################################
 
@@ -35,7 +36,7 @@ Layout/MultilineMethodCallIndentation:
   EnforcedStyle: indented_relative_to_receiver
 
 # Rails の生成するファイルに存在するので除外
-Layout/SpaceInsideBrackets:
+Layout/SpaceInsideArrayLiteralBrackets:
   Exclude:
     - "config/environments/*"
 
@@ -89,7 +90,7 @@ Metrics/CyclomaticComplexity:
 # * 警告 120文字
 # * 禁止 160文字
 # のイメージ
-Metrics/LineLength:
+Layout/LineLength:
   Max: 160
 
 # 20 行超えるのは migration ファイル以外滅多に無い
@@ -98,6 +99,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 +167,10 @@ Style/HashSyntax:
 Style/IfUnlessModifier:
   Enabled: false
 
+# scope 等は複数行でも lambda ではなく ->{} で揃えた方が見た目が綺麗
+Style/Lambda:
+  EnforcedStyle: literal
+
 # 特に model 内において、ローカル変数とメソッド呼び出しの区別をつけた方が分かりやすい場合が多い
 Style/RedundantSelf:
   Enabled: false
@@ -162,6 +180,10 @@ Style/RedundantSelf:
 Style/RescueModifier:
   AutoCorrect: false
 
+# 無指定だと StandardError を rescue するのは常識の範疇なので。
+Style/RescueStandardError:
+  EnforcedStyle: implicit
+
 # Rubocop は %w[] を押すが、%w() で問題ないとする
 Style/PercentLiteralDelimiters:
   PreferredDelimiters:
@@ -198,7 +220,7 @@ Style/StringLiterals:
     - "db/migrate/*"
 
 # AutoCorrect すると .to_s するが、実際には .to_s が必要ない場合が多いので、手動で行う
-Style/UnneededInterpolation:
+Style/RedundantInterpolation:
   AutoCorrect: false
 
 # 条件式で arr.size > 0 が使われた時に

data/lib/retrieva/cop/version.rb

@@ -1,5 +1,5 @@
 module Retrieva
   module Cop
-    VERSION = "0.1.3"
+    VERSION = "0.1.9"
   end
 end

data/retrieva-cop.gemspec

@@ -20,7 +20,8 @@ Gem::Specification.new do |spec|
   spec.executables   = spec.files.grep(%r{^exe/}) { |f| File.basename(f) }
   spec.require_paths = ["lib"]
 
-  spec.add_dependency "rubocop", "~> 0.50.0"
+  spec.add_dependency "rubocop", ">= 0.72"
+  spec.add_dependency "rubocop-rails", ">= 2.3.2"
   spec.add_dependency "rubocop-rspec", ">= 1.18.0"
   spec.add_development_dependency "bundler"
   spec.add_development_dependency "rake"

metadata

@@ -1,29 +1,43 @@
 --- !ruby/object:Gem::Specification
 name: retrieva-cop
 version: !ruby/object:Gem::Version
-  version: 0.1.3
+  version: 0.1.9
 platform: ruby
 authors:
 - Kei Shiratsuchi
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2017-12-06 00:00:00.000000000 Z
+date: 2020-07-13 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rubocop
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: 0.50.0
+        version: '0.72'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0.72'
+- !ruby/object:Gem::Dependency
+  name: rubocop-rails
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 2.3.2
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
       - !ruby/object:Gem::Version
-        version: 0.50.0
+        version: 2.3.2
 - !ruby/object:Gem::Dependency
   name: rubocop-rspec
   requirement: !ruby/object:Gem::Requirement
@@ -104,8 +118,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubyforge_project: 
-rubygems_version: 2.6.11
+rubygems_version: 3.1.2
 signing_key: 
 specification_version: 4
 summary: RetrievaCop is a RuboCop configuration gem.