onkcop 0.47.1.2 → 0.48.0.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 842bb27a00cbeab57f69808103045d43b603f5a7
-  data.tar.gz: 2ba0ef8f31bc53fd70a7f23565606f3d34f6d3ce
+  metadata.gz: fd88a1e7c280b9906ee97779d4ab30ededc487c7
+  data.tar.gz: 40667f4dccfe647bf057ca1d9ae47761fee83e40
 SHA512:
-  metadata.gz: bdb0eb4be5b8754c8412a4f9e17e62465d2b84750295b7e976362f1b44b18e380adfaef09f993a639cf6b0d015c9380d10fd4c7d3cc0371352e07f8d9d1b3c4f
-  data.tar.gz: 6f29e9370610ee8ca5609951b5408032ab3172ecc5f9ff5b509ec5b2c24173ed5ee36dfe68ad3f097f0f1209f7de32d353b32d2e4bd42530358428b453b8dd11
+  metadata.gz: 5022365b6f6c2c676255435abe44815ad9f2b0d92700432849c58c2344ad4020ba8c37217f6eb699fbb984061c35b4dab07f0e0bd54aeea611f3453fedb150c2
+  data.tar.gz: 5cd30da46be0569c8e6b1e2d9f083fbc543c86942ba566372292d908a68d2bdd15b603eaf424a4504bb0cb1bd5cf619c28bf179229304095ad605b89e6ec25cc

data/CHANGELOG.md

@@ -1,5 +1,19 @@
 # onkcop
 
