unifacop 0.49.1.0 → 1.11.0.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
-SHA1:
-  metadata.gz: 3e6794874824e26df487a552c89f232229793fcd
-  data.tar.gz: d3dda778e1ec37f3c882315137462e4e5d3fba0f
+SHA256:
+  metadata.gz: e69317a0ab38a8a0d86c214a4de03efdd1c0a6f19e9d4b31d66fa2011dc050ef
+  data.tar.gz: 1055b85255d8e935bf4c3ac315451df6c853a7533c0a3326b35708fa967d8b9b
 SHA512:
-  metadata.gz: 340d3dae78c3a0ac5b384063e60861bf44bc91ff22cc7fec412c3bf74fa178c8c0a8a8c3dcbb6fd9b8f6aec846d32e978a910458bf1b15339b509301d7bf3837
-  data.tar.gz: 01a8a9605452929520e4d8242d624e0b7cff9bddd829e2c55114898be7fb897cf8eaa10697a7e3a5f86f59bef9dbe36ce61c99e139c8e4edd3e8b56817add81d
+  metadata.gz: f397526874ae4507c3c9151b3e14e9ad65354baeda448b7b857a86421d07b32182225a7223645a7964433728b781a910650e324fe591c64fabf1bb9c5871c819
+  data.tar.gz: 41c06f7f1804e79b42f33f37af49b6ae10932e3012430e4cc8140fed8ad01f165870a8f54e428c78cdeefe28b6322b9f77e4fdbfb0425f41ab62ad28d9cc4750

data/Gemfile

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

data/config/performance.yml

@@ -0,0 +1,10 @@
+require: 'rubocop-performance'
+
+## Performance
+# https://github.com/rubocop-hq/rubocop-performance/blob/master/manual/cops_performance.md
+
+# onkcop参照
+# > downcase or upcase しての比較はイディオムの域なので、多少の
+# > パフォーマンスの違いがあろうが casecmp に変える意義を感じない
+Performance/Casecmp:
+  Enabled: false

data/config/rails.yml

@@ -1,6 +1,41 @@
+require: 'rubocop-rails'
+
 Rails:
   Enabled: true
 
+# ActiveRecord aliases can also be understood
+# For example update_attributes! for update!
+Rails/ActiveRecordAliases:
+  Enabled: false
+
+# 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
+
+# :not_found より 404 の方がわかりやすいケースもあるので、どちらも使用可とする。
+# プロジェクト参加メンバーの好みでnumeric、symbolicに変更してほしい。
+Rails/HttpStatus:
+  Enabled: false
+
 # onkcop参照
 # > slug とか created_by とか、NOT NULL だが create_table 時に
 # > default 値を定義できないカラムは存在する。
@@ -11,3 +46,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より直接クラス名を参照する方が、使い方が見えやすくてよい。
@@ -11,6 +16,11 @@ RSpec/DescribedClass:
 RSpec/ExampleLength:
   Max: 10
 
+# expect(run).to change(Foo, :bar) だとどうも何をやっているのかわかりづらい。
+# expect(run).to change { Foo.bar } の方が何を確認しているのかわかりやすい。
+RSpec/ExpectChange:
+  EnforcedStyle: block
+
 # onkcop参照
 # > one-liner の should は書きやすいし意味が通りやすいし副作用も無いので撥ねる必要がない。
 # > ただ expect 派に対して強制するほどでもないので統一はしない。

data/config/rubocop.yml

@@ -24,25 +24,53 @@ Layout/DotPosition:
   Enabled: true
   EnforcedStyle: leading
 
+# 誤検知、誤修正が多い。安定するまで無効化。
+Layout/EmptyLinesAroundArguments:
+  Enabled: false
+
 # onkcop参照
 # > special_inside_parentheses (default) と比べて
 # > * 横に長くなりづらい
 # > * メソッド名の長さが変わったときに diff が少ない
-Layout/IndentArray:
+Layout/FirstArrayElementIndentation:
   EnforcedStyle: consistent
 
 # onkcop参照
 # > ({ と hash を開始した場合に ( の位置にインデントさせる
 # > そもそも {} が必要ない可能性が高いが Style/BracesAroundHashParameters はチェックしないことにしたので
