From f64a639bcfa1fc2bc89ca7db268f594306edfd7c Mon Sep 17 00:00:00 2001 From: GitLab Bot Date: Tue, 16 Mar 2021 18:18:33 +0000 Subject: Add latest changes from gitlab-org/gitlab@13-10-stable-ee --- .rubocop_todo.yml | 299 +++++++++++++++++++++++++++--------------------------- 1 file changed, 151 insertions(+), 148 deletions(-) (limited to '.rubocop_todo.yml') diff --git a/.rubocop_todo.yml b/.rubocop_todo.yml index 0189a3d6372..deb7f287f76 100644 --- a/.rubocop_todo.yml +++ b/.rubocop_todo.yml @@ -1,17 +1,17 @@ # This configuration was generated by # `rubocop --auto-gen-config` -# on 2021-01-11 15:49:32 UTC using RuboCop version 0.91.1. +# on 2021-02-24 14:52:20 UTC using RuboCop version 0.93.1. # The point is for the user to remove these configuration records # one by one as the offenses are removed from the code base. # Note that changes in the inspected code, or installation of new # versions of RuboCop, may require this file to be generated again. -# Offense count: 313 +# Offense count: 314 # Cop supports --auto-correct. Capybara/CurrentPathExpectation: Enabled: false -# Offense count: 218 +# Offense count: 221 Capybara/VisibilityMatcher: Enabled: false @@ -25,7 +25,7 @@ Graphql/IDType: Exclude: - 'app/graphql/mutations/boards/issues/issue_move_list.rb' -# Offense count: 2054 +# Offense count: 2270 # Cop supports --auto-correct. # Configuration parameters: EnforcedStyle, IndentationWidth. # SupportedStyles: with_first_argument, with_fixed_indentation @@ -50,28 +50,28 @@ Layout/BeginEndAlignment: - 'lib/gitlab/phabricator_import/project_creator.rb' - 'scripts/gitaly_test.rb' -# Offense count: 52 +# Offense count: 54 # Cop supports --auto-correct. # Configuration parameters: AllowAliasSyntax, AllowedMethods. # AllowedMethods: alias_method, public, protected, private Layout/EmptyLinesAroundAttributeAccessor: Enabled: false -# Offense count: 721 +# Offense count: 771 # Cop supports --auto-correct. # Configuration parameters: EnforcedStyle, IndentationWidth. # SupportedStyles: special_inside_parentheses, consistent, align_brackets Layout/FirstArrayElementIndentation: Enabled: false -# Offense count: 1592 +# Offense count: 1684 # Cop supports --auto-correct. # Configuration parameters: EnforcedStyle, IndentationWidth. # SupportedStyles: special_inside_parentheses, consistent, align_braces Layout/FirstHashElementIndentation: Enabled: false -# Offense count: 3019 +# Offense count: 3233 # Cop supports --auto-correct. # Configuration parameters: AllowMultipleStyles, EnforcedHashRocketStyle, EnforcedColonStyle, EnforcedLastArgumentHashStyle. # SupportedHashRocketStyles: key, separator, table @@ -80,14 +80,14 @@ Layout/FirstHashElementIndentation: Layout/HashAlignment: Enabled: false -# Offense count: 73 +# Offense count: 14544 # Cop supports --auto-correct. # Configuration parameters: AutoCorrect, AllowHeredoc, AllowURI, URISchemes, IgnoreCopDirectives, IgnoredPatterns. # URISchemes: http, https Layout/LineLength: Max: 1313 -# Offense count: 163 +# Offense count: 170 # Cop supports --auto-correct. # Configuration parameters: EnforcedStyle, IndentationWidth. # SupportedStyles: aligned, indented @@ -113,22 +113,23 @@ Layout/RescueEnsureAlignment: Layout/SpaceAroundMethodCallOperator: Enabled: false -# Offense count: 725 +# Offense count: 754 # Cop supports --auto-correct. # Configuration parameters: EnforcedStyle. # SupportedStyles: require_no_space, require_space Layout/SpaceInLambdaLiteral: Enabled: false -# Offense count: 218 +# Offense count: 1209 # Cop supports --auto-correct. -# Configuration parameters: EnforcedStyle, EnforcedStyleForEmptyBraces, SpaceBeforeBlockParameters. +# Configuration parameters: EnforcedStyleForEmptyBraces, SpaceBeforeBlockParameters. # SupportedStyles: space, no_space # SupportedStylesForEmptyBraces: space, no_space Layout/SpaceInsideBlockBraces: Enabled: false + EnforcedStyle: space -# Offense count: 559 +# Offense count: 585 # Cop supports --auto-correct. # Configuration parameters: EnforcedStyle. # SupportedStyles: space, no_space @@ -141,7 +142,6 @@ Lint/BinaryOperatorWithIdenticalOperands: - 'ee/spec/lib/ee/gitlab/application_context_spec.rb' - 'spec/helpers/visibility_level_helper_spec.rb' - 'spec/lib/gitlab/conan_token_spec.rb' - - 'spec/tooling/danger/sidekiq_queues_spec.rb' - 'spec/lib/gitlab/git/conflict/parser_spec.rb' - 'spec/lib/gitlab/graphql/lazy_spec.rb' - 'spec/models/ci/build_trace_chunk_spec.rb' @@ -150,8 +150,9 @@ Lint/BinaryOperatorWithIdenticalOperands: - 'spec/models/metrics/dashboard/annotation_spec.rb' - 'spec/models/repository_spec.rb' - 'spec/models/ssh_host_key_spec.rb' + - 'spec/tooling/danger/sidekiq_queues_spec.rb' -# Offense count: 83 +# Offense count: 84 Lint/ConstantDefinitionInBlock: Enabled: false @@ -174,12 +175,12 @@ Lint/IdentityComparison: - 'spec/tooling/danger/weightage/maintainers_spec.rb' - 'spec/tooling/danger/weightage/reviewers_spec.rb' -# Offense count: 184 +# Offense count: 208 # Configuration parameters: MaximumRangeSize. Lint/MissingCopEnableDirective: Enabled: false -# Offense count: 14 +# Offense count: 13 Lint/MixedRegexpCaptureTypes: Exclude: - 'app/models/alert_management/alert.rb' @@ -188,7 +189,6 @@ Lint/MixedRegexpCaptureTypes: - 'ee/lib/gitlab/code_owners/reference_extractor.rb' - 'lib/gitlab/ci/pipeline/expression/lexeme/string.rb' - 'lib/gitlab/dependency_linker/gemfile_linker.rb' - - 'lib/gitlab/diff/inline_diff.rb' - 'lib/gitlab/diff/suggestions_parser.rb' - 'lib/gitlab/github_import/representation/note.rb' - 'lib/gitlab/metrics/system.rb' @@ -197,11 +197,26 @@ Lint/MixedRegexpCaptureTypes: - 'lib/gitlab/slash_commands/issue_new.rb' - 'lib/gitlab/slash_commands/run.rb' -# Offense count: 141 +# Offense count: 200 # Cop supports --auto-correct. Lint/RedundantCopDisableDirective: Enabled: false +# Offense count: 9 +# Cop supports --auto-correct. +# Configuration parameters: AllowedMethods. +# AllowedMethods: instance_of?, kind_of?, is_a?, eql?, respond_to?, equal? +Lint/RedundantSafeNavigation: + Exclude: + - 'app/controllers/concerns/labels_as_hash.rb' + - 'app/policies/note_policy.rb' + - 'app/services/users/update_canonical_email_service.rb' + - 'ee/app/presenters/iteration_presenter.rb' + - 'ee/app/services/ee/members/destroy_service.rb' + - 'ee/lib/ee/gitlab/email/handler/reply_processing.rb' + - 'qa/qa/specs/helpers/quarantine.rb' + - 'spec/controllers/boards/issues_controller_spec.rb' + # Offense count: 1 Lint/SelfAssignment: Exclude: @@ -225,57 +240,20 @@ Lint/UnreachableLoop: Lint/UselessMethodDefinition: Enabled: false -# Offense count: 7 -# Configuration parameters: IgnoredMethods, Max. +# Offense count: 5 +# Configuration parameters: IgnoredMethods. Metrics/AbcSize: - Exclude: - - 'app/helpers/issuables_helper.rb' - - 'app/services/merge_requests/build_service.rb' - - 'app/services/projects/create_service.rb' - - 'lib/api/helpers.rb' - - 'lib/gitlab/lograge/custom_options.rb' - - 'lib/gitlab/rack_attack.rb' - - 'qa/qa/resource/repository/push.rb' + Max: 61 -# Offense count: 15 -# Configuration parameters: IgnoredMethods, Max. +# Offense count: 13 +# Configuration parameters: IgnoredMethods. Metrics/CyclomaticComplexity: - Exclude: - - 'app/services/projects/create_service.rb' - - 'app/services/system_hooks_service.rb' - - 'ee/app/controllers/ee/groups_controller.rb' - - 'ee/app/helpers/ee/groups_helper.rb' - - 'ee/lib/security/ci_configuration/sast_build_actions.rb' - - 'lib/banzai/filter/abstract_reference_filter.rb' - - 'lib/declarative_policy/runner.rb' - - 'lib/gitlab/conflict/file.rb' - - 'tooling/danger/roulette.rb' - - 'lib/gitlab/diff/parser.rb' - - 'lib/gitlab/rack_attack.rb' - - 'lib/gitlab/sidekiq_cluster/cli.rb' - - 'lib/gitlab/utils/merge_hash.rb' - - 'lib/kramdown/parser/atlassian_document_format.rb' - - 'spec/support/cycle_analytics_helpers/test_generation.rb' + Max: 25 -# Offense count: 15 -# Configuration parameters: IgnoredMethods, Max. +# Offense count: 14 +# Configuration parameters: IgnoredMethods. Metrics/PerceivedComplexity: - Exclude: - - 'app/helpers/submodule_helper.rb' - - 'app/helpers/tab_helper.rb' - - 'app/services/projects/create_service.rb' - - 'ee/app/controllers/ee/groups_controller.rb' - - 'ee/app/helpers/ee/groups_helper.rb' - - 'ee/lib/security/ci_configuration/sast_build_actions.rb' - - 'lib/banzai/filter/abstract_reference_filter.rb' - - 'lib/banzai/renderer.rb' - - 'lib/declarative_policy/runner.rb' - - 'lib/gitlab/conflict/file.rb' - - 'tooling/danger/roulette.rb' - - 'lib/gitlab/rack_attack.rb' - - 'lib/gitlab/sidekiq_cluster/cli.rb' - - 'lib/gitlab/utils/merge_hash.rb' - - 'spec/support/cycle_analytics_helpers/test_generation.rb' + Max: 25 # Offense count: 1 # Cop supports --auto-correct. @@ -283,7 +261,7 @@ Migration/DepartmentName: Exclude: - 'app/models/commit.rb' -# Offense count: 184 +# Offense count: 196 # Configuration parameters: ExpectMatchingDefinition, CheckDefinitionPathHierarchy, Regex, IgnoreExecutableScripts, AllowedAcronyms. # AllowedAcronyms: CLI, DSL, ACL, API, ASCII, CPU, CSS, DNS, EOF, GUID, HTML, HTTP, HTTPS, ID, IP, JSON, LHS, QPS, RAM, RHS, RPC, SLA, SMTP, SQL, SSH, TCP, TLS, TTL, UDP, UI, UID, UUID, URI, URL, UTF8, VM, XML, XMPP, XSRF, XSS Naming/FileName: @@ -300,7 +278,7 @@ Naming/HeredocDelimiterCase: - 'spec/support/helpers/repo_helpers.rb' - 'spec/support/helpers/seed_repo.rb' -# Offense count: 308 +# Offense count: 321 # Configuration parameters: ForbiddenDelimiters. # ForbiddenDelimiters: (?-mix:(^|\s)(EO[A-Z]{1}|END)(\s|$)) Naming/HeredocDelimiterNaming: @@ -314,7 +292,7 @@ Naming/MethodParameterName: - 'lib/gitlab/diff/inline_diff.rb' - 'spec/support/helpers/key_generator_helper.rb' -# Offense count: 206 +# Offense count: 218 # Cop supports --auto-correct. # Configuration parameters: PreferredName. Naming/RescuedExceptionsVariableName: @@ -326,17 +304,17 @@ Performance/AncestorsInclude: Exclude: - 'lib/gitlab/ci/config/extendable/entry.rb' -# Offense count: 39 +# Offense count: 40 # Cop supports --auto-correct. Performance/BlockGivenWithExplicitBlock: Enabled: false -# Offense count: 28 +# Offense count: 29 # Configuration parameters: MinSize. Performance/CollectionLiteralInLoop: Enabled: false -# Offense count: 37 +# Offense count: 41 # Cop supports --auto-correct. Performance/ConstantRegexp: Enabled: false @@ -382,7 +360,6 @@ Performance/DeleteSuffix: - 'app/workers/concerns/application_worker.rb' - 'ee/app/models/geo/upload_registry.rb' - 'ee/app/workers/geo/file_download_dispatch_worker/attachment_job_finder.rb' - - 'lib/sentry/client/issue.rb' # Offense count: 13 # Cop supports --auto-correct. @@ -395,7 +372,7 @@ Performance/Detect: - 'spec/lib/gitlab/import_export/project/tree_restorer_spec.rb' - 'spec/models/event_spec.rb' -# Offense count: 116 +# Offense count: 121 Performance/MethodObjectAsBlock: Enabled: false @@ -439,7 +416,7 @@ Performance/Sum: - 'lib/peek/views/detailed_view.rb' - 'spec/models/namespace/root_storage_statistics_spec.rb' -# Offense count: 14717 +# Offense count: 15209 # Configuration parameters: Prefixes. # Prefixes: when, with, without RSpec/ContextWording: @@ -452,22 +429,17 @@ RSpec/EmptyExampleGroup: - 'ee/spec/services/personal_access_tokens/revoke_invalid_tokens_spec.rb' - 'spec/services/projects/prometheus/alerts/notify_service_spec.rb' -# Offense count: 1365 +# Offense count: 1428 # Cop supports --auto-correct. # Configuration parameters: EnforcedStyle. # SupportedStyles: method_call, block RSpec/ExpectChange: Enabled: false -# Offense count: 889 +# Offense count: 930 RSpec/ExpectInHook: Enabled: false -# Offense count: 16403 -# Configuration parameters: AllowSubject. -RSpec/MultipleMemoizedHelpers: - Max: 40 - # Offense count: 2352 # Cop supports --auto-correct. # Configuration parameters: Strict, EnforcedStyle, AllowedExplicitMatchers. @@ -475,27 +447,27 @@ RSpec/MultipleMemoizedHelpers: RSpec/PredicateMatcher: Enabled: false -# Offense count: 112 +# Offense count: 118 RSpec/RepeatedExampleGroupBody: Enabled: false -# Offense count: 219 +# Offense count: 225 RSpec/RepeatedExampleGroupDescription: Enabled: false -# Offense count: 655 +# Offense count: 667 # Cop supports --auto-correct. # Configuration parameters: EnforcedStyle. # SupportedStyles: and_return, block RSpec/ReturnFromStub: Enabled: false -# Offense count: 596 +# Offense count: 610 # Cop supports --auto-correct. RSpec/ScatteredLet: Enabled: false -# Offense count: 6 +# Offense count: 4 RSpec/ScatteredSetup: Exclude: - 'spec/requests/api/jobs_spec.rb' @@ -520,7 +492,7 @@ RSpec/VariableName: - 'spec/support/shared_examples/requests/api/graphql/group_and_project_boards_query_shared_examples.rb' - 'spec/support/shared_examples/services/boards/boards_list_service_shared_examples.rb' -# Offense count: 25 +# Offense count: 26 # Cop supports --auto-correct. # Configuration parameters: Include. # Include: app/models/**/*.rb @@ -549,25 +521,25 @@ Rails/BelongsTo: - 'app/models/deployment.rb' - 'app/models/environment.rb' -# Offense count: 90 +# Offense count: 93 # Configuration parameters: Database, Include. # SupportedDatabases: mysql, postgresql # Include: db/migrate/*.rb Rails/BulkChangeTable: Enabled: false -# Offense count: 153 +# Offense count: 155 # Cop supports --auto-correct. Rails/ContentTag: Enabled: false -# Offense count: 300 +# Offense count: 313 # Configuration parameters: Include. # Include: db/migrate/*.rb Rails/CreateTableWithTimestamps: Enabled: false -# Offense count: 347 +# Offense count: 361 # Configuration parameters: EnforcedStyle. # SupportedStyles: slashes, arguments Rails/FilePath: @@ -589,13 +561,13 @@ Rails/FindById: - 'spec/finders/concerns/finder_methods_spec.rb' - 'spec/finders/concerns/finder_with_cross_project_access_spec.rb' -# Offense count: 346 +# Offense count: 354 # Configuration parameters: Include. # Include: app/models/**/*.rb Rails/HasManyOrHasOneDependent: Enabled: false -# Offense count: 539 +# Offense count: 537 # Configuration parameters: Include. # Include: app/helpers/**/*.rb Rails/HelperInstanceVariable: @@ -615,11 +587,11 @@ Rails/IndexBy: - 'ee/lib/gitlab/analytics/type_of_work/tasks_by_type.rb' - 'ee/lib/gitlab/elastic/document_reference.rb' - 'ee/lib/gitlab/group_plans_preloader.rb' - - 'tooling/danger/sidekiq_queues.rb' - 'lib/gitlab/database/count/reltuples_count_strategy.rb' - 'lib/gitlab/language_detection.rb' + - 'tooling/danger/sidekiq_queues.rb' -# Offense count: 45 +# Offense count: 47 # Cop supports --auto-correct. Rails/IndexWith: Enabled: false @@ -629,24 +601,23 @@ Rails/Inquiry: Exclude: - 'spec/helpers/labels_helper_spec.rb' -# Offense count: 115 +# Offense count: 118 # Configuration parameters: Include. # Include: app/models/**/*.rb Rails/InverseOf: Enabled: false -# Offense count: 60 +# Offense count: 62 # Configuration parameters: Include. # Include: app/controllers/**/*.rb Rails/LexicallyScopedActionFilter: Enabled: false -# Offense count: 3 +# Offense count: 2 # Cop supports --auto-correct. Rails/LinkToBlank: Exclude: - 'app/helpers/projects_helper.rb' - - 'app/helpers/wiki_helper.rb' - 'ee/app/helpers/ee/user_callouts_helper.rb' # Offense count: 1 @@ -662,12 +633,12 @@ Rails/MailerName: Rails/NegateInclude: Enabled: false -# Offense count: 44 +# Offense count: 46 # Cop supports --auto-correct. Rails/Pick: Enabled: false -# Offense count: 110 +# Offense count: 123 # Cop supports --auto-correct. Rails/Pluck: Enabled: false @@ -679,7 +650,7 @@ Rails/Pluck: Rails/RakeEnvironment: Enabled: false -# Offense count: 58 +# Offense count: 62 # Cop supports --auto-correct. Rails/RedundantForeignKey: Enabled: false @@ -699,23 +670,23 @@ Rails/ShortI18n: - 'app/uploaders/content_type_whitelist.rb' - 'spec/views/shared/runners/show.html.haml_spec.rb' -# Offense count: 1080 +# Offense count: 1144 # Configuration parameters: ForbiddenMethods, AllowedMethods. # ForbiddenMethods: decrement!, decrement_counter, increment!, increment_counter, insert, insert!, insert_all, insert_all!, toggle!, touch, touch_all, update_all, update_attribute, update_column, update_columns, update_counters, upsert, upsert_all Rails/SkipsModelValidations: Enabled: false -# Offense count: 251 +# Offense count: 278 # Cop supports --auto-correct. Rails/SquishedSQLHeredocs: Enabled: false -# Offense count: 45 +# Offense count: 44 # Cop supports --auto-correct. Rails/WhereEquals: Enabled: false -# Offense count: 40 +# Offense count: 44 # Cop supports --auto-correct. # Configuration parameters: EnforcedStyle. # SupportedStyles: exists, where @@ -744,7 +715,7 @@ Security/YAMLLoad: Style/AccessorGrouping: Enabled: false -# Offense count: 12 +# Offense count: 11 # Cop supports --auto-correct. Style/ArrayCoercion: Exclude: @@ -756,12 +727,11 @@ Style/ArrayCoercion: - 'db/post_migrate/20200311130802_schedule_populate_user_highest_roles_table.rb' - 'db/post_migrate/20200805152108_migrate_null_external_diff_store_to_local_value.rb' - 'db/post_migrate/20200806173633_migrate_null_package_files_file_store_to_local_value.rb' - - 'ee/app/services/geo/blob_verification_secondary_service.rb' - 'ee/app/services/geo/repository_verification_secondary_service.rb' - 'ee/lib/ee/banzai/pipeline/gfm_pipeline.rb' - 'spec/support/helpers/lfs_http_helpers.rb' -# Offense count: 183 +# Offense count: 188 # Cop supports --auto-correct. # Configuration parameters: EnforcedStyle. # SupportedStyles: percent_q, bare_percent @@ -775,20 +745,35 @@ Style/BisectedAttrAccessor: - 'lib/system_check/base_check.rb' - 'qa/qa/resource/api_fabricator.rb' -# Offense count: 36 +# Offense count: 42 # Cop supports --auto-correct. Style/CaseLikeIf: Enabled: false -# Offense count: 15 +# Offense count: 10 +# Cop supports --auto-correct. +# Configuration parameters: IgnoredMethods. +# IgnoredMethods: ==, equal?, eql? +Style/ClassEqualityComparison: + Exclude: + - 'app/finders/security/jobs_finder.rb' + - 'app/services/projects/overwrite_project_service.rb' + - 'app/uploaders/dependency_proxy/file_uploader.rb' + - 'ee/app/graphql/resolvers/vulnerabilities/issue_links_resolver.rb' + - 'lib/gitlab/background_migration/user_mentions/models/note.rb' + - 'lib/gitlab/diff/file.rb' + - 'lib/gitlab/git.rb' + - 'lib/gitlab/import_export/relation_tree_restorer.rb' + - 'spec/requests/api/services_spec.rb' + - 'spec/support/shared_examples/lib/gitlab/import_export/relation_factory_shared_examples.rb' + +# Offense count: 13 Style/CombinableLoops: Exclude: - - 'app/models/application_setting.rb' - 'ee/db/fixtures/development/30_customizable_cycle_analytics.rb' - 'ee/lib/gitlab/audit/events/preloader.rb' - 'ee/spec/finders/snippets_finder_spec.rb' - 'ee/spec/lib/ee/gitlab/background_migration/remove_duplicate_cs_findings_spec.rb' - - 'rubocop/code_reuse_helpers.rb' - 'spec/features/merge_request/user_suggests_changes_on_diff_spec.rb' - 'spec/finders/packages/group_packages_finder_spec.rb' - 'spec/migrations/cleanup_optimistic_locking_nulls_pt2_fixed_spec.rb' @@ -814,14 +799,14 @@ Style/EachWithObject: - 'lib/gitlab/i18n/po_linter.rb' - 'lib/gitlab/import_export/members_mapper.rb' -# Offense count: 53 +# Offense count: 55 # Cop supports --auto-correct. # Configuration parameters: EnforcedStyle. # SupportedStyles: empty, nil, both Style/EmptyElse: Enabled: false -# Offense count: 197 +# Offense count: 205 # Cop supports --auto-correct. # Configuration parameters: EnforcedStyle. # SupportedStyles: compact, expanded @@ -835,66 +820,75 @@ Style/ExpandPathArguments: - 'cable/config.ru' - 'config.ru' -# Offense count: 116 +# Offense count: 118 # Cop supports --auto-correct. Style/ExplicitBlockArgument: Enabled: false -# Offense count: 555 +# Offense count: 581 # Cop supports --auto-correct. # Configuration parameters: EnforcedStyle. # SupportedStyles: format, sprintf, percent Style/FormatString: Enabled: false -# Offense count: 61 +# Offense count: 67 # Cop supports --auto-correct. Style/GlobalStdStream: Enabled: false -# Offense count: 879 +# Offense count: 897 # Configuration parameters: MinBodyLength. Style/GuardClause: Enabled: false -# Offense count: 56 +# Offense count: 59 # Cop supports --auto-correct. # Configuration parameters: EnforcedStyle. # SupportedStyles: braces, no_braces Style/HashAsLastArrayItem: Enabled: false -# Offense count: 66 +# Offense count: 70 # Cop supports --auto-correct. Style/HashEachMethods: Enabled: false -# Offense count: 33 +# Offense count: 34 # Configuration parameters: AllowIfModifier. Style/IfInsideElse: Enabled: false -# Offense count: 1888 +# Offense count: 1934 # Cop supports --auto-correct. Style/IfUnlessModifier: Enabled: false -# Offense count: 68 +# Offense count: 64 # Cop supports --auto-correct. Style/KeywordParametersOrder: Enabled: false -# Offense count: 431 +# Offense count: 458 # Cop supports --auto-correct. # Configuration parameters: EnforcedStyle. # SupportedStyles: line_count_dependent, lambda, literal Style/Lambda: Enabled: false -# Offense count: 20 +# Offense count: 21 Style/MissingRespondToMissing: Enabled: false +# Offense count: 5 +Style/MixinUsage: + Exclude: + - 'spec/factories/ci/builds.rb' + - 'spec/factories/ci/job_artifacts.rb' + - 'spec/factories/lfs_objects.rb' + - 'spec/factories/notes.rb' + - 'spec/lib/gitlab/import_export/version_checker_spec.rb' + # Offense count: 35 # Cop supports --auto-correct. # Configuration parameters: EnforcedStyle, MinBodyLength. @@ -902,36 +896,44 @@ Style/MissingRespondToMissing: Style/Next: Enabled: false -# Offense count: 98 +# Offense count: 101 # Cop supports --auto-correct. # Configuration parameters: EnforcedOctalStyle. # SupportedOctalStyles: zero_with_o, zero_only Style/NumericLiteralPrefix: Enabled: false -# Offense count: 135 +# Offense count: 140 # Cop supports --auto-correct. Style/ParallelAssignment: Enabled: false -# Offense count: 2601 +# Offense count: 2698 # Cop supports --auto-correct. # Configuration parameters: PreferredDelimiters. Style/PercentLiteralDelimiters: Enabled: false -# Offense count: 1 +# Offense count: 247 # Cop supports --auto-correct. -# Configuration parameters: EnforcedStyle. +# Configuration parameters: . # SupportedStyles: compact, exploded Style/RaiseArgs: Enabled: false + EnforcedStyle: exploded -# Offense count: 65 +# Offense count: 73 # Cop supports --auto-correct. Style/RedundantAssignment: Enabled: false +# Offense count: 2 +# Cop supports --auto-correct. +Style/RedundantBegin: + Exclude: + - 'app/services/clusters/applications/check_installation_progress_service.rb' + - 'lib/gitlab/database/postgres_hll/batch_distinct_counter.rb' + # Offense count: 26 # Cop supports --auto-correct. # Configuration parameters: SafeForConstants. @@ -949,21 +951,23 @@ Style/RedundantFetchBlock: Style/RedundantFileExtensionInRequire: Enabled: false -# Offense count: 220 +# Offense count: 248 # Cop supports --auto-correct. Style/RedundantFreeze: Enabled: false -# Offense count: 182 +# Offense count: 206 # Cop supports --auto-correct. Style/RedundantInterpolation: Enabled: false -# Offense count: 8 +# Offense count: 10 # Cop supports --auto-correct. Style/RedundantRegexpCharacterClass: Exclude: - 'app/models/concerns/taskable.rb' + - 'ee/app/models/saml_provider.rb' + - 'lib/banzai/filter/markdown_pre_escape_filter.rb' - 'lib/gitlab/authorized_keys.rb' - 'lib/gitlab/fogbugz_import/repository.rb' - 'lib/gitlab/quick_actions/substitution_definition.rb' @@ -971,12 +975,12 @@ Style/RedundantRegexpCharacterClass: - 'spec/features/merge_request/user_views_open_merge_request_spec.rb' - 'spec/tasks/gitlab/usage_data_rake_spec.rb' -# Offense count: 270 +# Offense count: 279 # Cop supports --auto-correct. Style/RedundantRegexpEscape: Enabled: false -# Offense count: 920 +# Offense count: 968 # Cop supports --auto-correct. Style/RedundantSelf: Enabled: false @@ -988,26 +992,26 @@ Style/RedundantSelfAssignment: - 'app/models/concerns/issuable.rb' - 'spec/db/schema_spec.rb' -# Offense count: 196 +# Offense count: 213 # Cop supports --auto-correct. # Configuration parameters: EnforcedStyle, AllowInnerSlashes. # SupportedStyles: slashes, percent_r, mixed Style/RegexpLiteral: Enabled: false -# Offense count: 50 +# Offense count: 53 # Cop supports --auto-correct. Style/RescueModifier: Enabled: false -# Offense count: 346 +# Offense count: 359 # Cop supports --auto-correct. # Configuration parameters: EnforcedStyle. # SupportedStyles: implicit, explicit Style/RescueStandardError: Enabled: false -# Offense count: 115 +# Offense count: 123 # Cop supports --auto-correct. Style/SingleArgumentDig: Enabled: false @@ -1017,34 +1021,33 @@ Style/SingleArgumentDig: Style/SlicingWithRange: Enabled: false -# Offense count: 61 +# Offense count: 63 # Configuration parameters: AllowModifier. Style/SoleNestedConditional: Enabled: false -# Offense count: 121 +# Offense count: 120 # Cop supports --auto-correct. # Configuration parameters: . # SupportedStyles: use_perl_names, use_english_names Style/SpecialGlobalVars: EnforcedStyle: use_perl_names -# Offense count: 545 +# Offense count: 562 # Cop supports --auto-correct. Style/StringConcatenation: Enabled: false -# Offense count: 108 +# Offense count: 109 # Cop supports --auto-correct. # Configuration parameters: EnforcedStyle. # SupportedStyles: single_quotes, double_quotes Style/StringLiteralsInInterpolation: Enabled: false -# Offense count: 292 +# Offense count: 293 # Cop supports --auto-correct. # Configuration parameters: IgnoredMethods. # IgnoredMethods: respond_to, define_method Style/SymbolProc: Enabled: false - -- cgit v1.2.3