+## v0.48.0.0 (2017-03-29)
+
+[full changelog](https://github.com/onk/onkcop/compare/v0.47.1.2...v0.48.0.0)
+
+* Update `rubocop` v0.48.0.
+* Disable new `Lint/AmbiguousBlockAssociation` cop.
+* Disable new `Rails/Blank` cop.
+* Disable `Style/SymbolArray`, `StyleWordArray` cop.
+* Change `Style/MultilineMethodCallIndentation` to `indented_relative_to_receiver` style.
+* Exclude RSpec directory from new `Style/MixinGrouping` cop.
+* Ignore `node_modules` dir.
+* Add `TargetRailsVersion` to README and template
+
+
 ## v0.47.1.2 (2017-03-01)
 
 [full changelog](https://github.com/onk/onkcop/compare/v0.47.1.1...v0.47.1.2)

data/README.md

@@ -26,6 +26,8 @@ inherit_gem:
 
 AllCops:
   TargetRubyVersion: 2.4
+  # uncomment if use rails cops
+  # TargetRailsVersion: 5.0
 ```
 
 ```sh

data/config/rails.yml

@@ -1,6 +1,10 @@
 Rails:
   Enabled: true
 
+# v0.48.0 に `||` でエラーになるバグがあるため。 pull/4175
+Rails/Blank:
+  Enabled: false
+
 # slug とか created_by とか、NOT NULL だが create_table 時に
 # default 値を定義できないカラムは存在する。
 Rails/NotNullColumn:

data/config/rspec.yml

@@ -21,7 +21,6 @@ RSpec/InstanceVariable:
   Enabled: false
 
 # 強く 1 example 1 assertion の立場は取らないが、多すぎてもツラいので。
-# aggregate_failures で囲われていたら無視する的なオプション欲しい。
 RSpec/MultipleExpectations:
   Max: 3
 

data/config/rubocop.yml

@@ -1,14 +1,12 @@
-# target_version:
-#   rubocop v0.47.1
-
 # 自動生成されるものはチェック対象から除外する
 AllCops:
   Exclude:
     - "vendor/**/*" # rubocop config/default.yml
     - "db/schema.rb"
+    - "node_modules/**/*"
   DisplayCopNames: true
 
-##################### Style ##################################
+#################### Style #################################
 
 # レキシカルスコープの扱いが alias_method の方が自然。
 # https://ernie.io/2014/10/23/in-defense-of-alias/ のように
@@ -147,9 +145,23 @@ Style/Lambda:
 Style/MethodCalledOnDoEndBlock:
   Enabled: true
 
+# RSpec の matcher を誤検知するため
+#   expect(array).to include("a", "b")
+Style/MixinGrouping:
+  Exclude:
+    - "spec/**/*"
+
+# メソッドチェーン感がより感じられるインデントにする
+Style/MultilineMethodCallIndentation:
+  EnforcedStyle: indented_relative_to_receiver
+
 # 1_000_000 と区切り文字が 2 個以上必要になる場合のみ _ 区切りを必須にする
+# 10_000_00 は許可しない。(これは例えば 10000 ドルをセント単位にする時に便利だが
+# 頻出しないので foolproof に振る
+# TODO: Strict パラメータは warning が出るので一旦無効に。 pull/4180
 Style/NumericLiterals:
   MinDigits: 7
+  # Strict: true
 
 # foo.positive? は foo > 0 に比べて意味が曖昧になる
 # foo.zero? は許可したいけどメソッドごとに指定できないので一括で disable に
@@ -206,6 +218,11 @@ Style/StringLiteralsInInterpolation:
 Style/StringMethods:
   Enabled: true
 
+# %w() と %i() が見分けづらいので Style/WordArray と合わせて無効に。
+# 書き手に委ねるという意味で、Enabled: false にしています。使っても良い。
+Style/SymbolArray:
+  Enabled: false
+
 # 三項演算子は分かりやすく使いたい。
 # () を外さない方が条件式が何なのか読み取りやすいと感じる。
 Style/TernaryParentheses:
@@ -215,6 +232,11 @@ Style/TernaryParentheses:
 Style/TrailingCommaInLiteral:
   EnforcedStyleForMultiline: comma
 
+# %w() と %i() が見分けづらいので Style/SymbolArray と合わせて無効に。
+# 書き手に委ねるという意味で、Enabled: false にしています。使っても良い。
+Style/WordArray:
+  Enabled: false
+
 # 条件式で arr.size > 0 が使われた時に
 #   if !arr.empty?
 #   else
@@ -224,7 +246,18 @@ Style/TrailingCommaInLiteral:
 Style/ZeroLengthPredicate:
   Enabled: false
 
-##################### Lint ##################################
+#################### Lint ##################################
+
+# メソッドの引数に lambda を渡すパターンが検出されるため。
+# この場合は単一引数だし Ambiguous ではない (と思う) ので誤検知っぽい。
+#   some_method ->(a) { a }
+# pull/4191 で修正済み
+#
+# assignment(`=`) が OK で `==`, `!=` が NG なのも考慮漏れっぽく見える。
+#   if [a, b] != array.sort_by { |e| e["id"] }
+# は右から評価されるのを意図しているし違和感無い。
+Lint/AmbiguousBlockAssociation:
+  Enabled: false
 
 # Style/EmptyCaseCondition と同じく網羅の表現力が empty when を認めた方が高いし、
 # 頻出する対象を最初の when で撥ねるのはパフォーマンス向上で頻出する。
@@ -255,7 +288,7 @@ Lint/UnderscorePrefixedVariableName:
 Lint/UnusedMethodArgument:
   Enabled: false
 
-##################### Metrics ##################################
+#################### Metrics ###############################
 
 # 30 まではギリギリ許せる範囲だったけど
 # リリースごとに 3 ずつぐらい下げていきます。20 まで下げたい。
@@ -298,7 +331,7 @@ Metrics/MethodLength:
 Metrics/PerceivedComplexity:
   Max: 8
 
-##################### Security ##################################
+#################### Security ##############################
 
 # 毎回 YAML.safe_load(yaml_str, [Date, Time]) するのは面倒で。。
 Security/YAMLLoad:

data/lib/onkcop/version.rb

@@ -1,3 +1,3 @@
 module Onkcop
-  VERSION = "0.47.1.2"
+  VERSION = "0.48.0.0"
 end

data/onkcop.gemspec

@@ -19,8 +19,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.47.1"
-  spec.add_dependency "rubocop-rspec", ">= 1.12.0"
+  spec.add_dependency "rubocop", "~> 0.48.0"
+  spec.add_dependency "rubocop-rspec", ">= 1.15.0"
   spec.add_development_dependency "bundler"
   spec.add_development_dependency "rake"
 end

data/templates/.rubocop.yml

@@ -8,3 +8,5 @@ inherit_gem:
 
 AllCops:
   TargetRubyVersion: 2.4
+  # uncomment if use rails cops
+  # TargetRailsVersion: 5.0

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: onkcop
 version: !ruby/object:Gem::Version
-  version: 0.47.1.2
+  version: 0.48.0.0
 platform: ruby
 authors:
 - Takafumi ONAKA
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2017-03-01 00:00:00.000000000 Z
+date: 2017-03-29 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rubocop
@@ -16,28 +16,28 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.47.1
+        version: 0.48.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.47.1
+        version: 0.48.0
 - !ruby/object:Gem::Dependency
   name: rubocop-rspec
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 1.12.0
+        version: 1.15.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 1.12.0
+        version: 1.15.0
 - !ruby/object:Gem::Dependency
   name: bundler
   requirement: !ruby/object:Gem::Requirement