Commit fc25343d authored by Nick Thomas's avatar Nick Thomas

Merge branch 'pl-rubocop-enable-performance-openstruct' into 'master'

Enable RuboCop rule `Performance/OpenStruct`

See merge request gitlab-org/gitlab!57257
parents 2b214311 c1b9b7ac
...@@ -640,3 +640,7 @@ Cop/UserAdmin: ...@@ -640,3 +640,7 @@ Cop/UserAdmin:
- 'lib/gitlab/auth/current_user_mode.rb' - 'lib/gitlab/auth/current_user_mode.rb'
- 'spec/**/*.rb' - 'spec/**/*.rb'
- 'ee/spec/**/*.rb' - 'ee/spec/**/*.rb'
Performance/OpenStruct:
Exclude:
- 'ee/spec/**/*.rb'
...@@ -2573,3 +2573,20 @@ Cop/UserAdmin: ...@@ -2573,3 +2573,20 @@ Cop/UserAdmin:
- 'lib/gitlab/performance_bar.rb' - 'lib/gitlab/performance_bar.rb'
- 'lib/gitlab/visibility_level.rb' - 'lib/gitlab/visibility_level.rb'
- 'qa/qa/runtime/api/client.rb' - 'qa/qa/runtime/api/client.rb'
# WIP https://gitlab.com/gitlab-org/gitlab/-/issues/325744
Performance/OpenStruct:
Exclude:
- 'Guardfile'
- 'app/finders/snippets_finder.rb'
- 'app/helpers/application_settings_helper.rb'
- 'app/models/cycle_analytics/project_level_stage_adapter.rb'
- 'ee/lib/gitlab/graphql/aggregations/epics/epic_node.rb'
- 'ee/lib/gitlab/graphql/aggregations/epics/epic_node.rb'
- 'lib/api/wikis.rb'
- 'lib/gitlab/ci/ansi2html.rb'
- 'lib/gitlab/ci/reports/test_suite_comparer.rb'
- 'lib/gitlab/git/diff_collection.rb'
- 'lib/gitlab/import_export/after_export_strategies/base_after_export_strategy.rb'
- 'lib/gitlab/testing/request_inspector_middleware.rb'
- 'lib/mattermost/session.rb'
...@@ -323,10 +323,6 @@ Performance/Detect: ...@@ -323,10 +323,6 @@ Performance/Detect:
Performance/MethodObjectAsBlock: Performance/MethodObjectAsBlock:
Enabled: false Enabled: false
# Offense count: 42
Performance/OpenStruct:
Enabled: false
# Offense count: 18 # Offense count: 18
# Cop supports --auto-correct. # Cop supports --auto-correct.
# Configuration parameters: AutoCorrect. # Configuration parameters: AutoCorrect.
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment