onkcop 0.51.0.1 → 0.52.0.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 98ebeeaf9fdd8e7aec88b437c111120bc0b2b3535dce8148de2cb678f4aea372
-  data.tar.gz: d3ff27055afeacdf48eeafb9f7ba44ca9232f772b03f8a6a35293d34d893c881
+  metadata.gz: dcacadb1be034be9f03284c424f2012e367df69582e230119d4ad03b403a5d2b
+  data.tar.gz: b87737a2f5be1d8152459585fb140a151005ec9624c5f49bc812336e39035b92
 SHA512:
-  metadata.gz: d0f9d3c5ebf2f5bc16ae9f281418fa357b4a5a8e99e30d2061564d7c68b7adcb704934e3fd4b2ce8ce9aeec5ccab7950c843eb0db399eecb05fb596aa9cce915
-  data.tar.gz: dd3dda89165695b72873c4cc1892ec6752f6b20aaa5a434b59b4b78e5199056328304edf9739b2a8b31c0b0e8564d575a0157c3f3a3ed03467a14149a0e4e861
+  metadata.gz: e292b2682ff77e8d4c07f032542fbd5f1356eda858c3e20cfd41ffd9ed127ade97b0d7f4d41e3dad42937a4041c3a0d8776c3cb40d32847cc3606b3ff83f6283
+  data.tar.gz: d24900443c0d954fe777919d84d86dc742728ebb1b572c443a48d0095843266c6faf397fb835d0cafaff751086f4739695f4441b85a8aea1ffb79c6511b96447

data/.gitignore

@@ -7,3 +7,4 @@
 /pkg/
 /spec/reports/
 /tmp/
+/.rspec_status

data/.rspec

@@ -0,0 +1 @@
+--require spec_helper

data/.travis.yml

@@ -0,0 +1,4 @@
+language: ruby
+cache: bundler
+rvm:
+  - ruby-head

data/CHANGELOG.md

@@ -1,6 +1,18 @@
 # onkcop
 
-## v0.51.0.1 (2017-10-20)
+## v0.52.0.0 (2017-12-20)
+
+* Update `rubocop` v0.52.0 and `rubocop-rspec` v1.21.0
+* Remove `DisplayCopNames` configuration
+* Enable new `Style/ClassStructure` cop
+* Disalbe `Style/FormatStringToken` cop
+* Change `Layout/SpaceBeforeBlockBraces` cop's empty braces style to `space`
+* Change `Style/RescueStandardError` cop to implicit style
+* Change `RSpec/MultipleExpectations` cop to `AggregateFailuresByDefault` style
+* Exclude `bin/setup`, `bin/update` from `Style/MixinUsage` cop
+
+
+## v0.51.0.1 (2017-11-11)
 
 * Update `rubocop-rspec` to v1.20.0
 * Disable new `RSpec/ContextWording` cop

data/README.md

