onkcop 0.48.1.1 → 0.49.0.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: eb8544568eb34d87eb40827c47d147f70a3b4523
-  data.tar.gz: 832d7217df7e6624ab6b9886602fa0572bd12756
+  metadata.gz: 73fb717715a1d6ff381ce9093eff0cc8861555cb
+  data.tar.gz: 39ca4d5292fa908b35d26971746bcd131fed9602
 SHA512:
-  metadata.gz: c4edd4e51e062133e7767dede4f2296e27e0bf871c6c9cf28378042f2519856fca67e3897b78937ae423b0ac6b63011272f80fa85b8f273db21e01acb59fd86b
-  data.tar.gz: dd69ff5ca1e62c17cba8d6768177d0058fb0a786181a85600f1d1bf15031cf7efdad6adfd5a11758c28da5f7e9659077530f647bfe247b7e8b755696c55d0a13
+  metadata.gz: 8487a087fc8c97fb9e2aa39abbd6b5ef132802f3f2cf1d35101b85ff26bbebd8748c5ba2f822868ce9b30ee34990227662405d51c0e0dcfbd2b5b321cbcdd311
+  data.tar.gz: a9d6df313b3867b9ac8f9889f90c04a12408ce5942c24cf65c1fd85719dc2db8d58197606a575c56611ac36ecd48dbc83b95f8261b92e0dfdb33cb7f802e42c5

data/CHANGELOG.md

@@ -1,7 +1,19 @@
 # onkcop
 
