onkcop 0.52.1.1 → 0.53.0.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: f2829a1ab9edf1262b0dedc9e9cd2a39e2243ac7e2b5b00a483924963705822e
-  data.tar.gz: cdd0e610daafc9cafad70cc0e20004891e18b7081a0f8d999fc133eb39921738
+  metadata.gz: c970deb16bf2194384e45644f5e4502bf65745da4fddc87ab90b10d97daa2a04
+  data.tar.gz: 8f8b981b913402ca8b5f440494054f0ac33e4adfd48b7db7ece9658a5c2c0317
 SHA512:
-  metadata.gz: ab357b4c69996f6b795f0f7fe672f9ad308e13ac7fd249c1b69a166cfbb6fa0d34ca89368069123581521044b46c53735c2bc139406dc5b5d46e7bab39a0fd31
-  data.tar.gz: 23381aafd40181296850e952c23a5243af5ef2a909ff303400a609521ce4f6fb4cef804145c964596a1f100e9779804fdeb85ff0fb5d5df1c1fee844b66e3a87
+  metadata.gz: 114714c7b6dbb9cfd205025c641084f578cc20108e5680554a679f34bbd738446665e8c314369ad78d446b9c941cb7e1387e7d2187002b85095f78bed4a9f447
+  data.tar.gz: 239638bc2b71115480fc09a92c63bf87eacac43fd4b686bbf65a5f6eb1f1104d6caa335a612ebac71e0b6c8e4f151436d526ed5f3baee9c6f81baeee1194de19

data/.travis.yml

@@ -1,8 +1,5 @@
 language: ruby
-cache: bundler
+# NOTE: travis build fails when `install.bunler` with cache.
+# cache: bundler
 rvm:
   - ruby-head
-# NOTE: travis build fails on bundler v1.16.1 && rubygems v2.7.3.
-# Update travis's rubygems to v2.7.4.
-before_install:
-  - gem update --system

data/CHANGELOG.md

@@ -1,5 +1,17 @@
 # onkcop
 
+## v0.53.0.0 (2018-03-09)
+
+* Update `rubocop` v0.53.0 and `rubocop-rspec` v1.24.0
+* `Style/TrailingCommaInLiteral` cop is separated to `Style/TrailingCommaInArrayLiteral` and `Style/TrailingCommaInHashLiteral`
+* `Performance/HashEachMethods` cop is removed
+* Disable new `Naming/UncommunicativeMethodParamName` cop
+* Enable `Style/FormatStringToken` cop
+* Enable new `Style/EmptyLineAfterGuardClause` cop
+* Enable `Lint/Void` cop's mutating methods check
+* Remove `TargetRailsVersion` from template
+* Change `Layout/SpaceInsideBlockBraces` cop option
+
 ## v0.52.1.1 (2018-01-11)
 
 * Update `rubocop-rspec` to v1.22.0

data/README.md

@@ -27,8 +27,6 @@ inherit_gem:
 
 AllCops:
   TargetRubyVersion: 2.5
-  # uncomment if use rails cops
-  # TargetRailsVersion: 5.1
 ```
 
 ```sh

data/config/rubocop.yml

@@ -45,6 +45,11 @@ Layout/IndentationConsistency:
 Layout/MultilineMethodCallIndentation:
   EnforcedStyle: indented_relative_to_receiver
 
+# {} は 1 行で書くときに主に使われるので、スペースよりも
+# 横に長くならない方が嬉しさが多い。
+# そもそも {| のスタイルの方が一般的だったと認識している。
+Layout/SpaceInsideBlockBraces:
+  SpaceBeforeBlockParameters: false
 
 #################### Lint ##################################
 
@@ -88,6 +93,12 @@ Lint/UnderscorePrefixedVariableName:
 Lint/UnusedMethodArgument:
   Enabled: false
 
+# select 以外では引っかからないと思うので
+# mutating_methods のチェックを有効に。
+# TODO: select は引数が無い (ブロックのみ) の場合にだけチェックする
+# ようにすると誤検知がほぼ無くなる？
+Lint/Void:
+  CheckForMethodsWithNoSideEffects: true
 
 #################### Metrics ###############################
 
@@ -145,6 +156,12 @@ Naming/PredicateName:
     - "is_"
     - "have_"
 
+# 3 文字未満だと指摘されるが、未使用を示す _ や e(rror), b(lock),
+# n(umber) といった 1 文字変数は頻出するし、前置詞(by, to, ...)や
+# よく知られた省略語 (op: operator とか pk: primary key とか) も妥当。
+# 変数 s にどんな文字列かを形容したい場合と、不要な場合とがある＝無効
+Naming/UncommunicativeMethodParamName:
+  Enabled: false
 
 #################### Performance ###########################
 
@@ -153,11 +170,6 @@ Naming/PredicateName:
 Performance/Casecmp:
   Enabled: false
 
-# 2.4 以降では each_key でも特にパフォーマンスに差が無いので
-# 読みやすさはほとんど変わらないし、書きやすさを取る。
-Performance/HashEachMethods:
-  Enabled: false
-
 
 #################### Security ##############################
 
@@ -237,6 +249,10 @@ Style/EmptyCaseCondition:
 Style/EmptyElse:
   EnforcedStyle: empty
 
+# ガード句と本質を分けるのは良いコードスタイルなので有効化
+Style/EmptyLineAfterGuardClause:
+  Enabled: true
+
 # 空メソッドの場合だけ1行で書かなければいけない理由が無い
 # 「セミコロンは使わない」に寄せた方がルールがシンプル
 Style/EmptyMethod:
@@ -246,13 +262,6 @@ Style/EmptyMethod:
 Style/FormatString:
   EnforcedStyle: percent
 
-# strftime("%Y%m%d") の %d で引っかかる false positive がある。
-# また、url escape でも引っかかるらしい。
-# see: pull/5230, issues/5242
-# 上記 PR はマージされたが、まだダメっぽいので引き続き disable にする
-Style/FormatStringToken:
-  Enabled: false
-
 # まだ対応するには早い
 Style/FrozenStringLiteralComment:
   Enabled: false
@@ -385,12 +394,16 @@ Style/TernaryParentheses:
 Style/TrailingCommaInArguments:
   EnforcedStyleForMultiline: comma
 
-# 複数行の場合はケツカンマを入れる(リテラル)
+# 複数行の場合はケツカンマを入れる(Arrayリテラル)
 # JSON がケツカンマを許していないという反対意見もあるが、
 # 古い JScript の仕様に縛られる必要は無い。
 # IE9 以降はリテラルでケツカンマ OK なので正しい差分行の検出に寄せる。
 # 2 insertions(+), 1 deletion(-) ではなく、1 insertions
-Style/TrailingCommaInLiteral:
+Style/TrailingCommaInArrayLiteral:
+  EnforcedStyleForMultiline: comma
+
+# 複数行の場合はケツカンマを入れる(Hashリテラル)
+Style/TrailingCommaInHashLiteral:
   EnforcedStyleForMultiline: comma
 
 # %w() と %i() が見分けづらいので Style/SymbolArray と合わせて無効に。

data/lib/onkcop/version.rb

@@ -1,3 +1,3 @@
 module Onkcop
-  VERSION = "0.52.1.1"
+  VERSION = "0.53.0.0"
 end

data/onkcop.gemspec

@@ -1,5 +1,4 @@
-# coding: utf-8
-lib = File.expand_path("../lib", __FILE__)
+lib = File.expand_path("lib", __dir__)
 $LOAD_PATH.unshift(lib) unless $LOAD_PATH.include?(lib)
 require "onkcop/version"
 
@@ -14,13 +13,13 @@ Gem::Specification.new do |spec|
   spec.homepage      = "https://github.com/onk/onkcop"
   spec.license       = "MIT"
 
-  spec.files         = `git ls-files -z`.split("\x0").reject { |f| f.match(%r{^(test|spec|features)/}) }
+  spec.files         = `git ls-files -z`.split("\x0").reject {|f| f.match(%r{^(test|spec|features)/}) }
   spec.bindir        = "exe"
-  spec.executables   = spec.files.grep(%r{^exe/}) { |f| File.basename(f) }
+  spec.executables   = spec.files.grep(%r{^exe/}) {|f| File.basename(f) }
   spec.require_paths = ["lib"]
 
-  spec.add_dependency "rubocop", "~> 0.52.1"
-  spec.add_dependency "rubocop-rspec", ">= 1.22.0"
+  spec.add_dependency "rubocop", "~> 0.53.0"
+  spec.add_dependency "rubocop-rspec", ">= 1.24.0"
   spec.add_development_dependency "bundler"
   spec.add_development_dependency "rake"
   spec.add_development_dependency "rspec"

data/templates/.rubocop.yml

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

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: onkcop
 version: !ruby/object:Gem::Version
-  version: 0.52.1.1
+  version: 0.53.0.0
 platform: ruby
 authors:
 - Takafumi ONAKA
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2018-01-11 00:00:00.000000000 Z
+date: 2018-03-08 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rubocop
@@ -16,28 +16,28 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.52.1
+        version: 0.53.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.52.1
+        version: 0.53.0
 - !ruby/object:Gem::Dependency
   name: rubocop-rspec
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 1.22.0
+        version: 1.24.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 1.22.0
+        version: 1.24.0
 - !ruby/object:Gem::Dependency
   name: bundler
   requirement: !ruby/object:Gem::Requirement
@@ -128,7 +128,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.7.3
+rubygems_version: 2.7.6
 signing_key: 
 specification_version: 4
 summary: OnkCop is a RuboCop configration gem.