34c083a184
Enable Rubocop cops that check access modifiers ## What does this MR do? This MR enables Rubocop cops that detect methods that should be restricted but are the part of public API because of access modifiers used improperly. This also fixes existing offenses. ## Why was this MR needed? Some method in our codebase are public instead of being private because it is sometimes difficult to get it right without static analysis. ## What are the relevant issue numbers? See #17478 Closes #17372 See merge request !5014
450 lines
11 KiB
YAML
450 lines
11 KiB
YAML
# This configuration was generated by
|
|
# `rubocop --auto-gen-config --exclude-limit 0`
|
|
# on 2016-07-13 12:36:08 -0600 using RuboCop version 0.41.2.
|
|
# 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: 154
|
|
Lint/AmbiguousRegexpLiteral:
|
|
Enabled: false
|
|
|
|
# Offense count: 43
|
|
# Configuration parameters: AllowSafeAssignment.
|
|
Lint/AssignmentInCondition:
|
|
Enabled: false
|
|
|
|
# Offense count: 14
|
|
Lint/HandleExceptions:
|
|
Enabled: false
|
|
|
|
# Offense count: 2
|
|
Lint/Loop:
|
|
Enabled: false
|
|
|
|
# Offense count: 15
|
|
Lint/ShadowingOuterLocalVariable:
|
|
Enabled: false
|
|
|
|
# Offense count: 3
|
|
# Cop supports --auto-correct.
|
|
Lint/StringConversionInInterpolation:
|
|
Enabled: false
|
|
|
|
# Offense count: 44
|
|
# Cop supports --auto-correct.
|
|
# Configuration parameters: IgnoreEmptyBlocks, AllowUnusedKeywordArguments.
|
|
Lint/UnusedBlockArgument:
|
|
Enabled: false
|
|
|
|
# Offense count: 129
|
|
# Cop supports --auto-correct.
|
|
# Configuration parameters: AllowUnusedKeywordArguments, IgnoreEmptyMethods.
|
|
Lint/UnusedMethodArgument:
|
|
Enabled: false
|
|
|
|
# Offense count: 12
|
|
# Cop supports --auto-correct.
|
|
Performance/PushSplat:
|
|
Enabled: false
|
|
|
|
# Offense count: 2
|
|
# Cop supports --auto-correct.
|
|
Performance/RedundantBlockCall:
|
|
Enabled: false
|
|
|
|
# Offense count: 4
|
|
# Cop supports --auto-correct.
|
|
Performance/RedundantMatch:
|
|
Enabled: false
|
|
|
|
# Offense count: 24
|
|
# Cop supports --auto-correct.
|
|
# Configuration parameters: MaxKeyValuePairs.
|
|
Performance/RedundantMerge:
|
|
Enabled: false
|
|
|
|
# Offense count: 60
|
|
Rails/OutputSafety:
|
|
Enabled: false
|
|
|
|
# Offense count: 128
|
|
# Configuration parameters: EnforcedStyle, SupportedStyles.
|
|
# SupportedStyles: strict, flexible
|
|
Rails/TimeZone:
|
|
Enabled: false
|
|
|
|
# Offense count: 12
|
|
# Cop supports --auto-correct.
|
|
# Configuration parameters: Include.
|
|
# Include: app/models/**/*.rb
|
|
Rails/Validation:
|
|
Enabled: false
|
|
|
|
# Offense count: 217
|
|
# Cop supports --auto-correct.
|
|
# Configuration parameters: EnforcedStyle, SupportedStyles, IndentationWidth.
|
|
# SupportedStyles: with_first_parameter, with_fixed_indentation
|
|
Style/AlignParameters:
|
|
Enabled: false
|
|
|
|
# Offense count: 32
|
|
# Cop supports --auto-correct.
|
|
# Configuration parameters: EnforcedStyle, SupportedStyles.
|
|
# SupportedStyles: always, conditionals
|
|
Style/AndOr:
|
|
Enabled: false
|
|
|
|
# Offense count: 47
|
|
# Cop supports --auto-correct.
|
|
# Configuration parameters: EnforcedStyle, SupportedStyles.
|
|
# SupportedStyles: percent_q, bare_percent
|
|
Style/BarePercentLiterals:
|
|
Enabled: false
|
|
|
|
# Offense count: 258
|
|
# Cop supports --auto-correct.
|
|
# Configuration parameters: EnforcedStyle, SupportedStyles.
|
|
# SupportedStyles: braces, no_braces, context_dependent
|
|
Style/BracesAroundHashParameters:
|
|
Enabled: false
|
|
|
|
# Offense count: 5
|
|
Style/CaseEquality:
|
|
Enabled: false
|
|
|
|
# Offense count: 19
|
|
# Cop supports --auto-correct.
|
|
Style/ColonMethodCall:
|
|
Enabled: false
|
|
|
|
# Offense count: 3
|
|
# Cop supports --auto-correct.
|
|
# Configuration parameters: Keywords.
|
|
# Keywords: TODO, FIXME, OPTIMIZE, HACK, REVIEW
|
|
Style/CommentAnnotation:
|
|
Enabled: false
|
|
|
|
# Offense count: 34
|
|
# Cop supports --auto-correct.
|
|
# Configuration parameters: EnforcedStyle, SupportedStyles, SingleLineConditionsOnly.
|
|
# SupportedStyles: assign_to_condition, assign_inside_condition
|
|
Style/ConditionalAssignment:
|
|
Enabled: false
|
|
|
|
# Offense count: 789
|
|
# Cop supports --auto-correct.
|
|
# Configuration parameters: EnforcedStyle, SupportedStyles.
|
|
# SupportedStyles: leading, trailing
|
|
Style/DotPosition:
|
|
Enabled: false
|
|
|
|
# Offense count: 13
|
|
Style/DoubleNegation:
|
|
Enabled: false
|
|
|
|
# Offense count: 3
|
|
Style/EachWithObject:
|
|
Enabled: false
|
|
|
|
# Offense count: 30
|
|
# Cop supports --auto-correct.
|
|
# Configuration parameters: EnforcedStyle, SupportedStyles.
|
|
# SupportedStyles: empty, nil, both
|
|
Style/EmptyElse:
|
|
Enabled: false
|
|
|
|
# Offense count: 3
|
|
# Cop supports --auto-correct.
|
|
Style/EmptyLiteral:
|
|
Enabled: false
|
|
|
|
# Offense count: 123
|
|
# Cop supports --auto-correct.
|
|
# Configuration parameters: AllowForAlignment, ForceEqualSignAlignment.
|
|
Style/ExtraSpacing:
|
|
Enabled: false
|
|
|
|
# Offense count: 7
|
|
# Configuration parameters: EnforcedStyle, SupportedStyles.
|
|
# SupportedStyles: format, sprintf, percent
|
|
Style/FormatString:
|
|
Enabled: false
|
|
|
|
# Offense count: 48
|
|
# Configuration parameters: MinBodyLength.
|
|
Style/GuardClause:
|
|
Enabled: false
|
|
|
|
# Offense count: 11
|
|
Style/IfInsideElse:
|
|
Enabled: false
|
|
|
|
# Offense count: 177
|
|
# Cop supports --auto-correct.
|
|
# Configuration parameters: MaxLineLength.
|
|
Style/IfUnlessModifier:
|
|
Enabled: false
|
|
|
|
# Offense count: 52
|
|
# Cop supports --auto-correct.
|
|
# Configuration parameters: EnforcedStyle, SupportedStyles, IndentationWidth.
|
|
# SupportedStyles: special_inside_parentheses, consistent, align_brackets
|
|
Style/IndentArray:
|
|
Enabled: false
|
|
|
|
# Offense count: 89
|
|
# Cop supports --auto-correct.
|
|
# Configuration parameters: EnforcedStyle, SupportedStyles, IndentationWidth.
|
|
# SupportedStyles: special_inside_parentheses, consistent, align_braces
|
|
Style/IndentHash:
|
|
Enabled: false
|
|
|
|
# Offense count: 12
|
|
# Cop supports --auto-correct.
|
|
# Configuration parameters: EnforcedStyle, SupportedStyles.
|
|
# SupportedStyles: line_count_dependent, lambda, literal
|
|
Style/Lambda:
|
|
Enabled: false
|
|
|
|
# Offense count: 6
|
|
# Cop supports --auto-correct.
|
|
Style/LineEndConcatenation:
|
|
Enabled: false
|
|
|
|
# Offense count: 13
|
|
# Cop supports --auto-correct.
|
|
Style/MethodCallParentheses:
|
|
Enabled: false
|
|
|
|
# Offense count: 62
|
|
# Cop supports --auto-correct.
|
|
Style/MutableConstant:
|
|
Enabled: false
|
|
|
|
# Offense count: 10
|
|
# Cop supports --auto-correct.
|
|
Style/NestedParenthesizedCalls:
|
|
Enabled: false
|
|
|
|
# Offense count: 12
|
|
# Cop supports --auto-correct.
|
|
# Configuration parameters: EnforcedStyle, MinBodyLength, SupportedStyles.
|
|
# SupportedStyles: skip_modifier_ifs, always
|
|
Style/Next:
|
|
Enabled: false
|
|
|
|
# Offense count: 8
|
|
# Cop supports --auto-correct.
|
|
# Configuration parameters: EnforcedOctalStyle, SupportedOctalStyles.
|
|
# SupportedOctalStyles: zero_with_o, zero_only
|
|
Style/NumericLiteralPrefix:
|
|
Enabled: false
|
|
|
|
# Offense count: 29
|
|
# Cop supports --auto-correct.
|
|
Style/ParallelAssignment:
|
|
Enabled: false
|
|
|
|
# Offense count: 208
|
|
# Cop supports --auto-correct.
|
|
# Configuration parameters: PreferredDelimiters.
|
|
Style/PercentLiteralDelimiters:
|
|
Enabled: false
|
|
|
|
# Offense count: 11
|
|
# Cop supports --auto-correct.
|
|
# Configuration parameters: EnforcedStyle, SupportedStyles.
|
|
# SupportedStyles: lower_case_q, upper_case_q
|
|
Style/PercentQLiterals:
|
|
Enabled: false
|
|
|
|
# Offense count: 13
|
|
# Cop supports --auto-correct.
|
|
Style/PerlBackrefs:
|
|
Enabled: false
|
|
|
|
# Offense count: 32
|
|
# Configuration parameters: NamePrefix, NamePrefixBlacklist, NameWhitelist.
|
|
# NamePrefix: is_, has_, have_
|
|
# NamePrefixBlacklist: is_, has_, have_
|
|
# NameWhitelist: is_a?
|
|
Style/PredicateName:
|
|
Enabled: false
|
|
|
|
# Offense count: 28
|
|
# Cop supports --auto-correct.
|
|
Style/PreferredHashMethods:
|
|
Enabled: false
|
|
|
|
# Offense count: 6
|
|
# Cop supports --auto-correct.
|
|
Style/Proc:
|
|
Enabled: false
|
|
|
|
# Offense count: 20
|
|
# Cop supports --auto-correct.
|
|
# Configuration parameters: EnforcedStyle, SupportedStyles.
|
|
# SupportedStyles: compact, exploded
|
|
Style/RaiseArgs:
|
|
Enabled: false
|
|
|
|
# Offense count: 3
|
|
# Cop supports --auto-correct.
|
|
Style/RedundantBegin:
|
|
Enabled: false
|
|
|
|
# Offense count: 1
|
|
# Cop supports --auto-correct.
|
|
Style/RedundantException:
|
|
Enabled: false
|
|
|
|
# Offense count: 23
|
|
# Cop supports --auto-correct.
|
|
Style/RedundantFreeze:
|
|
Enabled: false
|
|
|
|
# Offense count: 377
|
|
# Cop supports --auto-correct.
|
|
Style/RedundantSelf:
|
|
Enabled: false
|
|
|
|
# Offense count: 94
|
|
# Cop supports --auto-correct.
|
|
# Configuration parameters: EnforcedStyle, SupportedStyles, AllowInnerSlashes.
|
|
# SupportedStyles: slashes, percent_r, mixed
|
|
Style/RegexpLiteral:
|
|
Enabled: false
|
|
|
|
# Offense count: 17
|
|
# Cop supports --auto-correct.
|
|
Style/RescueModifier:
|
|
Enabled: false
|
|
|
|
# Offense count: 2
|
|
# Cop supports --auto-correct.
|
|
Style/SelfAssignment:
|
|
Enabled: false
|
|
|
|
# Offense count: 2
|
|
# Configuration parameters: Methods.
|
|
# Methods: {"reduce"=>["a", "e"]}, {"inject"=>["a", "e"]}
|
|
Style/SingleLineBlockParams:
|
|
Enabled: false
|
|
|
|
# Offense count: 50
|
|
# Cop supports --auto-correct.
|
|
# Configuration parameters: AllowIfMethodIsEmpty.
|
|
Style/SingleLineMethods:
|
|
Enabled: false
|
|
|
|
# Offense count: 14
|
|
# Cop supports --auto-correct.
|
|
# Configuration parameters: EnforcedStyle, SupportedStyles.
|
|
# SupportedStyles: space, no_space
|
|
Style/SpaceAroundEqualsInParameterDefault:
|
|
Enabled: false
|
|
|
|
# Offense count: 119
|
|
# Cop supports --auto-correct.
|
|
# Configuration parameters: EnforcedStyle, SupportedStyles.
|
|
# SupportedStyles: space, no_space
|
|
Style/SpaceBeforeBlockBraces:
|
|
Enabled: false
|
|
|
|
# Offense count: 11
|
|
# Cop supports --auto-correct.
|
|
# Configuration parameters: AllowForAlignment.
|
|
Style/SpaceBeforeFirstArg:
|
|
Enabled: false
|
|
|
|
# Offense count: 130
|
|
# Cop supports --auto-correct.
|
|
# Configuration parameters: EnforcedStyle, SupportedStyles, EnforcedStyleForEmptyBraces, SpaceBeforeBlockParameters.
|
|
# SupportedStyles: space, no_space
|
|
Style/SpaceInsideBlockBraces:
|
|
Enabled: false
|
|
|
|
# Offense count: 98
|
|
# Cop supports --auto-correct.
|
|
Style/SpaceInsideBrackets:
|
|
Enabled: false
|
|
|
|
# Offense count: 60
|
|
# Cop supports --auto-correct.
|
|
Style/SpaceInsideParens:
|
|
Enabled: false
|
|
|
|
# Offense count: 5
|
|
# Cop supports --auto-correct.
|
|
Style/SpaceInsidePercentLiteralDelimiters:
|
|
Enabled: false
|
|
|
|
# Offense count: 36
|
|
# Cop supports --auto-correct.
|
|
# Configuration parameters: SupportedStyles.
|
|
# SupportedStyles: use_perl_names, use_english_names
|
|
Style/SpecialGlobalVars:
|
|
EnforcedStyle: use_perl_names
|
|
|
|
# Offense count: 30
|
|
# Cop supports --auto-correct.
|
|
# Configuration parameters: EnforcedStyle, SupportedStyles.
|
|
# SupportedStyles: single_quotes, double_quotes
|
|
Style/StringLiteralsInInterpolation:
|
|
Enabled: false
|
|
|
|
# Offense count: 24
|
|
# Cop supports --auto-correct.
|
|
# Configuration parameters: IgnoredMethods.
|
|
# IgnoredMethods: respond_to, define_method
|
|
Style/SymbolProc:
|
|
Enabled: false
|
|
|
|
# Offense count: 23
|
|
# Cop supports --auto-correct.
|
|
# Configuration parameters: EnforcedStyleForMultiline, SupportedStyles.
|
|
# SupportedStyles: comma, consistent_comma, no_comma
|
|
Style/TrailingCommaInArguments:
|
|
Enabled: false
|
|
|
|
# Offense count: 113
|
|
# Cop supports --auto-correct.
|
|
# Configuration parameters: EnforcedStyleForMultiline, SupportedStyles.
|
|
# SupportedStyles: comma, consistent_comma, no_comma
|
|
Style/TrailingCommaInLiteral:
|
|
Enabled: false
|
|
|
|
# Offense count: 7
|
|
# Cop supports --auto-correct.
|
|
# Configuration parameters: AllowNamedUnderscoreVariables.
|
|
Style/TrailingUnderscoreVariable:
|
|
Enabled: false
|
|
|
|
# Offense count: 90
|
|
# Cop supports --auto-correct.
|
|
Style/TrailingWhitespace:
|
|
Enabled: false
|
|
|
|
# Offense count: 2
|
|
# Cop supports --auto-correct.
|
|
# 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
|
|
Style/TrivialAccessors:
|
|
Enabled: false
|
|
|
|
# Offense count: 3
|
|
# Cop supports --auto-correct.
|
|
Style/UnlessElse:
|
|
Enabled: false
|
|
|
|
# Offense count: 13
|
|
# Cop supports --auto-correct.
|
|
Style/UnneededInterpolation:
|
|
Enabled: false
|
|
|
|
# Offense count: 8
|
|
# Cop supports --auto-correct.
|
|
Style/ZeroLengthPredicate:
|
|
Enabled: false
|