+## v0.49.0.0 (2017-05-25)
+
+[full changelog](https://github.com/onk/onkcop/compare/v0.48.1.1...v0.49.0.0)
+
+* Update `rubocop` v0.49.0 and `rubocop-rspec` v1.15.1
+* Enable `Lint/AmbiguousBlockAssociation` cop except `spec/**/*`
+* Change some cops to `Layout` department
+* Disable new `Style/YodaCondition` cop
+
+
 ## v0.48.1.1 (2017-04-18)
 
+[full changelog](https://github.com/onk/onkcop/compare/v0.48.1.0...v0.48.1.1)
+
 * Disable `Lint/AmbiguousBlockAssociation` cop.
 
 

data/README.md

@@ -27,7 +27,7 @@ inherit_gem:
 AllCops:
   TargetRubyVersion: 2.4
   # uncomment if use rails cops
-  # TargetRailsVersion: 5.0
+  # TargetRailsVersion: 5.1
 ```
 
 ```sh

data/config/rubocop.yml

@@ -6,6 +6,38 @@ AllCops:
     - "node_modules/**/*"
   DisplayCopNames: true
 
+#################### Layout ################################
+
+# メソッドチェーンの改行は末尾に . を入れる
+# REPL に貼り付けた際の暴発を防ぐため
+Layout/DotPosition:
+  EnforcedStyle: trailing
+
+# 桁揃えが綺麗にならないことが多いので migration は除外
+Layout/ExtraSpacing:
+  Exclude:
+    - "db/migrate/*.rb"
+
+# special_inside_parentheses (default) と比べて
+# * 横に長くなりづらい
+# * メソッド名の長さが変わったときに diff が少ない
+Layout/IndentArray:
+  EnforcedStyle: consistent
+
+# ({ と hash を開始した場合に ( の位置にインデントさせる
+# そもそも {} が必要ない可能性が高いが Style/BracesAroundHashParameters はチェックしないことにしたので
+Layout/IndentHash:
+  EnforcedStyle: consistent
+
+# private/protected は一段深くインデントする
+Layout/IndentationConsistency:
+  EnforcedStyle: rails
+
+# メソッドチェーン感がより感じられるインデントにする
+Layout/MultilineMethodCallIndentation:
+  EnforcedStyle: indented_relative_to_receiver
+
+
 #################### Style #################################
 
 # レキシカルスコープの扱いが alias_method の方が自然。
@@ -61,11 +93,6 @@ Style/CollectionMethods:
 Style/Documentation:
   Enabled: false
 
-# メソッドチェーンの改行は末尾に . を入れる
-# REPL に貼り付けた際の暴発を防ぐため
-Style/DotPosition:
-  EnforcedStyle: trailing
-
 # !! のイディオムは積極的に使う
 Style/DoubleNegation:
   Enabled: false
@@ -87,11 +114,6 @@ Style/EmptyElse:
 Style/EmptyMethod:
   EnforcedStyle: expanded
 
-# 桁揃えが綺麗にならないことが多いので migration は除外
-Style/ExtraSpacing:
-  Exclude:
-    - "db/migrate/*.rb"
-
 # いずれかに揃えるのならば `sprintf` や `format` より String#% が好きです
 Style/FormatString:
   EnforcedStyle: percent
@@ -121,21 +143,6 @@ Style/IfInsideElse:
 Style/IfUnlessModifier:
   Enabled: false
 
-# special_inside_parentheses (default) と比べて
-# * 横に長くなりづらい
-# * メソッド名の長さが変わったときに diff が少ない
-Style/IndentArray:
-  EnforcedStyle: consistent
-
-# ({ と hash を開始した場合に ( の位置にインデントさせる
-# そもそも {} が必要ない可能性が高いが Style/BracesAroundHashParameters はチェックしないことにしたので
-Style/IndentHash:
-  EnforcedStyle: consistent
-
-# private/protected は一段深くインデントする
-Style/IndentationConsistency:
-  EnforcedStyle: rails
-
 # scope 等は複数行でも lambda ではなく ->{} で揃えた方が見た目が綺麗
 Style/Lambda:
   EnforcedStyle: literal
@@ -145,10 +152,6 @@ Style/Lambda:
 Style/MethodCalledOnDoEndBlock:
   Enabled: true
 
-# メソッドチェーン感がより感じられるインデントにする
-Style/MultilineMethodCallIndentation:
-  EnforcedStyle: indented_relative_to_receiver
-
 # 1_000_000 と区切り文字が 2 個以上必要になる場合のみ _ 区切りを必須にする
 # 10_000_00 は許可しない。(これは例えば 10000 ドルをセント単位にする時に便利だが
 # 頻出しないので foolproof に振る
@@ -225,6 +228,12 @@ Style/TernaryParentheses:
 Style/TrailingCommaInLiteral:
   EnforcedStyleForMultiline: comma
 
+# 0 <= foo && foo < 5 のように数直線上に並べるのは
+# コードを読みやすくするテクニック。
+# また、self == other, __FILE__ == $0 はイディオムなので許可。
+Style/YodaCondition:
+  Enabled: false
+
 # %w() と %i() が見分けづらいので Style/SymbolArray と合わせて無効に。
 # 書き手に委ねるという意味で、Enabled: false にしています。使っても良い。
 Style/WordArray:
@@ -241,12 +250,13 @@ Style/ZeroLengthPredicate:
 
 #################### Lint ##################################
 
-# メソッドの引数に lambda を渡すパターンが検出されるため。
-# DSL 的な記述 (ActiveRecord の scope や capistrano の set 等) でよく見る。
-#   scope :published, -> { where(published: true) }
-# pull/4237 で修正済み
+# spec 内では
+# expect { subject }.to change { foo }
+# という書き方をよく行うので () を省略したい。
+# { foo } は明らかに change に紐付く。
 Lint/AmbiguousBlockAssociation:
-  Enabled: false
+  Exclude:
+    - "spec/**/*"
 
 # Style/EmptyCaseCondition と同じく網羅の表現力が empty when を認めた方が高いし、
 # 頻出する対象を最初の when で撥ねるのはパフォーマンス向上で頻出する。

data/lib/onkcop/version.rb

@@ -1,3 +1,3 @@
 module Onkcop
-  VERSION = "0.48.1.1"
+  VERSION = "0.49.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.48.1"
-  spec.add_dependency "rubocop-rspec", ">= 1.15.0"
+  spec.add_dependency "rubocop", "~> 0.49.0"
+  spec.add_dependency "rubocop-rspec", ">= 1.15.1"
   spec.add_development_dependency "bundler"
   spec.add_development_dependency "rake"
 end

data/templates/.rubocop.yml

@@ -9,4 +9,4 @@ inherit_gem:
 AllCops:
   TargetRubyVersion: 2.4
   # uncomment if use rails cops
-  # TargetRailsVersion: 5.0
+  # TargetRailsVersion: 5.1

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: onkcop
 version: !ruby/object:Gem::Version
-  version: 0.48.1.1
+  version: 0.49.0.0
 platform: ruby
 authors:
 - Takafumi ONAKA
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2017-04-18 00:00:00.000000000 Z
+date: 2017-05-25 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rubocop
@@ -16,28 +16,28 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.48.1
+        version: 0.49.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.48.1
+        version: 0.49.0
 - !ruby/object:Gem::Dependency
   name: rubocop-rspec
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 1.15.0
+        version: 1.15.1
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 1.15.0
+        version: 1.15.1
 - !ruby/object:Gem::Dependency
   name: bundler
   requirement: !ruby/object:Gem::Requirement
@@ -112,7 +112,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.6.10
+rubygems_version: 2.6.12
 signing_key: 
 specification_version: 4
 summary: OnkCop is a RuboCop configration gem.