-Layout/IndentHash:
+Layout/FirstHashElementIndentation:
   EnforcedStyle: consistent
 
+# Allow both table and key hash alignment
+Layout/HashAlignment:
+  EnforcedColonStyle:
+    - key
+    - table
+  EnforcedHashRocketStyle:
+    - key
+    - table
+
+# 多くのRubyコーディング規約では「80」か「100」が主流な値であった。
+# 「80」だと厳しすぎるので、「100」に設定する。
+Layout/LineLength:
+  Max: 100
+  Exclude:
+    - 'db/migrate/*.rb'
+
 # onkcop参照
 # > メソッドチェーン感がより感じられるインデントにする
 # 個人的にもこの形式の方が、上の行とのつながりを感じられるのでいいと思う。
 Layout/MultilineMethodCallIndentation:
   EnforcedStyle: indented_relative_to_receiver
 
+# This is causing offences when a rescue block is used
+# with variable assignment
+Layout/RescueEnsureAlignment:
+  Enabled: false
+
+Layout/SpaceAroundMethodCallOperator:
+  Enabled: true
+
 ## Lint cops
 #  https://rubocop.readthedocs.io/en/latest/cops_lint/
 
@@ -80,6 +108,12 @@ Lint/EmptyWhen:
 Lint/InheritException:
   EnforcedStyle: standard_error
 
+Lint/RaiseException:
+  Enabled: true
+
+Lint/StructNewOverride:
+  Enabled: true
+
 # キーワード引数の先頭に「_」をつけてしまうと、呼び出し側で「_」をつけて呼ばないとキーが存在せずエラーになってしまうため。
 Lint/UnusedMethodArgument:
   AllowUnusedKeywordArguments: true
@@ -122,13 +156,6 @@ Metrics/ClassLength:
 Metrics/CyclomaticComplexity:
   Max: 10
 
-# 多くのRubyコーディング規約では「80」か「100」が主流な値であった。
-# 「80」だと厳しすぎるので、「100」に設定する。
-Metrics/LineLength:
-  Max: 100
-  Exclude:
-    - 'db/migrate/*.rb'
-
 # onkcop参照
 # > 20 行超えるのは migration ファイル以外滅多に無い
 Metrics/MethodLength:
@@ -149,10 +176,27 @@ Metrics/ParameterLists:
 Metrics/PerceivedComplexity:
   Max: 8
 
-## Performance
-#  https://rubocop.readthedocs.io/en/latest/cops_performance/
+## Naming
+#  http://rubocop.readthedocs.io/en/latest/cops_naming/
 
-# nothing to customize
+# Allow short method param names like e or _
+Naming/MethodParameterName:
+  Enabled: false
+
+# onkcop参照
+# > has_ から始まるメソッドは許可する
+Naming/PredicateName:
+  ForbiddenPrefixes:
+    - 'is_'
+    - 'have_'
+  NamePrefix:
+    - 'is_'
+    - 'have_'
+
+# This limits variable name exceptions
+# Can be set on a per project basis if needed
+Naming/RescuedExceptionsVariableName:
+  Enabled: false
 
 ## Security
 #  https://rubocop.readthedocs.io/en/latest/cops_security/
@@ -185,11 +229,6 @@ Style/BlockDelimiters:
   Exclude:
     - 'spec/**/*'
 
-# onkcop参照
-# > option 等、明示的にハッシュにした方が分かりやすい場合もある
-Style/BracesAroundHashParameters:
-  Enabled: false
-
 # onkcop参照
 # > scope が違うとか親 module の存在確認が必要とかデメリットはあるが、
 # > namespace 付きのクラスはかなり頻繁に作るので簡単に書きたい。
@@ -228,6 +267,10 @@ Style/Documentation:
 Style/EmptyElse:
   EnforcedStyle: empty
 
+# -> { ... } より ->() { ... } の方がLambdaとblockを見分けやすい。
+Style/EmptyLambdaParameter:
+  Enabled: false
+
 # onkcop参照
 # > 空メソッドの場合だけ1行で書かなければいけない理由が無い
 # > 「セミコロンは使わない」に寄せた方がルールがシンプル
@@ -238,6 +281,16 @@ Style/EmptyMethod:
 Style/Encoding:
   Enabled: true
 
+Style/ExponentialNotation:
+  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
@@ -246,6 +299,17 @@ Style/Encoding:
 Style/GuardClause:
   MinBodyLength: 5
 
+# Both .each_key and .keys.each are readable
+# and have different return values
+Style/HashEachMethods:
+  Enabled: false
+
+Style/HashTransformKeys:
+  Enabled: true
+
+Style/HashTransformValues:
+  Enabled: true
+
 # onkcop参照
 # > rake タスクの順序の hash は rocket を許可する。
 Style/HashSyntax:
@@ -283,6 +347,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 +372,12 @@ Style/NumericLiterals:
 Style/NumericPredicate:
   Enabled: false
 
+# onkcop参照
+# > false な場合という条件式の方を意識させたい場合がある。
+# > Style/IfUnlessModifier と同じ雰囲気。
+Style/OrAssignment:
+  Enabled: false
+
 # デフォルトでは %w[] が推奨だが、今いるメンバーでは %w() の方が合っているため
 # また %i() も %w() と統一感を取るため合わせる
 # %r{} はRubyStyleGuideの推奨を使用する。
@@ -316,16 +396,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 +412,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
 
@@ -378,9 +455,15 @@ Style/TernaryParentheses:
   EnforcedStyle: require_parentheses_when_complex
 
 # onkcop参照
-# > 複数行の場合はケツカンマを入れる
-# あとで行を追加するときに、変更行を少なくするため。
-Style/TrailingCommaInLiteral:
+# > 複数行の場合はケツカンマを入れる(Arrayリテラル)
+# > JSON がケツカンマを許していないという反対意見もあるが、
+# > 古い JScript の仕様に縛られる必要は無い。
+# > IE9 以降はリテラルでケツカンマ OK なので正しい差分行の検出に寄せる。
+Style/TrailingCommaInArrayLiteral:
+  EnforcedStyleForMultiline: comma
+
+# > 複数行の場合はケツカンマを入れる(Hashリテラル)
+Style/TrailingCommaInHashLiteral:
   EnforcedStyleForMultiline: comma
 
 # onkcop参照

data/exe/unifacop

@@ -1,6 +1,7 @@
 #!/usr/bin/env ruby
+# frozen_string_literal: true
 
-$LOAD_PATH.unshift File.dirname(__FILE__) + '/../lib'
+$LOAD_PATH.unshift "#{File.dirname(__FILE__)}/../lib"
 
 # Exit cleanly from an early interrupt
 Signal.trap('INT') { exit 1 }

data/lib/unifacop/cli.rb

@@ -18,15 +18,16 @@ module UniFaCop
       puts "Could not find command #{action_name}."
       print_help
       exit(1)
-    rescue => e
+    rescue StandardError => e
       puts e.message
       exit(1)
     end
 
     def self.retrieve_command_name(args)
       return nil if args.empty?
+
       meth = args.first.to_s
-      args.shift if meth !~ /^\-/
+      args.shift unless /^-/.match?(meth)
     end
 
     def self.print_help
@@ -34,10 +35,11 @@ module UniFaCop
       puts '  init - Setup .rubocop.yml'
     end
 
-    CONFIG_FILE_NAME = '.rubocop.yml'.freeze
+    CONFIG_FILE_NAME = '.rubocop.yml'
 
     def init(args)
       raise 'usage: unifacop init' unless args.empty?
+
       template_path = File.expand_path('../../templates', __dir__)
       puts "#{File.exist?(CONFIG_FILE_NAME) ? 'overwrite' : 'create'} #{CONFIG_FILE_NAME}"
       FileUtils.copy_file(File.join(template_path, CONFIG_FILE_NAME), CONFIG_FILE_NAME)

data/lib/unifacop/version.rb

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

data/templates/.rubocop.yml

