From c4902f76c5b162bca710805fd8af3c6a2c13b18f Mon Sep 17 00:00:00 2001 From: Doug Stull <dstull@gitlab.com> Date: Mon, 14 Dec 2020 13:39:57 -0500 Subject: [PATCH] Document rubocop manual todo file - for standards of use. --- .rubocop_manual_todo.yml | 31 +++--- .rubocop_todo.yml | 221 +++++++++++++++++++-------------------- 2 files changed, 119 insertions(+), 133 deletions(-) diff --git a/.rubocop_manual_todo.yml b/.rubocop_manual_todo.yml index 2ffbef850d0b2..9adbd2cc828a3 100644 --- a/.rubocop_manual_todo.yml +++ b/.rubocop_manual_todo.yml @@ -1,3 +1,15 @@ +--- +# This file is not meant to store permanent exclusions. +# - for permanent lists, place them in the .rubocop.yml. +# - for short (less than 15 lines line) temporary lists, place them in the .rubocop_todo.yml file (run auto-generation). +# +# Purpose of this file: +# - show long exclusion lists that are hidden in generic auto-generation of .rubocop_todo.yml. +# - provide an 'in-flight' list of exclusions being worked on when qualifying as a long list +# as mentioned above. +# - guidelines for use found in +# https://docs.gitlab.com/ee/development/contributing/style_guides.html#resolving-rubocop-exceptions. + FactoryBot/InlineAssociation: Exclude: - 'ee/spec/factories/analytics/cycle_analytics/group_stages.rb' @@ -17,25 +29,6 @@ FactoryBot/InlineAssociation: - 'spec/factories/uploads.rb' - 'spec/factories/wiki_pages.rb' -Graphql/IDType: - Exclude: - - 'ee/app/graphql/ee/mutations/issues/update.rb' - - 'app/graphql/mutations/boards/issues/issue_move_list.rb' - - 'app/graphql/resolvers/design_management/design_at_version_resolver.rb' - - 'app/graphql/resolvers/design_management/design_resolver.rb' - - 'app/graphql/resolvers/design_management/designs_resolver.rb' - - 'app/graphql/resolvers/design_management/version/design_at_version_resolver.rb' - - 'app/graphql/resolvers/design_management/version_in_collection_resolver.rb' - - 'app/graphql/resolvers/design_management/version_resolver.rb' - - 'app/graphql/resolvers/design_management/versions_resolver.rb' - - 'app/graphql/resolvers/error_tracking/sentry_detailed_error_resolver.rb' - - 'app/graphql/resolvers/error_tracking/sentry_error_stack_trace_resolver.rb' - - 'app/graphql/resolvers/user_merge_requests_resolver.rb' - -Gitlab/PolicyRuleBoolean: - Exclude: - - 'ee/app/policies/ee/identity_provider_policy.rb' - Rails/SaveBang: Exclude: - 'ee/spec/controllers/projects/merge_requests_controller_spec.rb' diff --git a/.rubocop_todo.yml b/.rubocop_todo.yml index 293c8fccf3315..6bf120b8fb875 100644 --- a/.rubocop_todo.yml +++ b/.rubocop_todo.yml @@ -1,49 +1,68 @@ # This configuration was generated by # `rubocop --auto-gen-config` -# on 2020-10-29 17:35:37 UTC using RuboCop version 0.89.1. +# on 2020-12-14 18:35:57 UTC using RuboCop version 0.89.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: 310 +# Offense count: 313 # Cop supports --auto-correct. Capybara/CurrentPathExpectation: Enabled: false -# Offense count: 210 +# Offense count: 216 Capybara/VisibilityMatcher: Enabled: false -# Offense count: 1903 +# Offense count: 1 +Gitlab/PolicyRuleBoolean: + Exclude: + - 'ee/app/policies/ee/identity_provider_policy.rb' + +# Offense count: 5 +Graphql/IDType: + Exclude: + - 'app/graphql/mutations/boards/issues/issue_move_list.rb' + +# Offense count: 5 +Graphql/ResolverType: + Exclude: + - 'app/graphql/resolvers/base_resolver.rb' + - 'app/graphql/resolvers/ci/jobs_resolver.rb' + - 'app/graphql/resolvers/ci/pipeline_stages_resolver.rb' + - 'app/graphql/resolvers/users/group_count_resolver.rb' + - 'ee/app/graphql/resolvers/vulnerabilities_base_resolver.rb' + +# Offense count: 2002 # Cop supports --auto-correct. # Configuration parameters: EnforcedStyle, IndentationWidth. # SupportedStyles: with_first_argument, with_fixed_indentation Layout/ArgumentAlignment: Enabled: false -# Offense count: 49 +# Offense count: 50 # Cop supports --auto-correct. # Configuration parameters: AllowAliasSyntax, AllowedMethods. # AllowedMethods: alias_method, public, protected, private Layout/EmptyLinesAroundAttributeAccessor: Enabled: false -# Offense count: 610 +# Offense count: 694 # Cop supports --auto-correct. # Configuration parameters: EnforcedStyle, IndentationWidth. # SupportedStyles: special_inside_parentheses, consistent, align_brackets Layout/FirstArrayElementIndentation: Enabled: false -# Offense count: 1535 +# Offense count: 1621 # Cop supports --auto-correct. # Configuration parameters: EnforcedStyle, IndentationWidth. # SupportedStyles: special_inside_parentheses, consistent, align_braces Layout/FirstHashElementIndentation: Enabled: false -# Offense count: 2754 +# Offense count: 2979 # Cop supports --auto-correct. # Configuration parameters: AllowMultipleStyles, EnforcedHashRocketStyle, EnforcedColonStyle, EnforcedLastArgumentHashStyle. # SupportedHashRocketStyles: key, separator, table @@ -52,14 +71,14 @@ Layout/FirstHashElementIndentation: Layout/HashAlignment: Enabled: false -# Offense count: 64 +# Offense count: 72 # Cop supports --auto-correct. # Configuration parameters: AutoCorrect, AllowHeredoc, AllowURI, URISchemes, IgnoreCopDirectives, IgnoredPatterns. # URISchemes: http, https Layout/LineLength: Max: 1313 -# Offense count: 118 +# Offense count: 162 # Cop supports --auto-correct. # Configuration parameters: EnforcedStyle, IndentationWidth. # SupportedStyles: aligned, indented @@ -71,14 +90,14 @@ Layout/MultilineOperationIndentation: Layout/SpaceAroundMethodCallOperator: Enabled: false -# Offense count: 790 +# Offense count: 737 # Cop supports --auto-correct. # Configuration parameters: EnforcedStyle. # SupportedStyles: require_no_space, require_space Layout/SpaceInLambdaLiteral: Enabled: false -# Offense count: 1110 +# Offense count: 1160 # Cop supports --auto-correct. # Configuration parameters: EnforcedStyle, EnforcedStyleForEmptyBraces, SpaceBeforeBlockParameters. # SupportedStyles: space, no_space @@ -86,14 +105,14 @@ Layout/SpaceInLambdaLiteral: Layout/SpaceInsideBlockBraces: Enabled: false -# Offense count: 501 +# Offense count: 539 # Cop supports --auto-correct. # Configuration parameters: EnforcedStyle. # SupportedStyles: space, no_space Layout/SpaceInsideParens: Enabled: false -# Offense count: 18 +# Offense count: 19 Lint/BinaryOperatorWithIdenticalOperands: Exclude: - 'app/finders/concerns/time_frame_filter.rb' @@ -118,19 +137,13 @@ Lint/DeprecatedOpenSSLConstant: - 'ee/lib/gitlab/geo/oauth/logout_state.rb' - 'lib/gitlab/conan_token.rb' - 'lib/gitlab/gitaly_client.rb' - - 'lib/gitlab/kubernetes/helm/certificate.rb' - 'lib/gitlab/kubernetes/helm/v2/certificate.rb' - 'spec/lib/gitlab/conan_token_spec.rb' - 'spec/services/pages_domains/obtain_lets_encrypt_certificate_service_spec.rb' - 'spec/support/helpers/smime_helper.rb' - 'spec/support/shared_contexts/requests/api/conan_packages_shared_context.rb' -# Offense count: 1 -Lint/FloatComparison: - Exclude: - - 'ee/app/models/ee/namespace.rb' - -# Offense count: 157 +# Offense count: 183 # Configuration parameters: MaximumRangeSize. Lint/MissingCopEnableDirective: Enabled: false @@ -159,7 +172,7 @@ Lint/NonDeterministicRequireOrder: Exclude: - 'rubocop/rubocop.rb' -# Offense count: 118 +# Offense count: 135 # Cop supports --auto-correct. Lint/RedundantCopDisableDirective: Enabled: false @@ -187,17 +200,17 @@ Lint/UnreachableLoop: Exclude: - 'qa/qa/runtime/feature.rb' -# Offense count: 5 +# Offense count: 7 # Configuration parameters: IgnoredMethods. Metrics/AbcSize: Max: 59 -# Offense count: 13 +# Offense count: 15 # Configuration parameters: IgnoredMethods. Metrics/CyclomaticComplexity: Max: 25 -# Offense count: 13 +# Offense count: 14 # Configuration parameters: IgnoredMethods. Metrics/PerceivedComplexity: Max: 25 @@ -208,7 +221,7 @@ Migration/DepartmentName: Exclude: - 'app/models/commit.rb' -# Offense count: 171 +# Offense count: 181 # 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: @@ -225,7 +238,7 @@ Naming/HeredocDelimiterCase: - 'spec/support/helpers/repo_helpers.rb' - 'spec/support/helpers/seed_repo.rb' -# Offense count: 263 +# Offense count: 303 # Configuration parameters: ForbiddenDelimiters. # ForbiddenDelimiters: (?-mix:(^|\s)(EO[A-Z]{1}|END)(\s|$)) Naming/HeredocDelimiterNaming: @@ -239,7 +252,7 @@ Naming/MethodParameterName: - 'lib/gitlab/diff/inline_diff.rb' - 'spec/support/helpers/key_generator_helper.rb' -# Offense count: 191 +# Offense count: 203 # Cop supports --auto-correct. # Configuration parameters: PreferredName. Naming/RescuedExceptionsVariableName: @@ -256,7 +269,7 @@ Performance/AncestorsInclude: Performance/CollectionLiteralInLoop: Enabled: false -# Offense count: 19 +# Offense count: 18 # Cop supports --auto-correct. Performance/Count: Exclude: @@ -267,10 +280,9 @@ Performance/Count: - 'spec/lib/gitlab/conflict/file_spec.rb' - 'spec/lib/gitlab/git/tree_spec.rb' - 'spec/models/ci/build_spec.rb' - - 'spec/support/matchers/exceed_query_limit.rb' - 'spec/support_specs/helpers/active_record/query_recorder_spec.rb' -# Offense count: 15 +# Offense count: 14 # Cop supports --auto-correct. # Configuration parameters: SafeMultiline. Performance/DeletePrefix: @@ -283,14 +295,13 @@ Performance/DeletePrefix: - 'lib/gitlab/gfm/uploads_rewriter.rb' - 'lib/gitlab/git/ref.rb' - 'lib/gitlab/project_template.rb' - - 'lib/gitlab/repo_path.rb' - 'lib/gitlab/setup_helper.rb' - 'lib/gitlab/time_tracking_formatter.rb' - 'spec/controllers/projects/artifacts_controller_spec.rb' - 'spec/lib/gitlab/gfm/uploads_rewriter_spec.rb' - 'spec/support/helpers/test_env.rb' -# Offense count: 6 +# Offense count: 5 # Cop supports --auto-correct. # Configuration parameters: SafeMultiline. Performance/DeleteSuffix: @@ -299,7 +310,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/gitlab/repo_path.rb' - 'lib/sentry/client/issue.rb' # Offense count: 13 @@ -313,7 +323,7 @@ Performance/Detect: - 'spec/lib/gitlab/import_export/project/tree_restorer_spec.rb' - 'spec/models/event_spec.rb' -# Offense count: 19 +# Offense count: 18 # Cop supports --auto-correct. # Configuration parameters: AutoCorrect. Performance/StringInclude: @@ -330,7 +340,6 @@ Performance/StringInclude: - 'qa/qa/ee/page/merge_request/show.rb' - 'qa/qa/specs/runner.rb' - 'spec/features/projects/jobs_spec.rb' - - 'spec/lib/gitlab/import_export/project/tree_restorer_spec.rb' - 'spec/spec_helper.rb' - 'spec/support_specs/helpers/active_record/query_recorder_spec.rb' @@ -346,7 +355,7 @@ Performance/Sum: - 'lib/gitlab/sherlock/transaction.rb' - 'lib/gitlab/usage_data.rb' -# Offense count: 13879 +# Offense count: 14469 # Configuration parameters: Prefixes. # Prefixes: when, with, without RSpec/ContextWording: @@ -359,65 +368,50 @@ RSpec/EmptyExampleGroup: - 'ee/spec/services/personal_access_tokens/revoke_invalid_tokens_spec.rb' - 'spec/services/projects/prometheus/alerts/notify_service_spec.rb' -# Offense count: 1310 +# Offense count: 1355 # Cop supports --auto-correct. # Configuration parameters: EnforcedStyle. # SupportedStyles: method_call, block RSpec/ExpectChange: Enabled: false -RSpec/ExpectGitlabTracking: - Exclude: - - 'ee/spec/requests/api/visual_review_discussions_spec.rb' - - 'ee/spec/services/epics/issue_promote_service_spec.rb' - - 'spec/controllers/groups/registry/repositories_controller_spec.rb' - - 'spec/controllers/projects/registry/repositories_controller_spec.rb' - - 'spec/controllers/projects/registry/tags_controller_spec.rb' - - 'spec/controllers/projects/settings/operations_controller_spec.rb' - - 'spec/lib/api/helpers_spec.rb' - - 'spec/requests/api/project_container_repositories_spec.rb' - - 'spec/support/shared_examples/controllers/trackable_shared_examples.rb' - - 'spec/support/shared_examples/requests/api/discussions_shared_examples.rb' - - 'spec/support/shared_examples/requests/api/packages_shared_examples.rb' - - 'spec/support/shared_examples/requests/api/tracking_shared_examples.rb' - -# Offense count: 888 +# Offense count: 903 RSpec/ExpectInHook: Enabled: false -# Offense count: 15945 +# Offense count: 16472 # Configuration parameters: AllowSubject. RSpec/MultipleMemoizedHelpers: Max: 40 -# Offense count: 2297 +# Offense count: 2344 # Cop supports --auto-correct. # Configuration parameters: Strict, EnforcedStyle, AllowedExplicitMatchers. # SupportedStyles: inflected, explicit RSpec/PredicateMatcher: Enabled: false -# Offense count: 98 +# Offense count: 112 RSpec/RepeatedExampleGroupBody: Enabled: false -# Offense count: 213 +# Offense count: 219 RSpec/RepeatedExampleGroupDescription: Enabled: false -# Offense count: 636 +# Offense count: 655 # Cop supports --auto-correct. # Configuration parameters: EnforcedStyle. # SupportedStyles: and_return, block RSpec/ReturnFromStub: Enabled: false -# Offense count: 574 +# Offense count: 587 # Cop supports --auto-correct. RSpec/ScatteredLet: Enabled: false -# Offense count: 4 +# Offense count: 6 RSpec/ScatteredSetup: Exclude: - 'spec/requests/api/jobs_spec.rb' @@ -429,7 +423,7 @@ RSpec/VariableDefinition: Exclude: - 'spec/initializers/mail_encoding_patch_spec.rb' -# Offense count: 25 +# Offense count: 24 # Configuration parameters: EnforcedStyle, IgnoredPatterns. # SupportedStyles: snake_case, camelCase RSpec/VariableName: @@ -437,13 +431,12 @@ RSpec/VariableName: - 'spec/features/projects/import_export/import_file_spec.rb' - 'spec/features/task_lists_spec.rb' - 'spec/initializers/mail_encoding_patch_spec.rb' - - 'spec/lib/gitlab/experimentation_spec.rb' - 'spec/models/board_spec.rb' - 'spec/support/shared_contexts/url_shared_context.rb' - '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: 27 +# Offense count: 25 # Cop supports --auto-correct. # Configuration parameters: Include. # Include: app/models/**/*.rb @@ -472,25 +465,25 @@ Rails/BelongsTo: - 'app/models/deployment.rb' - 'app/models/environment.rb' -# Offense count: 84 +# Offense count: 88 # Configuration parameters: Database, Include. # SupportedDatabases: mysql, postgresql # Include: db/migrate/*.rb Rails/BulkChangeTable: Enabled: false -# Offense count: 151 +# Offense count: 153 # Cop supports --auto-correct. Rails/ContentTag: Enabled: false -# Offense count: 270 +# Offense count: 292 # Configuration parameters: Include. # Include: db/migrate/*.rb Rails/CreateTableWithTimestamps: Enabled: false -# Offense count: 323 +# Offense count: 343 # Configuration parameters: EnforcedStyle. # SupportedStyles: slashes, arguments Rails/FilePath: @@ -512,13 +505,13 @@ Rails/FindById: - 'spec/finders/concerns/finder_methods_spec.rb' - 'spec/finders/concerns/finder_with_cross_project_access_spec.rb' -# Offense count: 329 +# Offense count: 343 # Configuration parameters: Include. # Include: app/models/**/*.rb Rails/HasManyOrHasOneDependent: Enabled: false -# Offense count: 507 +# Offense count: 538 # Configuration parameters: Include. # Include: app/helpers/**/*.rb Rails/HelperInstanceVariable: @@ -542,7 +535,7 @@ Rails/IndexBy: - 'lib/gitlab/database/count/reltuples_count_strategy.rb' - 'lib/gitlab/language_detection.rb' -# Offense count: 44 +# Offense count: 45 # Cop supports --auto-correct. Rails/IndexWith: Enabled: false @@ -552,7 +545,7 @@ Rails/Inquiry: Exclude: - 'spec/helpers/labels_helper_spec.rb' -# Offense count: 112 +# Offense count: 115 # Configuration parameters: Include. # Include: app/models/**/*.rb Rails/InverseOf: @@ -580,33 +573,34 @@ Rails/MailerName: Exclude: - 'app/mailers/notify.rb' -# Offense count: 48 +# Offense count: 49 # Cop supports --auto-correct. Rails/NegateInclude: Enabled: false -# Offense count: 41 +# Offense count: 42 # Cop supports --auto-correct. Rails/Pick: Enabled: false -# Offense count: 113 +# Offense count: 119 # Cop supports --auto-correct. Rails/Pluck: Enabled: false -# Offense count: 38 +# Offense count: 39 # Cop supports --auto-correct. # Configuration parameters: Include. # Include: **/Rakefile, **/*.rake Rails/RakeEnvironment: Enabled: false -# Offense count: 52 +# Offense count: 56 # Cop supports --auto-correct. Rails/RedundantForeignKey: Enabled: false +# Offense count: 1 Rails/RenderInline: Exclude: - 'ee/app/controllers/sitemap_controller.rb' @@ -621,18 +615,18 @@ Rails/ShortI18n: - 'app/uploaders/content_type_whitelist.rb' - 'spec/views/shared/runners/show.html.haml_spec.rb' -# Offense count: 1043 +# Offense count: 1064 # 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: 202 +# Offense count: 245 # Cop supports --auto-correct. Rails/SquishedSQLHeredocs: Enabled: false -# Offense count: 37 +# Offense count: 38 # Cop supports --auto-correct. # Configuration parameters: EnforcedStyle. # SupportedStyles: exists, where @@ -654,7 +648,7 @@ Security/YAMLLoad: - 'spec/initializers/secret_token_spec.rb' - 'spec/lib/gitlab/prometheus/additional_metrics_parser_spec.rb' -# Offense count: 240 +# Offense count: 247 # Cop supports --auto-correct. # Configuration parameters: EnforcedStyle. # SupportedStyles: separated, grouped @@ -679,7 +673,7 @@ Style/ArrayCoercion: - 'ee/lib/ee/banzai/pipeline/gfm_pipeline.rb' - 'spec/support/helpers/lfs_http_helpers.rb' -# Offense count: 176 +# Offense count: 184 # Cop supports --auto-correct. # Configuration parameters: EnforcedStyle. # SupportedStyles: percent_q, bare_percent @@ -693,7 +687,7 @@ Style/BisectedAttrAccessor: - 'lib/system_check/base_check.rb' - 'qa/qa/resource/api_fabricator.rb' -# Offense count: 37 +# Offense count: 36 # Cop supports --auto-correct. Style/CaseLikeIf: Enabled: false @@ -716,14 +710,14 @@ Style/EachWithObject: - 'lib/gitlab/i18n/po_linter.rb' - 'lib/gitlab/import_export/members_mapper.rb' -# Offense count: 53 +# Offense count: 54 # Cop supports --auto-correct. # Configuration parameters: EnforcedStyle. # SupportedStyles: empty, nil, both Style/EmptyElse: Enabled: false -# Offense count: 193 +# Offense count: 198 # Cop supports --auto-correct. # Configuration parameters: EnforcedStyle. # SupportedStyles: compact, expanded @@ -737,51 +731,51 @@ Style/ExpandPathArguments: - 'cable/config.ru' - 'config.ru' -# Offense count: 118 +# Offense count: 114 # Cop supports --auto-correct. Style/ExplicitBlockArgument: Enabled: false -# Offense count: 521 +# Offense count: 542 # Cop supports --auto-correct. # Configuration parameters: EnforcedStyle. # SupportedStyles: format, sprintf, percent Style/FormatString: Enabled: false -# Offense count: 48 +# Offense count: 59 # Cop supports --auto-correct. Style/GlobalStdStream: Enabled: false -# Offense count: 835 +# Offense count: 857 # Configuration parameters: MinBodyLength. Style/GuardClause: Enabled: false -# Offense count: 93 +# Offense count: 92 # Cop supports --auto-correct. # Configuration parameters: EnforcedStyle. # SupportedStyles: braces, no_braces Style/HashAsLastArrayItem: Enabled: false -# Offense count: 55 +# Offense count: 60 # Cop supports --auto-correct. Style/HashEachMethods: Enabled: false -# Offense count: 30 +# Offense count: 34 # Configuration parameters: AllowIfModifier. Style/IfInsideElse: Enabled: false -# Offense count: 1798 +# Offense count: 1856 # Cop supports --auto-correct. Style/IfUnlessModifier: Enabled: false -# Offense count: 447 +# Offense count: 439 # Cop supports --auto-correct. # Configuration parameters: EnforcedStyle. # SupportedStyles: line_count_dependent, lambda, literal @@ -792,39 +786,39 @@ Style/Lambda: Style/MissingRespondToMissing: Enabled: false -# Offense count: 32 +# Offense count: 36 # Cop supports --auto-correct. # Configuration parameters: EnforcedStyle, MinBodyLength. # SupportedStyles: skip_modifier_ifs, always Style/Next: Enabled: false -# Offense count: 95 +# Offense count: 98 # Cop supports --auto-correct. # Configuration parameters: EnforcedOctalStyle. # SupportedOctalStyles: zero_with_o, zero_only Style/NumericLiteralPrefix: Enabled: false -# Offense count: 132 +# Offense count: 136 # Cop supports --auto-correct. Style/ParallelAssignment: Enabled: false -# Offense count: 2473 +# Offense count: 2557 # Cop supports --auto-correct. # Configuration parameters: PreferredDelimiters. Style/PercentLiteralDelimiters: Enabled: false -# Offense count: 246 +# Offense count: 251 # Cop supports --auto-correct. # Configuration parameters: EnforcedStyle. # SupportedStyles: compact, exploded Style/RaiseArgs: Enabled: false -# Offense count: 66 +# Offense count: 68 # Cop supports --auto-correct. Style/RedundantAssignment: Enabled: false @@ -846,22 +840,21 @@ Style/RedundantFetchBlock: Style/RedundantFileExtensionInRequire: Enabled: false -# Offense count: 260 +# Offense count: 272 # Cop supports --auto-correct. Style/RedundantFreeze: Enabled: false -# Offense count: 167 +# Offense count: 179 # Cop supports --auto-correct. Style/RedundantInterpolation: Enabled: false -# Offense count: 11 +# Offense count: 9 # Cop supports --auto-correct. Style/RedundantRegexpCharacterClass: Exclude: - 'app/models/concerns/taskable.rb' - - 'ee/lib/ee/gitlab/path_regex.rb' - 'lib/gitlab/authorized_keys.rb' - 'lib/gitlab/fogbugz_import/repository.rb' - 'lib/gitlab/prometheus/internal.rb' @@ -870,17 +863,17 @@ Style/RedundantRegexpCharacterClass: - 'spec/features/merge_request/user_views_open_merge_request_spec.rb' - 'spec/tasks/gitlab/usage_data_rake_spec.rb' -# Offense count: 250 +# Offense count: 270 # Cop supports --auto-correct. Style/RedundantRegexpEscape: Enabled: false -# Offense count: 868 +# Offense count: 914 # Cop supports --auto-correct. Style/RedundantSelf: Enabled: false -# Offense count: 179 +# Offense count: 190 # Cop supports --auto-correct. # Configuration parameters: EnforcedStyle, AllowInnerSlashes. # SupportedStyles: slashes, percent_r, mixed @@ -892,43 +885,43 @@ Style/RegexpLiteral: Style/RescueModifier: Enabled: false -# Offense count: 329 +# Offense count: 338 # Cop supports --auto-correct. # Configuration parameters: EnforcedStyle. # SupportedStyles: implicit, explicit Style/RescueStandardError: Enabled: false -# Offense count: 104 +# Offense count: 110 # Cop supports --auto-correct. Style/SingleArgumentDig: Enabled: false -# Offense count: 47 +# Offense count: 46 # Cop supports --auto-correct. Style/SlicingWithRange: Enabled: false -# Offense count: 109 +# Offense count: 120 # Cop supports --auto-correct. # Configuration parameters: . # SupportedStyles: use_perl_names, use_english_names Style/SpecialGlobalVars: EnforcedStyle: use_perl_names -# Offense count: 516 +# Offense count: 532 # Cop supports --auto-correct. Style/StringConcatenation: Enabled: false -# Offense count: 103 +# Offense count: 104 # Cop supports --auto-correct. # Configuration parameters: EnforcedStyle. # SupportedStyles: single_quotes, double_quotes Style/StringLiteralsInInterpolation: Enabled: false -# Offense count: 276 +# Offense count: 296 # Cop supports --auto-correct. # Configuration parameters: IgnoredMethods. # IgnoredMethods: respond_to, define_method -- GitLab