Add rubocop-performance (#31509).

git-svn-id: http://svn.redmine.org/redmine/trunk@18378 e93f8b46-1217-0410-a6f0-8f06a7374b81

diff --git a/.rubocop.yml b/.rubocop.yml
index 10c3b50ec13e5a9814ccbb43254c8368dbed3beb..91af1ddf1491f1ed64120634696798e38a51f87d 100644 (file)
--- a/.rubocop.yml
+++ b/.rubocop.yml
@@ -17,6 +17,7 @@ AllCops:
 # Enable extensions
 
 require:
+  - rubocop-performance
   - rubocop-rails
 
 # Rules for Redmine

diff --git a/.rubocop_todo.yml b/.rubocop_todo.yml
index 735b9bf5462f8c618cecb83282bf563a343f1e32..9005a86ef4e273d1e9c19f48d3f17518827ca4e4 100644 (file)
--- a/.rubocop_todo.yml
+++ b/.rubocop_todo.yml
@@ -818,6 +818,70 @@ Naming/VariableNumber:
     - 'test/unit/lib/redmine/scm/adapters/mercurial_adapter_test.rb'
     - 'test/unit/project_test.rb'
 
+# Cop supports --auto-correct.
+Performance/Casecmp:
+  Exclude:
+    - 'lib/redmine/codeset_util.rb'
+    - 'lib/redmine/scm/adapters/bazaar_adapter.rb'
+
+Performance/FixedSize:
+  Exclude:
+    - 'test/integration/api_test/issues_test.rb'
+    - 'test/integration/attachments_test.rb'
+
+# Cop supports --auto-correct.
+Performance/InefficientHashSearch:
+  Exclude:
+    - 'test/functional/issues_custom_fields_visibility_test.rb'
+    - 'test/functional/search_custom_fields_visibility_test.rb'
+    - 'test/functional/timelog_custom_fields_visibility_test.rb'
+    - 'test/unit/lib/redmine/mime_type_test.rb'
+    - 'test/unit/query_test.rb'
+
+# Cop supports --auto-correct.
+Performance/RedundantBlockCall:
+  Exclude:
+    - 'app/controllers/application_controller.rb'
+    - 'lib/redmine/views/builders/structure.rb'
+    - 'test/functional/repositories_git_controller_test.rb'
+    - 'test/unit/lib/redmine/views/builders/json_test.rb'
+    - 'test/unit/lib/redmine/views/builders/xml_test.rb'
+
+# Cop supports --auto-correct.
+Performance/RedundantMatch:
+  Exclude:
+    - 'app/controllers/auto_completes_controller.rb'
+    - 'app/models/issue_relation.rb'
+    - 'lib/redmine/wiki_formatting/textile/formatter.rb'
+    - 'lib/redmine/wiki_formatting/textile/redcloth3.rb'
+
+# Cop supports --auto-correct.
+# Configuration parameters: MaxKeyValuePairs.
+Performance/RedundantMerge:
+  Exclude:
+    - 'app/controllers/issues_controller.rb'
+    - 'app/helpers/application_helper.rb'
+    - 'app/helpers/avatars_helper.rb'
+    - 'app/helpers/custom_fields_helper.rb'
+    - 'app/helpers/workflows_helper.rb'
+    - 'app/models/auth_source_ldap.rb'
+    - 'app/models/principal.rb'
+    - 'lib/redmine/access_control.rb'
+    - 'test/functional/imports_controller_test.rb'
+    - 'test/unit/issue_import_test.rb'
+    - 'test/unit/time_entry_import_test.rb'
+
+# Cop supports --auto-correct.
+Performance/StringReplacement:
+  Exclude:
+    - 'app/helpers/application_helper.rb'
+    - 'app/helpers/imports_helper.rb'
+    - 'app/helpers/settings_helper.rb'
+    - 'lib/redmine/core_ext/string/conversions.rb'
+    - 'lib/redmine/wiki_formatting/textile/redcloth3.rb'
+    - 'test/helpers/application_helper_test.rb'
+    - 'test/unit/lib/redmine/scm/adapters/mercurial_adapter_test.rb'
+
 # Configuration parameters: Include.
 # Include: app/models/**/*.rb
 Rails/ActiveRecordOverride:

diff --git a/Gemfile b/Gemfile
index 12fc054fb52e0122defad092162add8c1abf1103..f6856d3c52dcafb30f5b264efb1741a4a89d7ee0 100644 (file)
--- a/Gemfile
+++ b/Gemfile
@@ -85,6 +85,7 @@ group :test do
   gem "selenium-webdriver"
   # RuboCop
   gem 'rubocop', '~> 0.74.0'
+  gem 'rubocop-performance', '~> 1.4.1'
   gem 'rubocop-rails', '~> 2.3.0'
 end