@@ -1,9 +1,10 @@
 inherit_gem:
   unifacop:
     - 'config/rubocop.yml'
+    - 'config/performance.yml'
     # Uncomment if your project uses rails. Do not forget to specify TargetRailsVersion.
     #- 'config/rails.yml'
-    # Uncomment if use rspec cops
+    # Uncomment to use rspec cops
     #- 'config/rspec.yml'
 
 inherit_from:
@@ -12,7 +13,9 @@ inherit_from:
 AllCops:
   # Please specify your Ruby version
   TargetRubyVersion: 2.4
-  # uncomment if use rails cops
+  # Uncomment if using rails cops
   #TargetRailsVersion: 5.0
+  NewCops: enable
+  SuggestExtensions: false
 
 # Add your project specific configurations here.

data/unifacop.gemspec

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-lib = File.expand_path('../lib', __FILE__)
+lib = File.expand_path('lib', __dir__)
 $LOAD_PATH.unshift(lib) unless $LOAD_PATH.include?(lib)
 require 'unifacop/version'
 
@@ -14,16 +14,21 @@ Gem::Specification.new do |spec|
   spec.description   = 'A rubocop configuration gem for UniFa Coding Style Guide.'
   spec.homepage      = 'https://bitbucket.org/lookme/unifacop'
 
+  exclude_files      = ['Gemfile', 'unifacop.gemspec'].freeze
   spec.files         = `git ls-files -z`.split("\x0").select do |f|
-    f.match(%r{^(config|exe|lib|templates)/}) || ['Gemfile', 'unifacop.gemspec'].include?(f)
+    f.match(%r{^(config|exe|lib|templates)/}) || exclude_files.include?(f)
   end
   spec.bindir        = 'exe'
   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.4.0'
+
+  spec.add_runtime_dependency 'rubocop', '>= 1.11.0'
+  spec.add_runtime_dependency 'rubocop-performance', '>= 1.10.1'
+  spec.add_runtime_dependency 'rubocop-rails', '>= 2.9.1'
+  spec.add_runtime_dependency 'rubocop-rspec', '>= 2.2.0'
+
   spec.add_development_dependency 'bundler'
   spec.add_development_dependency 'rake'
 end

metadata

@@ -1,43 +1,71 @@
 --- !ruby/object:Gem::Specification
 name: unifacop
 version: !ruby/object:Gem::Version
-  version: 0.49.1.0
+  version: 1.11.0.0
 platform: ruby
 authors:
 - UniFa Co. Ltd.
-autorequire: 
+autorequire:
 bindir: exe
 cert_chain: []
-date: 2017-08-25 00:00:00.000000000 Z
+date: 2021-03-15 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rubocop
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: 0.49.1
+        version: 1.11.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 1.11.0
+- !ruby/object:Gem::Dependency
+  name: rubocop-performance
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 1.10.1
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
       - !ruby/object:Gem::Version
-        version: 0.49.1
+        version: 1.10.1
+- !ruby/object:Gem::Dependency
+  name: rubocop-rails
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 2.9.1
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 2.9.1
 - !ruby/object:Gem::Dependency
   name: rubocop-rspec
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 1.4.0
+        version: 2.2.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 1.4.0
+        version: 2.2.0
 - !ruby/object:Gem::Dependency
   name: bundler
   requirement: !ruby/object:Gem::Requirement
@@ -75,6 +103,7 @@ extensions: []
 extra_rdoc_files: []
 files:
 - Gemfile
+- config/performance.yml
 - config/rails.yml
 - config/rspec.yml
 - config/rubocop.yml
@@ -87,7 +116,7 @@ files:
 homepage: https://bitbucket.org/lookme/unifacop
 licenses: []
 metadata: {}
-post_install_message: 
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -95,16 +124,15 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: '0'
+      version: 2.4.0
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubyforge_project: 
-rubygems_version: 2.6.11
-signing_key: 
+rubygems_version: 3.1.4
+signing_key:
 specification_version: 4
 summary: A rubocop configuration gem for UniFa Coding Style Guide.
 test_files: []