@@ -1,5 +1,6 @@
 # onkcop
 [![Gem Version](https://badge.fury.io/rb/onkcop.svg)](https://badge.fury.io/rb/onkcop)
+[![Build Status](https://travis-ci.org/onk/onkcop.svg?branch=master)](https://travis-ci.org/onk/onkcop)
 
 OnkCop is a RuboCop configration gem.
 

data/Rakefile

@@ -1 +1,4 @@
 require "bundler/gem_tasks"
+require "rspec/core/rake_task"
+RSpec::Core::RakeTask.new(:rspec)
+task :default => [:rspec]

data/config/rspec.yml

@@ -32,9 +32,14 @@ RSpec/ImplicitExpect:
 RSpec/InstanceVariable:
   Enabled: false
 
-# 強く 1 example 1 assertion の立場は取らないが、多すぎてもツラいので。
+# spec_helper で meta[:aggregate_failures] を設定することで
+# aggregate_failures が全ての spec で有効になる。
+#
+# ほぼ MultipleExpectations についてはチェックされなくなる設定なので注意。
+# パフォーマンスの問題さえ無ければ 1 example 1 assertion にしておく方が
+# 読みやすいテストになりやすいので、そこはレビューで担保していく必要がある。
 RSpec/MultipleExpectations:
-  Max: 3
+  AggregateFailuresByDefault: true
 
 # 変に名前つけて呼ぶ方が分かりづらい。
 # テスト対象メソッドを呼ぶだけの subject 以外を書かないようにする方が効く。

data/config/rubocop.yml

@@ -4,10 +4,17 @@ AllCops:
     - "node_modules/**/*" # rubocop config/default.yml
     - "vendor/**/*"       # rubocop config/default.yml
     - "db/schema.rb"
-  DisplayCopNames: true
 
 #################### Layout ################################
 
+# メソッドをグループ分けして書き順を揃えておくと読みやすくなる。
+# 多少のツラミはあるかもしれない。
+# TODO: Categories を調整することで
+# https://github.com/pocke/rubocop-rails-order_model_declarative_methods
+# を再現できそう。
+Layout/ClassStructure:
+  Enabled: true
+
 # メソッドチェーンの改行は末尾に . を入れる
 # REPL に貼り付けた際の暴発を防ぐため
 Layout/DotPosition:
@@ -37,6 +44,130 @@ Layout/IndentationConsistency:
 Layout/MultilineMethodCallIndentation:
   EnforcedStyle: indented_relative_to_receiver
 
+# デフォルト値がおかしい。
+# v0.52.1 で解消。 see: pull/5263
+Layout/SpaceBeforeBlockBraces:
+  EnforcedStyleForEmptyBraces: space
+
+#################### Lint ##################################
+
+# spec 内では
+# expect { subject }.to change { foo }
+# という書き方をよく行うので () を省略したい。
+# { foo } は明らかに change に紐付く。
+Lint/AmbiguousBlockAssociation:
+  Exclude:
+    - "spec/**/*"
+
+# Style/EmptyCaseCondition と同じく網羅の表現力が empty when を認めた方が高いし、
+# 頻出する対象を最初の when で撥ねるのはパフォーマンス向上で頻出する。
+# また、
+#   case foo
+#   when 42
+#     # nop
+#   when 1..100
+#     ...
+#   end
+# と、下の when がキャッチしてしまう場合等に対応していない。
+# See. http://tech.sideci.com/entry/2016/11/01/105900
+Lint/EmptyWhen:
+  Enabled: false
+
+# RuntimeError は「特定の Error を定義できない場合」なので、
+# 定義できるエラーは RuntimeError ではなく StandardError を継承する。
+Lint/InheritException:
+  EnforcedStyle: standard_error
+
+# * 同名のメソッドがある場合にローカル変数に `_` を付ける
+# * 一時変数として `_` を付ける
+# というテクニックは頻出する
+Lint/UnderscorePrefixedVariableName:
+  Enabled: false
+
+# 子クラスで実装させるつもりで中身が
+#   raise NotImplementedError
+# のみのメソッドが引っかかるので。
+# (raise せずに中身が空だと IgnoreEmptyMethods でセーフ)
+Lint/UnusedMethodArgument:
+  Enabled: false
+
+
+#################### Metrics ###############################
+
+# 30 まではギリギリ許せる範囲だったけど
+# リリースごとに 3 ずつぐらい下げていきます。20 まで下げたい。
+Metrics/AbcSize:
+  Max: 24
+
+# Gemfile, Guardfile は DSL 的で基本的に複雑にはならないので除外
+# rake, rspec, environments, routes は巨大な block 不可避なので除外
+# TODO: ExcludedMethods の精査
+Metrics/BlockLength:
+  Exclude:
+    - "Rakefile"
+    - "**/*.rake"
+    - "spec/**/*.rb"
+    - "Gemfile"
+    - "Guardfile"
+    - "config/environments/*.rb"
+    - "config/routes.rb"
+    - "config/routes/**/*.rb"
+    - "*.gemspec"
+
+# 6 は強すぎるので緩める
+Metrics/CyclomaticComplexity:
+  Max: 10
+
+# * 警告 120文字
+# * 禁止 160文字
+# のイメージ
+Metrics/LineLength:
+  Max: 160
+  Exclude:
+    - "db/migrate/*.rb"
+
+# 20 行超えるのは migration ファイル以外滅多に無い
+Metrics/MethodLength:
+  Max: 20
+  Exclude:
+    - "db/migrate/*.rb"
+
+# 分岐の数。ガード句を多用しているとデフォルト 7 だと厳しい
+Metrics/PerceivedComplexity:
+  Max: 8
+
+
+#################### Naming ################################
+
+# has_ から始まるメソッドは許可する
+Naming/PredicateName:
+  NamePrefixBlacklist:
+    - "is_"
+    - "have_"
+  NamePrefix:
+    - "is_"
+    - "have_"
+
+
+#################### Performance ###########################
+
+# downcase or upcase しての比較はイディオムの域なので、多少の
+# パフォーマンスの違いがあろうが casecmp に変える意義を感じない
+Performance/Casecmp:
+  Enabled: false
+
+# 2.4 以降では each_key でも特にパフォーマンスに差が無いので
+# 読みやすさはほとんど変わらないし、書きやすさを取る。
+Performance/HashEachMethods:
+  Enabled: false
+
+
+#################### Security ##############################
+
+# 毎回 YAML.safe_load(yaml_str, [Date, Time]) するのは面倒で。。
+Security/YAMLLoad:
+  Enabled: false
+
 
 #################### Style #################################
 
@@ -118,6 +249,12 @@ Style/EmptyMethod:
 Style/FormatString:
   EnforcedStyle: percent
 
+# strftime("%Y%m%d") の %d で引っかかる false positive がある。
+# また、url escape でも引っかかるらしい。
+# see: pull/5230, issues/5242
+Style/FormatStringToken:
+  Enabled: false
+
 # まだ対応するには早い
 Style/FrozenStringLiteralComment:
   Enabled: false
@@ -152,6 +289,15 @@ Style/Lambda:
 Style/MethodCalledOnDoEndBlock:
   Enabled: true
 
+# この 2 つは単発で動かすのが分かっているので Object を汚染しても問題ない。
+# spec/dummy は Rails Engine を開発するときに絶対に引っかかるので入れておく。
+Style/MixinUsage:
+  Exclude:
+    - "bin/setup"
+    - "bin/update"
+    - "spec/dummy/bin/setup"
+    - "spec/dummy/bin/update"
+
 # 1_000_000 と区切り文字が 2 個以上必要になる場合のみ _ 区切りを必須にする
 # 10_000_00 は許可しない。(これは例えば 10000 ドルをセント単位にする時に便利だが
 # 頻出しないので foolproof に振る
@@ -186,6 +332,10 @@ Style/RedundantReturn:
 Style/RedundantSelf:
   Enabled: false
 
+# 無指定だと StandardError を rescue するのは常識の範疇なので。
+Style/RescueStandardError:
+  EnforcedStyle: implicit
+
 # user&.admin? が、[nil, true, false] の 3 値を返すことに一瞬で気づけず
 # boolean を返すっぽく見えてしまうので無効に。
 # user && user.admin? なら短絡評価で nil が返ってくるのが一目で分かるので。
@@ -245,16 +395,16 @@ Style/TrailingCommaInArguments:
 Style/TrailingCommaInLiteral:
   EnforcedStyleForMultiline: comma
 
-# 0 <= foo && foo < 5 のように数直線上に並べるのは
-# コードを読みやすくするテクニックなので equality_operators_only に。
-Style/YodaCondition:
-  EnforcedStyle: equality_operators_only
-
 # %w() と %i() が見分けづらいので Style/SymbolArray と合わせて無効に。
 # 書き手に委ねるという意味で、Enabled: false にしています。使っても良い。
 Style/WordArray:
   Enabled: false
 
+# 0 <= foo && foo < 5 のように数直線上に並べるのは
+# コードを読みやすくするテクニックなので equality_operators_only に。
+Style/YodaCondition:
+  EnforcedStyle: equality_operators_only
+
 # 条件式で arr.size > 0 が使われた時に
 #   if !arr.empty?
 #   else
@@ -263,125 +413,3 @@ Style/WordArray:
 # 中身を入れ替えて否定外しても良いんだけど、どちらが例外的な処理なのかが分かりづらくなる。
 Style/ZeroLengthPredicate:
   Enabled: false
-
-#################### Naming ################################
-
-# has_ から始まるメソッドは許可する
-Naming/PredicateName:
-  NamePrefixBlacklist:
-    - "is_"
-    - "have_"
-  NamePrefix:
-    - "is_"
-    - "have_"
-
-#################### Lint ##################################
-
-# spec 内では
-# expect { subject }.to change { foo }
-# という書き方をよく行うので () を省略したい。
-# { foo } は明らかに change に紐付く。
-Lint/AmbiguousBlockAssociation:
-  Exclude:
-    - "spec/**/*"
-
-# Style/EmptyCaseCondition と同じく網羅の表現力が empty when を認めた方が高いし、
-# 頻出する対象を最初の when で撥ねるのはパフォーマンス向上で頻出する。
-# また、
-#   case foo
-#   when 42
-#     # nop
-#   when 1..100
-#     ...
-#   end
-# と、下の when がキャッチしてしまう場合等に対応していない。
-# See. http://tech.sideci.com/entry/2016/11/01/105900
-Lint/EmptyWhen:
-  Enabled: false
-
-# RuntimeError は「特定の Error を定義できない場合」なので、
-# 定義できるエラーは RuntimeError ではなく StandardError を継承する。
-Lint/InheritException:
-  EnforcedStyle: standard_error
-
-# 無指定だと StandardError を rescue するのは常識の範疇なので。
-# https://github.com/bbatsov/rubocop/issues/2943#issuecomment-330498533
-# rescue 後に何をするかは Lint/HandleExceptions でチェックするので
-# ココで想起させる必要は無さそう。
-Lint/RescueWithoutErrorClass:
-  Enabled: false
-
-# * 同名のメソッドがある場合にローカル変数に `_` を付ける
-# * 一時変数として `_` を付ける
-# というテクニックは頻出する
-Lint/UnderscorePrefixedVariableName:
-  Enabled: false
-
-# 子クラスで実装させるつもりで中身が
-#   raise NotImplementedError
-# のみのメソッドが引っかかるので。
-# (raise せずに中身が空だと IgnoreEmptyMethods でセーフ)
-Lint/UnusedMethodArgument:
-  Enabled: false
-
-#################### Metrics ###############################
-
-# 30 まではギリギリ許せる範囲だったけど
-# リリースごとに 3 ずつぐらい下げていきます。20 まで下げたい。
-Metrics/AbcSize:
-  Max: 24
-
-# Gemfile, Guardfile は DSL 的で基本的に複雑にはならないので除外
-# rake, rspec, environments, routes は巨大な block 不可避なので除外
-# TODO: ExcludedMethods の精査
-Metrics/BlockLength:
-  Exclude:
-    - "Rakefile"
-    - "**/*.rake"
-    - "spec/**/*.rb"
-    - "Gemfile"
-    - "Guardfile"
-    - "config/environments/*.rb"
-    - "config/routes.rb"
-    - "config/routes/**/*.rb"
-    - "*.gemspec"
-
-# 6 は強すぎるので緩める
-Metrics/CyclomaticComplexity:
-  Max: 10
-
-# * 警告 120文字
-# * 禁止 160文字
-# のイメージ
-Metrics/LineLength:
-  Max: 160
-  Exclude:
-    - "db/migrate/*.rb"
-
-# 20 行超えるのは migration ファイル以外滅多に無い
-Metrics/MethodLength:
-  Max: 20
-  Exclude:
-    - "db/migrate/*.rb"
-
-# 分岐の数。ガード句を多用しているとデフォルト 7 だと厳しい
-Metrics/PerceivedComplexity:
-  Max: 8
-
-#################### Performance ###########################
-
-# downcase or upcase しての比較はイディオムの域なので、多少の
-# パフォーマンスの違いがあろうが casecmp に変える意義を感じない
-Performance/Casecmp:
-  Enabled: false
-
-# 2.4 以降では each_key でも特にパフォーマンスに差が無いので
-# 読みやすさはほとんど変わらないし、書きやすさを取る。
-Performance/HashEachMethods:
-  Enabled: false
-
-#################### Security ##############################
-
-# 毎回 YAML.safe_load(yaml_str, [Date, Time]) するのは面倒で。。
-Security/YAMLLoad:
-  Enabled: false

data/lib/onkcop/version.rb

@@ -1,3 +1,3 @@
 module Onkcop
-  VERSION = "0.51.0.1"
+  VERSION = "0.52.0.0"
 end

data/onkcop.gemspec

@@ -19,8 +19,9 @@ 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.51.0"
-  spec.add_dependency "rubocop-rspec", ">= 1.20.0"
+  spec.add_dependency "rubocop", "~> 0.52.0"
+  spec.add_dependency "rubocop-rspec", ">= 1.21.0"
   spec.add_development_dependency "bundler"
   spec.add_development_dependency "rake"
+  spec.add_development_dependency "rspec"
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: onkcop
 version: !ruby/object:Gem::Version
-  version: 0.51.0.1
+  version: 0.52.0.0
 platform: ruby
 authors:
 - Takafumi ONAKA
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2017-11-11 00:00:00.000000000 Z
+date: 2017-12-19 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rubocop
@@ -16,28 +16,28 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.51.0
+        version: 0.52.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.51.0
+        version: 0.52.0
 - !ruby/object:Gem::Dependency
   name: rubocop-rspec
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 1.20.0
+        version: 1.21.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 1.20.0
+        version: 1.21.0
 - !ruby/object:Gem::Dependency
   name: bundler
   requirement: !ruby/object:Gem::Requirement
@@ -66,6 +66,20 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
+- !ruby/object:Gem::Dependency
+  name: rspec
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 description: OnkCop is a RuboCop configration gem.
 email:
 - takafumi.onaka@gmail.com
@@ -75,7 +89,9 @@ extensions: []
 extra_rdoc_files: []
 files:
 - ".gitignore"
+- ".rspec"
 - ".rubocop.yml"
+- ".travis.yml"
 - CHANGELOG.md
 - Gemfile
 - LICENSE.txt
@@ -112,7 +128,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.6.14
+rubygems_version: 2.7.3
 signing_key: 
 specification_version: 4
 summary: OnkCop is a RuboCop configration gem.