Commit 3ca2b825 authored by Grzegorz Bizon's avatar Grzegorz Bizon

Fix Rubocop configuration files

parent 127e299f
...@@ -343,10 +343,6 @@ Style/ParenthesesAroundCondition: ...@@ -343,10 +343,6 @@ Style/ParenthesesAroundCondition:
Style/RedundantParentheses: Style/RedundantParentheses:
Enabled: true Enabled: true
# Don't use return where it's not required.
Style/RedundantReturn:
Enabled: true
# Don't use semicolons to terminate expressions. # Don't use semicolons to terminate expressions.
Style/Semicolon: Style/Semicolon:
Enabled: true Enabled: true
......
# This configuration was generated by # This configuration was generated by
# `rubocop --auto-gen-config` # `rubocop --auto-gen-config --exclude-limit 0`
# on 2017-01-06 20:23:01 -0800 using RuboCop version 0.46.0. # on 2017-01-11 09:34:02 +0000 using RuboCop version 0.46.0.
# The point is for the user to remove these configuration records # The point is for the user to remove these configuration records
# one by one as the offenses are removed from the code base. # one by one as the offenses are removed from the code base.
# Note that changes in the inspected code, or installation of new # Note that changes in the inspected code, or installation of new
...@@ -10,98 +10,69 @@ ...@@ -10,98 +10,69 @@
# Configuration parameters: Include. # Configuration parameters: Include.
# Include: **/Gemfile, **/gems.rb # Include: **/Gemfile, **/gems.rb
Bundler/OrderedGems: Bundler/OrderedGems:
Exclude: Enabled: false
- 'Gemfile'
# Offense count: 181 # Offense count: 181
Lint/AmbiguousRegexpLiteral: Lint/AmbiguousRegexpLiteral:
Enabled: false Enabled: false
# Offense count: 59 # Offense count: 60
# Configuration parameters: AllowSafeAssignment. # Configuration parameters: AllowSafeAssignment.
Lint/AssignmentInCondition: Lint/AssignmentInCondition:
Enabled: false Enabled: false
# Offense count: 1 # Offense count: 1
Lint/EmptyWhen: Lint/EmptyWhen:
Exclude: Enabled: false
- 'lib/gitlab/diff/parser.rb'
# Offense count: 21 # Offense count: 22
Lint/HandleExceptions: Lint/HandleExceptions:
Enabled: false Enabled: false
# Offense count: 2 # Offense count: 2
Lint/Loop: Lint/Loop:
Exclude: Enabled: false
- 'app/mailers/notify.rb'
- 'lib/gitlab/path_locks_finder.rb'
# Offense count: 25 # Offense count: 27
Lint/ShadowingOuterLocalVariable: Lint/ShadowingOuterLocalVariable:
Enabled: false Enabled: false
# Offense count: 10 # Offense count: 10
# Cop supports --auto-correct. # Cop supports --auto-correct.
Lint/UnifiedInteger: Lint/UnifiedInteger:
Exclude: Enabled: false
- 'spec/lib/gitlab/metrics/method_call_spec.rb'
- 'spec/lib/gitlab/metrics/metric_spec.rb'
- 'spec/lib/gitlab/metrics/system_spec.rb'
- 'spec/lib/gitlab/metrics/transaction_spec.rb'
- 'spec/models/repository_spec.rb'
# Offense count: 16 # Offense count: 22
# Cop supports --auto-correct. # Cop supports --auto-correct.
Lint/UnneededSplatExpansion: Lint/UnneededSplatExpansion:
Exclude: Enabled: false
- 'app/services/projects/update_pages_service.rb'
- 'config/application.rb' # Offense count: 95
- 'lib/banzai/filter/gollum_tags_filter.rb'
- 'lib/bitbucket/page.rb'
- 'lib/gitlab/markup_helper.rb'
- 'spec/helpers/auth_helper_spec.rb'
- 'spec/models/project_wiki_spec.rb'
- 'spec/requests/api/projects_spec.rb'
- 'spec/requests/api/tags_spec.rb'
- 'spec/support/test_env.rb'
# Offense count: 93
# Cop supports --auto-correct. # Cop supports --auto-correct.
# Configuration parameters: IgnoreEmptyBlocks, AllowUnusedKeywordArguments. # Configuration parameters: IgnoreEmptyBlocks, AllowUnusedKeywordArguments.
Lint/UnusedBlockArgument: Lint/UnusedBlockArgument:
Enabled: false Enabled: false
# Offense count: 181 # Offense count: 186
# Cop supports --auto-correct. # Cop supports --auto-correct.
# Configuration parameters: AllowUnusedKeywordArguments, IgnoreEmptyMethods. # Configuration parameters: AllowUnusedKeywordArguments, IgnoreEmptyMethods.
Lint/UnusedMethodArgument: Lint/UnusedMethodArgument:
Enabled: false Enabled: false
# Offense count: 114 # Offense count: 112
# Configuration parameters: CountComments. # Configuration parameters: CountComments.
Metrics/BlockLength: Metrics/BlockLength:
Max: 300 Max: 303
# Offense count: 4 # Offense count: 4
# Cop supports --auto-correct. # Cop supports --auto-correct.
Performance/RedundantBlockCall: Performance/RedundantBlockCall:
Exclude: Enabled: false
- 'app/controllers/application_controller.rb'
- 'app/services/projects/update_mirror_service.rb'
- 'lib/gitlab/backend/shell.rb'
- 'lib/gitlab/optimistic_locking.rb'
# Offense count: 6 # Offense count: 6
# Cop supports --auto-correct. # Cop supports --auto-correct.
Performance/RedundantMatch: Performance/RedundantMatch:
Exclude: Enabled: false
- 'app/helpers/audit_events_helper.rb'
- 'app/models/external_issue.rb'
- 'lib/ci/api/helpers.rb'
- 'lib/extracts_path.rb'
- 'lib/gitlab/diff/highlight.rb'
- 'lib/gitlab/diff/parser.rb'
# Offense count: 36 # Offense count: 36
# Cop supports --auto-correct. # Cop supports --auto-correct.
...@@ -111,30 +82,18 @@ Performance/RedundantMerge: ...@@ -111,30 +82,18 @@ Performance/RedundantMerge:
# Offense count: 10 # Offense count: 10
RSpec/BeEql: RSpec/BeEql:
Exclude: Enabled: false
- 'spec/lib/gitlab/ldap/user_spec.rb'
- 'spec/lib/gitlab/o_auth/user_spec.rb'
- 'spec/lib/gitlab/saml/user_spec.rb'
- 'spec/workers/admin_emails_worker_spec.rb'
# Offense count: 26 # Offense count: 26
# Configuration parameters: CustomIncludeMethods. # Configuration parameters: CustomIncludeMethods.
RSpec/EmptyExampleGroup: RSpec/EmptyExampleGroup:
Exclude: Enabled: false
- 'spec/features/issues_spec.rb'
- 'spec/features/notes_on_merge_requests_spec.rb'
- 'spec/lib/gitlab/git_access_spec.rb'
- 'spec/models/project_snippet_spec.rb'
- 'spec/models/project_spec.rb'
- 'spec/models/protected_branch_spec.rb'
- 'spec/routing/routing_spec.rb'
# Offense count: 24 # Offense count: 24
RSpec/ExpectActual: RSpec/ExpectActual:
Exclude: Enabled: false
- 'spec/lib/gitlab/regex_spec.rb'
# Offense count: 70 # Offense count: 83
# Configuration parameters: EnforcedStyle, SupportedStyles. # Configuration parameters: EnforcedStyle, SupportedStyles.
# SupportedStyles: implicit, each, example # SupportedStyles: implicit, each, example
RSpec/HookArgument: RSpec/HookArgument:
...@@ -144,85 +103,46 @@ RSpec/HookArgument: ...@@ -144,85 +103,46 @@ RSpec/HookArgument:
# Configuration parameters: EnforcedStyle, SupportedStyles. # Configuration parameters: EnforcedStyle, SupportedStyles.
# SupportedStyles: is_expected, should # SupportedStyles: is_expected, should
RSpec/ImplicitExpect: RSpec/ImplicitExpect:
Exclude: Enabled: false
- 'spec/lib/banzai/renderer_spec.rb'
- 'spec/mailers/notify_spec.rb' # Offense count: 265
- 'spec/models/approval_spec.rb'
- 'spec/models/approver_group_spec.rb'
- 'spec/models/group_spec.rb'
- 'spec/models/project_group_link_spec.rb'
- 'spec/models/project_services/external_wiki_service_spec.rb'
- 'spec/models/project_spec.rb'
- 'spec/models/push_rule_spec.rb'
- 'spec/models/user_spec.rb'
# Offense count: 255
RSpec/LeadingSubject: RSpec/LeadingSubject:
Enabled: false Enabled: false
# Offense count: 255 # Offense count: 261
RSpec/LetSetup: RSpec/LetSetup:
Enabled: false Enabled: false
# Offense count: 19 # Offense count: 19
RSpec/MessageChain: RSpec/MessageChain:
Exclude: Enabled: false
- 'spec/finders/move_to_project_finder_spec.rb'
- 'spec/helpers/groups_helper_spec.rb' # Offense count: 518
- 'spec/helpers/issues_helper_spec.rb'
- 'spec/initializers/secret_token_spec.rb'
- 'spec/lib/banzai/filter/upload_link_filter_spec.rb'
- 'spec/models/build_spec.rb'
- 'spec/models/group_spec.rb'
- 'spec/models/project_spec.rb'
- 'spec/models/user_spec.rb'
- 'spec/requests/api/internal_spec.rb'
- 'spec/services/geo/enqueue_wiki_update_service_spec.rb'
- 'spec/services/merge_requests/build_service_spec.rb'
- 'spec/services/notes/create_service_spec.rb'
- 'spec/workers/emails_on_push_worker_spec.rb'
# Offense count: 513
# Configuration parameters: EnforcedStyle, SupportedStyles. # Configuration parameters: EnforcedStyle, SupportedStyles.
# SupportedStyles: have_received, receive # SupportedStyles: have_received, receive
RSpec/MessageSpies: RSpec/MessageSpies:
Enabled: false Enabled: false
# Offense count: 3220 # Offense count: 3278
RSpec/MultipleExpectations: RSpec/MultipleExpectations:
Max: 37 Max: 37
# Offense count: 2214 # Offense count: 2287
RSpec/NamedSubject: RSpec/NamedSubject:
Enabled: false Enabled: false
# Offense count: 2001 # Offense count: 2111
# Configuration parameters: MaxNesting. # Configuration parameters: MaxNesting.
RSpec/NestedGroups: RSpec/NestedGroups:
Enabled: false Enabled: false
# Offense count: 36 # Offense count: 36
RSpec/RepeatedDescription: RSpec/RepeatedDescription:
Exclude: Enabled: false
- 'spec/controllers/projects/todo_controller_spec.rb'
- 'spec/features/boards/boards_spec.rb'
- 'spec/helpers/issuables_helper_spec.rb'
- 'spec/lib/banzai/filter/emoji_filter_spec.rb'
- 'spec/models/commit_range_spec.rb'
- 'spec/models/global_milestone_spec.rb'
- 'spec/models/hooks/system_hook_spec.rb'
- 'spec/models/path_lock_spec.rb'
- 'spec/requests/api/commits_spec.rb'
- 'spec/requests/api/merge_requests_spec.rb'
- 'spec/requests/api/users_spec.rb'
- 'spec/requests/lfs_http_spec.rb'
- 'spec/routing/admin_routing_spec.rb'
- 'spec/services/notification_service_spec.rb'
# Offense count: 1 # Offense count: 1
RSpec/SingleArgumentMessageChain: RSpec/SingleArgumentMessageChain:
Exclude: Enabled: false
- 'spec/requests/api/internal_spec.rb'
# Offense count: 137 # Offense count: 137
RSpec/SubjectStub: RSpec/SubjectStub:
...@@ -235,7 +155,7 @@ RSpec/SubjectStub: ...@@ -235,7 +155,7 @@ RSpec/SubjectStub:
Rails/DynamicFindBy: Rails/DynamicFindBy:
Enabled: false Enabled: false
# Offense count: 1003 # Offense count: 1010
# Cop supports --auto-correct. # Cop supports --auto-correct.
# Configuration parameters: Include. # Configuration parameters: Include.
# Include: spec/**/*, test/**/* # Include: spec/**/*, test/**/*
...@@ -246,43 +166,26 @@ Rails/HttpPositionalArguments: ...@@ -246,43 +166,26 @@ Rails/HttpPositionalArguments:
Rails/OutputSafety: Rails/OutputSafety:
Enabled: false Enabled: false
# Offense count: 191 # Offense count: 203
# Configuration parameters: EnforcedStyle, SupportedStyles. # Configuration parameters: EnforcedStyle, SupportedStyles.
# SupportedStyles: strict, flexible # SupportedStyles: strict, flexible
Rails/TimeZone: Rails/TimeZone:
Enabled: false Enabled: false
# Offense count: 15 # Offense count: 16
# Cop supports --auto-correct. # Cop supports --auto-correct.
# Configuration parameters: Include. # Configuration parameters: Include.
# Include: app/models/**/*.rb # Include: app/models/**/*.rb
Rails/Validation: Rails/Validation:
Exclude: Enabled: false
- 'app/models/ci/build.rb'
- 'app/models/ci/pipeline.rb'
- 'app/models/ci/runner_project.rb'
- 'app/models/ci/trigger.rb'
- 'app/models/commit_status.rb'
- 'app/models/members/group_member.rb'
- 'app/models/members/project_member.rb'
- 'app/models/pages_domain.rb'
- 'app/models/project.rb'
- 'app/models/protected_branch.rb'
# Offense count: 9 # Offense count: 9
# Cop supports --auto-correct. # Cop supports --auto-correct.
# Configuration parameters: AutoCorrect. # Configuration parameters: AutoCorrect.
Security/JSONLoad: Security/JSONLoad:
Exclude: Enabled: false
- 'db/migrate/20161019190736_migrate_sidekiq_queues_from_default.rb'
- 'db/migrate/20161021185735_migrate_ee_sidekiq_queues_from_default.rb'
- 'db/migrate/20161024042317_migrate_mailroom_queue_from_default.rb'
- 'db/migrate/20161124141322_migrate_process_commit_worker_jobs.rb'
- 'spec/migrations/migrate_process_commit_worker_jobs_spec.rb'
- 'spec/models/project_services/irker_service_spec.rb'
- 'spec/support/stub_gitlab_calls.rb'
# Offense count: 363 # Offense count: 365
# Cop supports --auto-correct. # Cop supports --auto-correct.
# Configuration parameters: EnforcedStyle, SupportedStyles, IndentationWidth. # Configuration parameters: EnforcedStyle, SupportedStyles, IndentationWidth.
# SupportedStyles: with_first_parameter, with_fixed_indentation # SupportedStyles: with_first_parameter, with_fixed_indentation
...@@ -303,7 +206,7 @@ Style/AndOr: ...@@ -303,7 +206,7 @@ Style/AndOr:
Style/BarePercentLiterals: Style/BarePercentLiterals:
Enabled: false Enabled: false
# Offense count: 368 # Offense count: 387
# Cop supports --auto-correct. # Cop supports --auto-correct.
# Configuration parameters: EnforcedStyle, SupportedStyles. # Configuration parameters: EnforcedStyle, SupportedStyles.
# SupportedStyles: braces, no_braces, context_dependent # SupportedStyles: braces, no_braces, context_dependent
...@@ -312,15 +215,9 @@ Style/BracesAroundHashParameters: ...@@ -312,15 +215,9 @@ Style/BracesAroundHashParameters:
# Offense count: 6 # Offense count: 6
Style/CaseEquality: Style/CaseEquality:
Exclude: Enabled: false
- 'app/helpers/auth_helper.rb'
- 'app/models/commit.rb'
- 'app/models/protected_branch.rb'
- 'app/services/projects/download_service.rb'
- 'config/initializers/trusted_proxies.rb'
- 'spec/features/calendar_spec.rb'
# Offense count: 40 # Offense count: 41
# Cop supports --auto-correct. # Cop supports --auto-correct.
Style/ColonMethodCall: Style/ColonMethodCall:
Enabled: false Enabled: false
...@@ -330,11 +227,7 @@ Style/ColonMethodCall: ...@@ -330,11 +227,7 @@ Style/ColonMethodCall:
# Configuration parameters: Keywords. # Configuration parameters: Keywords.
# Keywords: TODO, FIXME, OPTIMIZE, HACK, REVIEW # Keywords: TODO, FIXME, OPTIMIZE, HACK, REVIEW
Style/CommentAnnotation: Style/CommentAnnotation:
Exclude: Enabled: false
- 'config/routes/user.rb'
- 'lib/api/commits.rb'
- 'lib/api/entities.rb'
- 'spec/requests/api/project_snippets_spec.rb'
# Offense count: 33 # Offense count: 33
# Cop supports --auto-correct. # Cop supports --auto-correct.
...@@ -343,67 +236,42 @@ Style/CommentAnnotation: ...@@ -343,67 +236,42 @@ Style/CommentAnnotation:
Style/ConditionalAssignment: Style/ConditionalAssignment:
Enabled: false Enabled: false
# Offense count: 1258 # Offense count: 1279
# Cop supports --auto-correct. # Cop supports --auto-correct.
# Configuration parameters: EnforcedStyle, SupportedStyles. # Configuration parameters: EnforcedStyle, SupportedStyles.
# SupportedStyles: leading, trailing # SupportedStyles: leading, trailing
Style/DotPosition: Style/DotPosition:
Enabled: false Enabled: false
# Offense count: 13 # Offense count: 19
Style/DoubleNegation: Style/DoubleNegation:
Exclude: Enabled: false
- 'app/controllers/users_controller.rb'
- 'app/models/commit.rb'
- 'app/models/merge_request.rb'
- 'app/models/project.rb'
- 'app/models/project_services/buildkite_service.rb'
- 'app/models/project_services/drone_ci_service.rb'
- 'app/models/project_wiki.rb'
- 'app/models/push_rule.rb'
- 'app/models/user.rb'
- 'config/initializers/6_validations.rb'
- 'lib/banzai/filter/relative_link_filter.rb'
- 'lib/gitlab/sherlock.rb'
- 'lib/gitlab/sherlock/middleware.rb'
# Offense count: 10 # Offense count: 10
# Cop supports --auto-correct. # Cop supports --auto-correct.
Style/EachWithObject: Style/EachWithObject:
Exclude: Enabled: false
- 'app/controllers/projects/cycle_analytics_controller.rb'
- 'app/models/concerns/elastic/application_search.rb'
- 'app/models/protected_branch.rb'
- 'app/services/merge_requests/get_urls_service.rb'
- 'lib/ci/ansi2html.rb'
- 'lib/expand_variables.rb'
- 'lib/gitlab/ee_compat_check.rb'
- 'lib/gitlab/import_export/members_mapper.rb'
- 'lib/gitlab/import_export/relation_factory.rb'
# Offense count: 24 # Offense count: 25
# Cop supports --auto-correct. # Cop supports --auto-correct.
# Configuration parameters: EnforcedStyle, SupportedStyles. # Configuration parameters: EnforcedStyle, SupportedStyles.
# SupportedStyles: empty, nil, both # SupportedStyles: empty, nil, both
Style/EmptyElse: Style/EmptyElse:
Enabled: false Enabled: false
# Offense count: 3 # Offense count: 4
# Cop supports --auto-correct. # Cop supports --auto-correct.
Style/EmptyLiteral: Style/EmptyLiteral:
Exclude: Enabled: false
- 'features/steps/project/commits/commits.rb'
- 'lib/gitlab/fogbugz_import/importer.rb'
- 'spec/lib/gitlab/workhorse_spec.rb'
# Offense count: 65 # Offense count: 64
# Cop supports --auto-correct. # Cop supports --auto-correct.
# Configuration parameters: EnforcedStyle, SupportedStyles. # Configuration parameters: EnforcedStyle, SupportedStyles.
# SupportedStyles: compact, expanded # SupportedStyles: compact, expanded
Style/EmptyMethod: Style/EmptyMethod:
Enabled: false Enabled: false
# Offense count: 155 # Offense count: 159
# Cop supports --auto-correct. # Cop supports --auto-correct.
# Configuration parameters: AllowForAlignment, ForceEqualSignAlignment. # Configuration parameters: AllowForAlignment, ForceEqualSignAlignment.
Style/ExtraSpacing: Style/ExtraSpacing:
...@@ -413,50 +281,31 @@ Style/ExtraSpacing: ...@@ -413,50 +281,31 @@ Style/ExtraSpacing:
# Configuration parameters: EnforcedStyle, SupportedStyles. # Configuration parameters: EnforcedStyle, SupportedStyles.
# SupportedStyles: format, sprintf, percent # SupportedStyles: format, sprintf, percent
Style/FormatString: Style/FormatString:
Exclude: Enabled: false
- 'app/models/ci/pipeline.rb'
- 'app/services/gravatar_service.rb' # Offense count: 287
- 'config/initializers/rack_lineprof.rb'
- 'lib/gitlab/version_info.rb'
- 'spec/factories/lfs_objects.rb'
- 'spec/services/issues/move_service_spec.rb'
# Offense count: 283
# Configuration parameters: MinBodyLength. # Configuration parameters: MinBodyLength.
Style/GuardClause: Style/GuardClause:
Enabled: false Enabled: false
# Offense count: 14 # Offense count: 14
Style/IfInsideElse: Style/IfInsideElse:
Exclude: Enabled: false
- 'app/controllers/projects/forks_controller.rb'
- 'app/controllers/projects/imports_controller.rb' # Offense count: 186
- 'app/helpers/milestones_helper.rb'
- 'app/helpers/projects_helper.rb'
- 'app/helpers/tree_helper.rb'
- 'app/services/system_note_service.rb'
- 'config/initializers/1_settings.rb'
- 'lib/gitlab/auth.rb'
- 'lib/gitlab/checks/change_access.rb'
- 'lib/gitlab/database.rb'
- 'lib/gitlab/elastic/project_search_results.rb'
- 'spec/controllers/projects/merge_requests_controller_spec.rb'
- 'spec/support/import_export/export_file_helper.rb'
# Offense count: 190
# Cop supports --auto-correct. # Cop supports --auto-correct.
# Configuration parameters: MaxLineLength. # Configuration parameters: MaxLineLength.
Style/IfUnlessModifier: Style/IfUnlessModifier:
Enabled: false Enabled: false
# Offense count: 57 # Offense count: 61
# Cop supports --auto-correct. # Cop supports --auto-correct.
# Configuration parameters: EnforcedStyle, SupportedStyles, IndentationWidth. # Configuration parameters: EnforcedStyle, SupportedStyles, IndentationWidth.
# SupportedStyles: special_inside_parentheses, consistent, align_brackets # SupportedStyles: special_inside_parentheses, consistent, align_brackets
Style/IndentArray: Style/IndentArray:
Enabled: false Enabled: false
# Offense count: 122 # Offense count: 128
# Cop supports --auto-correct. # Cop supports --auto-correct.
# Configuration parameters: EnforcedStyle, SupportedStyles, IndentationWidth. # Configuration parameters: EnforcedStyle, SupportedStyles, IndentationWidth.
# SupportedStyles: special_inside_parentheses, consistent, align_braces # SupportedStyles: special_inside_parentheses, consistent, align_braces
...@@ -473,48 +322,23 @@ Style/Lambda: ...@@ -473,48 +322,23 @@ Style/Lambda:
# Offense count: 5 # Offense count: 5
# Cop supports --auto-correct. # Cop supports --auto-correct.
Style/LineEndConcatenation: Style/LineEndConcatenation:
Exclude: Enabled: false
- 'app/helpers/preferences_helper.rb'
- 'app/helpers/tree_helper.rb'
- 'app/models/merge_request.rb'
- 'spec/lib/gitlab/gfm/reference_rewriter_spec.rb'
# Offense count: 18 # Offense count: 19
# Cop supports --auto-correct. # Cop supports --auto-correct.
Style/MethodCallParentheses: Style/MethodCallParentheses:
Exclude: Enabled: false
- 'lib/ci/ansi2html.rb'
- 'spec/controllers/dashboard/todos_controller_spec.rb'
- 'spec/controllers/projects/issues_controller_spec.rb'
- 'spec/controllers/projects/merge_requests_controller_spec.rb'
- 'spec/controllers/projects/snippets_controller_spec.rb'
- 'spec/features/calendar_spec.rb'
- 'spec/features/dashboard/datetime_on_tooltips_spec.rb'
- 'spec/helpers/submodule_helper_spec.rb'
- 'spec/requests/api/internal_spec.rb'
# Offense count: 9 # Offense count: 9
Style/MethodMissing: Style/MethodMissing:
Exclude: Enabled: false
- 'app/controllers/projects/application_controller.rb'
- 'app/models/commit.rb'
- 'app/models/network/commit.rb'
- 'app/models/repository.rb'
- 'config/initializers/gollum.rb'
- 'lib/bitbucket/collection.rb'
- 'lib/gitlab/github_import/client.rb'
- 'lib/gitlab/lazy.rb'
- 'lib/uploaded_file.rb'
# Offense count: 3 # Offense count: 3
# Cop supports --auto-correct. # Cop supports --auto-correct.
Style/MultilineIfModifier: Style/MultilineIfModifier:
Exclude: Enabled: false
- 'app/helpers/snippets_helper.rb'
- 'app/models/merge_request.rb'
- 'app/models/project_wiki.rb'
# Offense count: 161 # Offense count: 200
# Cop supports --auto-correct. # Cop supports --auto-correct.
Style/MutableConstant: Style/MutableConstant:
Enabled: false Enabled: false
...@@ -522,51 +346,21 @@ Style/MutableConstant: ...@@ -522,51 +346,21 @@ Style/MutableConstant:
# Offense count: 9 # Offense count: 9
# Cop supports --auto-correct. # Cop supports --auto-correct.
Style/NestedParenthesizedCalls: Style/NestedParenthesizedCalls:
Exclude: Enabled: false
- 'app/helpers/commits_helper.rb'
- 'app/workers/irker_worker.rb'
- 'spec/factories/groups.rb'
- 'spec/lib/gitlab/ci/build/artifacts/metadata/entry_spec.rb'
- 'spec/lib/gitlab/email/message/repository_push_spec.rb'
# Offense count: 16 # Offense count: 16
# Cop supports --auto-correct. # Cop supports --auto-correct.
# Configuration parameters: EnforcedStyle, MinBodyLength, SupportedStyles. # Configuration parameters: EnforcedStyle, MinBodyLength, SupportedStyles.
# SupportedStyles: skip_modifier_ifs, always # SupportedStyles: skip_modifier_ifs, always
Style/Next: Style/Next:
Exclude: Enabled: false
- 'app/models/namespace.rb'
- 'app/models/user.rb' # Offense count: 19
- 'app/services/merge_requests/refresh_service.rb'
- 'config/initializers/secret_token.rb'
- 'db/migrate/20161220141214_remove_dot_git_from_group_names.rb'
- 'db/migrate/20161226122833_remove_dot_git_from_usernames.rb'
- 'lib/banzai/filter/math_filter.rb'
- 'lib/banzai/filter/table_of_contents_filter.rb'
- 'lib/ee/gitlab/ldap/sync/group.rb'
- 'lib/gitlab/fogbugz_import/importer.rb'
- 'lib/gitlab/metrics/instrumentation.rb'
- 'lib/gitlab/metrics/sampler.rb'
- 'lib/gitlab/project_search_results.rb'
- 'lib/gitlab/sidekiq_cluster/cli.rb'
# Offense count: 15
# Cop supports --auto-correct. # Cop supports --auto-correct.
# Configuration parameters: EnforcedOctalStyle, SupportedOctalStyles. # Configuration parameters: EnforcedOctalStyle, SupportedOctalStyles.
# SupportedOctalStyles: zero_with_o, zero_only # SupportedOctalStyles: zero_with_o, zero_only
Style/NumericLiteralPrefix: Style/NumericLiteralPrefix:
Exclude: Enabled: false
- 'config/initializers/1_settings.rb'
- 'config/initializers/secret_token.rb'
- 'lib/gitlab/backend/shell.rb'
- 'lib/gitlab/import_export/command_line_util.rb'
- 'lib/gitlab/workhorse.rb'
- 'spec/features/projects/import_export/export_file_spec.rb'
- 'spec/lib/gitlab/git/hook_spec.rb'
- 'spec/lib/gitlab/workhorse_spec.rb'
- 'spec/support/import_export/export_file_helper.rb'
- 'spec/support/test_env.rb'
- 'spec/tasks/gitlab/backup_rake_spec.rb'
# Offense count: 24 # Offense count: 24
# Cop supports --auto-correct. # Cop supports --auto-correct.
...@@ -575,12 +369,12 @@ Style/NumericLiteralPrefix: ...@@ -575,12 +369,12 @@ Style/NumericLiteralPrefix:
Style/NumericPredicate: Style/NumericPredicate:
Enabled: false Enabled: false
# Offense count: 31 # Offense count: 35
# Cop supports --auto-correct. # Cop supports --auto-correct.
Style/ParallelAssignment: Style/ParallelAssignment:
Enabled: false Enabled: false
# Offense count: 423 # Offense count: 427
# Cop supports --auto-correct. # Cop supports --auto-correct.
# Configuration parameters: PreferredDelimiters. # Configuration parameters: PreferredDelimiters.
Style/PercentLiteralDelimiters: Style/PercentLiteralDelimiters:
...@@ -591,31 +385,14 @@ Style/PercentLiteralDelimiters: ...@@ -591,31 +385,14 @@ Style/PercentLiteralDelimiters:
# Configuration parameters: EnforcedStyle, SupportedStyles. # Configuration parameters: EnforcedStyle, SupportedStyles.
# SupportedStyles: lower_case_q, upper_case_q # SupportedStyles: lower_case_q, upper_case_q
Style/PercentQLiterals: Style/PercentQLiterals:
Exclude: Enabled: false
- 'spec/lib/gitlab/diff/highlight_spec.rb'
- 'spec/models/project_services/bamboo_service_spec.rb'
- 'spec/models/project_services/teamcity_service_spec.rb'
- 'spec/workers/repository_import_worker_spec.rb'
# Offense count: 17 # Offense count: 17
# Cop supports --auto-correct. # Cop supports --auto-correct.
Style/PerlBackrefs: Style/PerlBackrefs:
Exclude: Enabled: false
- 'app/controllers/projects/application_controller.rb'
- 'app/helpers/submodule_helper.rb' # Offense count: 69
- 'app/models/concerns/elastic/issues_search.rb'
- 'app/models/concerns/elastic/merge_requests_search.rb'
- 'app/workers/admin_emails_worker.rb'
- 'lib/banzai/filter/abstract_reference_filter.rb'
- 'lib/banzai/filter/autolink_filter.rb'
- 'lib/banzai/filter/emoji_filter.rb'
- 'lib/banzai/filter/gollum_tags_filter.rb'
- 'lib/expand_variables.rb'
- 'lib/gitlab/diff/highlight.rb'
- 'lib/gitlab/search_results.rb'
- 'lib/gitlab/sherlock/query.rb'
# Offense count: 66
# Configuration parameters: NamePrefix, NamePrefixBlacklist, NameWhitelist. # Configuration parameters: NamePrefix, NamePrefixBlacklist, NameWhitelist.
# NamePrefix: is_, has_, have_ # NamePrefix: is_, has_, have_
# NamePrefixBlacklist: is_, has_, have_ # NamePrefixBlacklist: is_, has_, have_
...@@ -633,73 +410,42 @@ Style/PreferredHashMethods: ...@@ -633,73 +410,42 @@ Style/PreferredHashMethods:
# Offense count: 8 # Offense count: 8
# Cop supports --auto-correct. # Cop supports --auto-correct.
Style/Proc: Style/Proc:
Exclude: Enabled: false
- 'app/mailers/base_mailer.rb'
- 'app/models/milestone.rb'
- 'app/models/service.rb'
- 'lib/api/api_guard.rb'
- 'lib/gitlab/ldap/config.rb'
- 'spec/initializers/trusted_proxies_spec.rb'
# Offense count: 39 # Offense count: 52
# Cop supports --auto-correct. # Cop supports --auto-correct.
# Configuration parameters: EnforcedStyle, SupportedStyles. # Configuration parameters: EnforcedStyle, SupportedStyles.
# SupportedStyles: compact, exploded # SupportedStyles: compact, exploded
Style/RaiseArgs: Style/RaiseArgs:
Enabled: false Enabled: false
# Offense count: 5 # Offense count: 6
# Cop supports --auto-correct. # Cop supports --auto-correct.
Style/RedundantBegin: Style/RedundantBegin:
Exclude: Enabled: false
- 'app/models/ci/build.rb'
- 'app/models/merge_request.rb'
- 'app/services/projects/import_service.rb'
- 'app/workers/repository_update_mirror_worker.rb'
- 'app/workers/repository_update_remote_mirror_worker.rb'
# Offense count: 1 # Offense count: 1
# Cop supports --auto-correct. # Cop supports --auto-correct.
Style/RedundantException: Style/RedundantException:
Exclude: Enabled: false
- 'app/helpers/preferences_helper.rb'
# Offense count: 29 # Offense count: 29
# Cop supports --auto-correct. # Cop supports --auto-correct.
Style/RedundantFreeze: Style/RedundantFreeze:
Exclude: Enabled: false
- 'app/models/merge_request.rb'
- 'app/validators/color_validator.rb'
- 'app/validators/line_code_validator.rb'
- 'lib/api/templates.rb'
- 'lib/banzai/filter/blockquote_fence_filter.rb'
- 'lib/banzai/filter/gollum_tags_filter.rb'
- 'lib/banzai/filter/yaml_front_matter_filter.rb'
- 'lib/gitlab/diff/inline_diff.rb'
- 'lib/gitlab/ee_compat_check.rb'
- 'lib/gitlab/import_export/relation_factory.rb'
- 'lib/gitlab/incoming_email.rb'
- 'lib/gitlab/middleware/static.rb'
- 'lib/gitlab/regex.rb'
# Offense count: 11 # Offense count: 11
# Cop supports --auto-correct. # Cop supports --auto-correct.
# Configuration parameters: AllowMultipleReturnValues. # Configuration parameters: AllowMultipleReturnValues.
Style/RedundantReturn: Style/RedundantReturn:
Exclude: Enabled: false
- 'app/controllers/concerns/issuable_actions.rb'
- 'app/controllers/groups/application_controller.rb'
- 'app/controllers/groups_controller.rb'
- 'app/controllers/projects/labels_controller.rb'
- 'app/controllers/projects/milestones_controller.rb'
- 'app/services/access_token_validation_service.rb'
# Offense count: 458 # Offense count: 462
# Cop supports --auto-correct. # Cop supports --auto-correct.
Style/RedundantSelf: Style/RedundantSelf:
Enabled: false Enabled: false
# Offense count: 103 # Offense count: 109
# Cop supports --auto-correct. # Cop supports --auto-correct.
# Configuration parameters: EnforcedStyle, SupportedStyles, AllowInnerSlashes. # Configuration parameters: EnforcedStyle, SupportedStyles, AllowInnerSlashes.
# SupportedStyles: slashes, percent_r, mixed # SupportedStyles: slashes, percent_r, mixed
...@@ -714,30 +460,21 @@ Style/RescueModifier: ...@@ -714,30 +460,21 @@ Style/RescueModifier:
# Offense count: 8 # Offense count: 8
# Cop supports --auto-correct. # Cop supports --auto-correct.
Style/SelfAssignment: Style/SelfAssignment:
Exclude: Enabled: false
- 'app/models/project_team.rb'
- 'app/services/notification_service.rb'
- 'app/workers/concerns/geo_dynamic_backoff.rb'
- 'lib/api/runners.rb'
- 'spec/features/merge_requests/diff_notes_resolve_spec.rb'
- 'spec/support/import_export/configuration_helper.rb'
# Offense count: 2 # Offense count: 2
# Configuration parameters: Methods. # Configuration parameters: Methods.
# Methods: {"reduce"=>["acc", "elem"]}, {"inject"=>["acc", "elem"]} # Methods: {"reduce"=>["acc", "elem"]}, {"inject"=>["acc", "elem"]}
Style/SingleLineBlockParams: Style/SingleLineBlockParams:
Exclude: Enabled: false
- 'app/models/commit.rb'
- 'spec/support/services_shared_context.rb'
# Offense count: 50 # Offense count: 50
# Cop supports --auto-correct. # Cop supports --auto-correct.
# Configuration parameters: AllowIfMethodIsEmpty. # Configuration parameters: AllowIfMethodIsEmpty.
Style/SingleLineMethods: Style/SingleLineMethods:
Exclude: Enabled: false
- 'lib/ci/ansi2html.rb'
# Offense count: 158 # Offense count: 166
# Cop supports --auto-correct. # Cop supports --auto-correct.
# Configuration parameters: EnforcedStyle, SupportedStyles. # Configuration parameters: EnforcedStyle, SupportedStyles.
# SupportedStyles: space, no_space # SupportedStyles: space, no_space
...@@ -748,13 +485,7 @@ Style/SpaceBeforeBlockBraces: ...@@ -748,13 +485,7 @@ Style/SpaceBeforeBlockBraces:
# Cop supports --auto-correct. # Cop supports --auto-correct.
# Configuration parameters: AllowForAlignment. # Configuration parameters: AllowForAlignment.
Style/SpaceBeforeFirstArg: Style/SpaceBeforeFirstArg:
Exclude: Enabled: false
- 'config/routes/project.rb'
- 'features/steps/project/source/browse_files.rb'
- 'features/steps/project/source/markdown_render.rb'
- 'spec/models/service_spec.rb'
- 'spec/routing/project_routing_spec.rb'
- 'spec/services/system_note_service_spec.rb'
# Offense count: 44 # Offense count: 44
# Cop supports --auto-correct. # Cop supports --auto-correct.
...@@ -763,19 +494,19 @@ Style/SpaceBeforeFirstArg: ...@@ -763,19 +494,19 @@ Style/SpaceBeforeFirstArg:
Style/SpaceInLambdaLiteral: Style/SpaceInLambdaLiteral:
Enabled: false Enabled: false
# Offense count: 168 # Offense count: 180
# Cop supports --auto-correct. # Cop supports --auto-correct.
# Configuration parameters: EnforcedStyle, SupportedStyles, EnforcedStyleForEmptyBraces, SpaceBeforeBlockParameters. # Configuration parameters: EnforcedStyle, SupportedStyles, EnforcedStyleForEmptyBraces, SpaceBeforeBlockParameters.
# SupportedStyles: space, no_space # SupportedStyles: space, no_space
Style/SpaceInsideBlockBraces: Style/SpaceInsideBlockBraces:
Enabled: false Enabled: false
# Offense count: 116 # Offense count: 118
# Cop supports --auto-correct. # Cop supports --auto-correct.
Style/SpaceInsideBrackets: Style/SpaceInsideBrackets:
Enabled: false Enabled: false
# Offense count: 87 # Offense count: 86
# Cop supports --auto-correct. # Cop supports --auto-correct.
Style/SpaceInsideParens: Style/SpaceInsideParens:
Enabled: false Enabled: false
...@@ -783,10 +514,9 @@ Style/SpaceInsideParens: ...@@ -783,10 +514,9 @@ Style/SpaceInsideParens:
# Offense count: 6 # Offense count: 6
# Cop supports --auto-correct. # Cop supports --auto-correct.
Style/SpaceInsidePercentLiteralDelimiters: Style/SpaceInsidePercentLiteralDelimiters:
Exclude: Enabled: false
- 'lib/gitlab/git_access.rb'
# Offense count: 51 # Offense count: 53
# Cop supports --auto-correct. # Cop supports --auto-correct.
# Configuration parameters: SupportedStyles. # Configuration parameters: SupportedStyles.
# SupportedStyles: use_perl_names, use_english_names # SupportedStyles: use_perl_names, use_english_names
...@@ -800,7 +530,7 @@ Style/SpecialGlobalVars: ...@@ -800,7 +530,7 @@ Style/SpecialGlobalVars:
Style/StringLiteralsInInterpolation: Style/StringLiteralsInInterpolation:
Enabled: false Enabled: false
# Offense count: 40 # Offense count: 56
# Cop supports --auto-correct. # Cop supports --auto-correct.
# Configuration parameters: IgnoredMethods. # Configuration parameters: IgnoredMethods.
# IgnoredMethods: respond_to, define_method # IgnoredMethods: respond_to, define_method
...@@ -812,40 +542,29 @@ Style/SymbolProc: ...@@ -812,40 +542,29 @@ Style/SymbolProc:
# Configuration parameters: EnforcedStyle, SupportedStyles, AllowSafeAssignment. # Configuration parameters: EnforcedStyle, SupportedStyles, AllowSafeAssignment.
# SupportedStyles: require_parentheses, require_no_parentheses, require_parentheses_when_complex # SupportedStyles: require_parentheses, require_no_parentheses, require_parentheses_when_complex
Style/TernaryParentheses: Style/TernaryParentheses:
Exclude: Enabled: false
- 'app/models/project_services/hipchat_service.rb'
- 'features/support/capybara.rb'
- 'lib/api/projects.rb'
- 'lib/banzai/filter/table_of_contents_filter.rb'
- 'lib/gitlab/ci/build/artifacts/metadata/entry.rb'
- 'spec/support/capybara.rb'
# Offense count: 34 # Offense count: 36
# Cop supports --auto-correct. # Cop supports --auto-correct.
# Configuration parameters: EnforcedStyleForMultiline, SupportedStyles. # Configuration parameters: EnforcedStyleForMultiline, SupportedStyles.
# SupportedStyles: comma, consistent_comma, no_comma # SupportedStyles: comma, consistent_comma, no_comma
Style/TrailingCommaInArguments: Style/TrailingCommaInArguments:
Enabled: false Enabled: false
# Offense count: 153 # Offense count: 166
# Cop supports --auto-correct. # Cop supports --auto-correct.
# Configuration parameters: EnforcedStyleForMultiline, SupportedStyles. # Configuration parameters: EnforcedStyleForMultiline, SupportedStyles.
# SupportedStyles: comma, consistent_comma, no_comma # SupportedStyles: comma, consistent_comma, no_comma
Style/TrailingCommaInLiteral: Style/TrailingCommaInLiteral:
Enabled: false Enabled: false
# Offense count: 6 # Offense count: 7
# Cop supports --auto-correct. # Cop supports --auto-correct.
# Configuration parameters: AllowNamedUnderscoreVariables. # Configuration parameters: AllowNamedUnderscoreVariables.
Style/TrailingUnderscoreVariable: Style/TrailingUnderscoreVariable:
Exclude: Enabled: false
- 'app/controllers/admin/background_jobs_controller.rb'
- 'app/controllers/invites_controller.rb'
- 'app/controllers/projects/git_http_client_controller.rb'
- 'app/helpers/tab_helper.rb'
- 'lib/gitlab/logger.rb'
# Offense count: 87 # Offense count: 86
# Cop supports --auto-correct. # Cop supports --auto-correct.
Style/TrailingWhitespace: Style/TrailingWhitespace:
Enabled: false Enabled: false
...@@ -855,46 +574,19 @@ Style/TrailingWhitespace: ...@@ -855,46 +574,19 @@ Style/TrailingWhitespace:
# Configuration parameters: ExactNameMatch, AllowPredicates, AllowDSLWriters, IgnoreClassMethods, Whitelist. # Configuration parameters: ExactNameMatch, AllowPredicates, AllowDSLWriters, IgnoreClassMethods, Whitelist.
# Whitelist: to_ary, to_a, to_c, to_enum, to_h, to_hash, to_i, to_int, to_io, to_open, to_path, to_proc, to_r, to_regexp, to_str, to_s, to_sym # Whitelist: to_ary, to_a, to_c, to_enum, to_h, to_hash, to_i, to_int, to_io, to_open, to_path, to_proc, to_r, to_regexp, to_str, to_s, to_sym
Style/TrivialAccessors: Style/TrivialAccessors:
Exclude: Enabled: false
- 'app/models/external_issue.rb'
- 'app/serializers/base_serializer.rb'
- 'lib/ee/gitlab/ldap/group.rb'
- 'lib/gitlab/ldap/person.rb'
# Offense count: 2 # Offense count: 2
# Cop supports --auto-correct. # Cop supports --auto-correct.
Style/UnlessElse: Style/UnlessElse:
Exclude: Enabled: false
- 'lib/gitlab/project_search_results.rb'
- 'spec/features/issues/award_emoji_spec.rb'
# Offense count: 17 # Offense count: 17
# Cop supports --auto-correct. # Cop supports --auto-correct.
Style/UnneededInterpolation: Style/UnneededInterpolation:
Exclude: Enabled: false
- 'app/helpers/boards_helper.rb'
- 'app/helpers/projects_helper.rb'
- 'app/helpers/search_helper.rb'
- 'app/helpers/time_helper.rb'
- 'app/models/merge_request.rb'
- 'app/models/project_services/pushover_service.rb'
- 'features/steps/project/forked_merge_requests.rb'
- 'spec/features/admin/admin_projects_spec.rb'
- 'spec/features/issues_spec.rb'
- 'spec/features/projects/branches_spec.rb'
- 'spec/features/projects/issuable_templates_spec.rb'
- 'spec/lib/banzai/filter/issue_reference_filter_spec.rb'
- 'spec/views/projects/commit/_commit_box.html.haml_spec.rb'
# Offense count: 9 # Offense count: 9
# Cop supports --auto-correct. # Cop supports --auto-correct.
Style/ZeroLengthPredicate: Style/ZeroLengthPredicate:
Exclude: Enabled: false
- 'app/models/deploy_key.rb'
- 'app/models/geo_node_key.rb'
- 'app/models/merge_request.rb'
- 'app/models/network/commit.rb'
- 'app/models/network/graph.rb'
- 'app/models/project_services/asana_service.rb'
- 'app/models/repository.rb'
- 'lib/extracts_path.rb'
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