unifacop 0.49.1.0 → 0.52.0.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 3e6794874824e26df487a552c89f232229793fcd
-  data.tar.gz: d3dda778e1ec37f3c882315137462e4e5d3fba0f
+  metadata.gz: e1e7645fb07188ba90fbb7165f563caba945bdad
+  data.tar.gz: 7a2fedeafc6480154828bbf4fff11ed38fcbfd3c
 SHA512:
-  metadata.gz: 340d3dae78c3a0ac5b384063e60861bf44bc91ff22cc7fec412c3bf74fa178c8c0a8a8c3dcbb6fd9b8f6aec846d32e978a910458bf1b15339b509301d7bf3837
-  data.tar.gz: 01a8a9605452929520e4d8242d624e0b7cff9bddd829e2c55114898be7fb897cf8eaa10697a7e3a5f86f59bef9dbe36ce61c99e139c8e4edd3e8b56817add81d
+  metadata.gz: 38e7f4c59cabe8b8fb5057fcc05b6f0dbb9c42fe9b311a1a30ae0ee70b25738c57a23cb2f6119667e7cdc53dad67830f27fe4a2e8504d1e9941b4611150dbde0
+  data.tar.gz: e21c1d840095b3b15d68870ee69c9495a0695bf15ab01c1030a714b77b37face68ecaa831512b9e209604b05c8a3a5edced336705061a811b08bb5e946c90a22

data/Gemfile

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 source 'https://rubygems.org'
 
 gemspec

data/config/rails.yml

@@ -1,6 +1,29 @@
 Rails:
   Enabled: true
 
+# onkcop参照
+# > ActiveRecord で association を安易に delegate すると
+# > N+1 を起こしまくるので、なるべく delegate 的なメソッドを使わずに
+# > コストが掛かっていることを自覚できるようにしておきたい。
+# > メソッドでも危ういが、DSL だと更に意識から抜けるので無効に。
+Rails/Delegate:
+  Enabled: false
+
+# onkcop参照
+# > -File.join(Rails.root, "app", "models")
+# > +Rails.root.join("app", "models")
+# > はともかく
+# > -Rails.root.join("app/models")
+# > +Rails.root.join("app", "models")
+# > は Pathname#plus が行っているので意味無いのでは？
+Rails/FilePath:
+  Enabled: false
+
+# 少しでもオプションつけるとすぐ引っかかってしまう。
+# inverse_ofが必要かどうかは、bulletでN+1を検知してから対応としたい。
+Rails/InverseOf:
+  Enabled: false
+
 # onkcop参照
 # > slug とか created_by とか、NOT NULL だが create_table 時に
 # > default 値を定義できないカラムは存在する。
@@ -11,3 +34,12 @@ Rails/NotNullColumn:
 # > valid? チェックし忘れを防ぎたい
 Rails/SaveBang:
   Enabled: true
+
+# 多くの場合、staging環境は用意するのでデフォルトで追加。
+# 他にもある場合、各プロジェクトで上書きしてもらう。
+Rails/UnknownEnv:
+  Environments:
+    - development # rubocop default.yml
+    - test        # rubocop default.yml
+    - production  # rubocop default.yml
+    - staging

data/config/rspec.yml

@@ -1,5 +1,10 @@
 require: 'rubocop-rspec'
 
+# 日本語でcontextを書く場合、「〜の場合」と書くのでPrefixesでは対応できない。
+# 英語と日本語が混じるケースでも対応が難しいので、各自の努力に任せる。
+RSpec/ContextWording:
+  Enabled: false
+
 # onkcop参照
 # > subject はコピペ可搬性よりもそのまま USAGE であって欲しい
 # described_classより直接クラス名を参照する方が、使い方が見えやすくてよい。

data/config/rubocop.yml

@@ -24,6 +24,10 @@ Layout/DotPosition:
   Enabled: true
   EnforcedStyle: leading
 
+# 誤検知、誤修正が多い。安定するまで無効化。
+Layout/EmptyLinesAroundArguments:
+  Enabled: false
+
 # onkcop参照
 # > special_inside_parentheses (default) と比べて
 # > * 横に長くなりづらい
@@ -149,10 +153,33 @@ Metrics/ParameterLists:
 Metrics/PerceivedComplexity:
   Max: 8
 
+## Naming
+#  http://rubocop.readthedocs.io/en/latest/cops_naming/
+
+# onkcop参照
+# > has_ から始まるメソッドは許可する
+Naming/PredicateName:
+  NamePrefixBlacklist:
+    - 'is_'
+    - 'have_'
+  NamePrefix:
+    - 'is_'
+    - 'have_'
+
 ## Performance
 #  https://rubocop.readthedocs.io/en/latest/cops_performance/
 
-# nothing to customize
+# onkcop参照
+# > downcase or upcase しての比較はイディオムの域なので、多少の
+# > パフォーマンスの違いがあろうが casecmp に変える意義を感じない
+Performance/Casecmp:
+  Enabled: false
+
+# onkcop参照
+# > 2.4 以降では each_key でも特にパフォーマンスに差が無いので
+# > 読みやすさはほとんど変わらないし、書きやすさを取る。
+Performance/HashEachMethods:
+  Enabled: false
 
 ## Security
 #  https://rubocop.readthedocs.io/en/latest/cops_security/
@@ -238,6 +265,13 @@ Style/EmptyMethod:
 Style/Encoding:
   Enabled: true
 
+# onkcop参照
+# > strftime("%Y%m%d") の %d で引っかかる false positive がある。
+# > また、url escape でも引っかかるらしい。
+# > see: pull/5230, issues/5242
+Style/FormatStringToken:
+  Enabled: false
+
 # onkcop参照
 # > if 文の中に 3 行程度のブロックを書くぐらいは許容した方が現実的
 # > NOTE: https://github.com/bbatsov/rubocop/commit/29945958034db13af9e8ff385ec58cb9eb464596
@@ -283,6 +317,16 @@ Style/MixinGrouping:
   Exclude:
     - 'spec/**/*'
 
+# onkcop参照
+# > この 2 つは単発で動かすのが分かっているので Object を汚染しても問題ない。
+# > spec/dummy は Rails Engine を開発するときに絶対に引っかかるので入れておく。
+Style/MixinUsage:
+  Exclude:
+    - 'bin/setup'
+    - 'bin/update'
+    - 'spec/dummy/bin/setup'
+    - 'spec/dummy/bin/update'
+
 # onkcop参照
 # > 1_000_000 と区切り文字が 2 個以上必要になる場合のみ _ 区切りを必須にする
 # > 10_000_00 は許可しない。(これは例えば 10000 ドルをセント単位にする時に便利だが
@@ -298,6 +342,12 @@ Style/NumericLiterals:
 Style/NumericPredicate:
   Enabled: false
 
+# onkcop参照
+# > false な場合という条件式の方を意識させたい場合がある。
+# > Style/IfUnlessModifier と同じ雰囲気。
+Style/OrAssignment:
+  Enabled: false
+
 # デフォルトでは %w[] が推奨だが、今いるメンバーでは %w() の方が合っているため
 # また %i() も %w() と統一感を取るため合わせる
 # %r{} はRubyStyleGuideの推奨を使用する。
@@ -316,16 +366,6 @@ Style/PercentLiteralDelimiters:
 Style/PerlBackrefs:
   AutoCorrect: false
 
-# onkcop参照
-# > has_ から始まるメソッドは許可する
-Style/PredicateName:
-  NamePrefixBlacklist:
-    - 'is_'
-    - 'have_'
-  NamePrefix:
-    - 'is_'
-    - 'have_'
-
 # onkcop参照
 # > Hash#has_key? の方が key? よりも意味が通る
 Style/PreferredHashMethods:
@@ -342,7 +382,14 @@ Style/RedundantSelf:
   Enabled: false
 
 # onkcop参照
-# > 無理して使うモンじゃない
+# > user&.admin? が、[nil, true, false] の 3 値を返すことに一瞬で気づけず
+# > boolean を返すっぽく見えてしまうので無効に。
+# > user && user.admin? なら短絡評価で nil が返ってくるのが一目で分かるので。
+# > (boolean を返すメソッド以外なら積極的に使いたいんだけどねぇ
+#
+# > 他に auto-correct してはいけないパターンとして
+# >   if hoge && hoge.count > 1
+# > がある。
 Style/SafeNavigation:
   Enabled: false
 

data/exe/unifacop

@@ -1,4 +1,5 @@
 #!/usr/bin/env ruby
+# frozen_string_literal: true
 
 $LOAD_PATH.unshift File.dirname(__FILE__) + '/../lib'
 

data/lib/unifacop/cli.rb

@@ -18,7 +18,7 @@ module UniFaCop
       puts "Could not find command #{action_name}."
       print_help
       exit(1)
-    rescue => e
+    rescue StandardError => e
       puts e.message
       exit(1)
     end

data/lib/unifacop/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module UniFaCop
-  VERSION = '0.49.1.0'.freeze
+  VERSION = '0.52.0.0'.freeze
 end

data/unifacop.gemspec

@@ -21,9 +21,10 @@ Gem::Specification.new do |spec|
   spec.executables   = spec.files.grep(%r{^exe/}) { |f| File.basename(f) }
   spec.require_paths = ['lib']
 
-  spec.add_runtime_dependency 'rubocop', '~> 0.49.1'
-  # Latest rubocop-rspec supports since Ruby 2.1. But we want to use it on Ruby 2.0.
-  spec.add_runtime_dependency 'rubocop-rspec', '>= 1.4.0'
+  spec.required_ruby_version = '>= 2.1.0'
+
+  spec.add_runtime_dependency 'rubocop', '~> 0.52.0'
+  spec.add_runtime_dependency 'rubocop-rspec', '>= 1.21.0'
   spec.add_development_dependency 'bundler'
   spec.add_development_dependency 'rake'
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: unifacop
 version: !ruby/object:Gem::Version
-  version: 0.49.1.0
+  version: 0.52.0.0
 platform: ruby
 authors:
 - UniFa Co. Ltd.
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2017-08-25 00:00:00.000000000 Z
+date: 2018-01-09 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rubocop
@@ -16,28 +16,28 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.49.1
+        version: 0.52.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.49.1
+        version: 0.52.0
 - !ruby/object:Gem::Dependency
   name: rubocop-rspec
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 1.4.0
+        version: 1.21.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 1.4.0
+        version: 1.21.0
 - !ruby/object:Gem::Dependency
   name: bundler
   requirement: !ruby/object:Gem::Requirement
@@ -95,7 +95,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: '0'
+      version: 2.1.0
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
@@ -103,7 +103,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.6.11
+rubygems_version: 2.6.13
 signing_key: 
 specification_version: 4
 summary: A rubocop configuration gem for UniFa Coding Style Guide.