require:
  - rubocop-rails
  - rubocop-rspec

# We exclude files that are generated automatically or test data.
AllCops:
  Exclude:
    - "vendor/**/*"
    - "db/schema.rb"
    - "db/migrate/*"
    - "db/fixtures/**/*"
    - "test/factories/**/*"
    - "spec/factories/**/*"
    - "spec/test_app/db/*"

  DisplayCopNames: true
  NewCops: enable

Rails:
  Enabled: true

# For rails >= 5. TODO: This should be enabled in my opinion.
Rails/HttpPositionalArguments:
  Enabled: false

Rails/InverseOf:
  Enabled: false

Rails/UnknownEnv:
  Environments:
    - production
    - staging
    - development
    - local
    - test

Gemspec/OrderedDependencies:
  Enabled: false

##################### Style ##################################

# We sometimes use non-ascii comments.
Style/AsciiComments:
  Enabled: false

# We prefer trailing comma all the time.
Style/TrailingCommaInArrayLiteral:
  EnforcedStyleForMultiline: comma
Style/TrailingCommaInHashLiteral:
  EnforcedStyleForMultiline: comma
Style/TrailingCommaInArguments:
  EnforcedStyleForMultiline: comma

# We learned documentation is important.
Style/Documentation:
    Enabled: true

# We sometimes want to use `self` anyway.
Style/RedundantSelf:
    Enabled: false

# We want to allow the code like this:
#
#   xs.select {
#     f x
#   }.map { |x|
#     f x
#     f x
#   }.compact
Style/BlockDelimiters:
  Enabled: false
Style/MultilineBlockChain:
  Enabled: false
Style/EmptyMethod:
  Enabled: false

# We sometimes want to use empty case when there are many conditions.
Style/EmptyCaseCondition:
  Enabled: false

Style/DoubleNegation:
  Enabled: false

Style/ParallelAssignment:
  Enabled: false

Style/MethodCallWithoutArgsParentheses:
  Enabled: false

Style/DefWithParentheses:
  Enabled: false

# This might not be a style cop because errors happen due to the order of tests with nested classes/modules.
Style/ClassAndModuleChildren:
  Exclude:
    - "test/**/*.rb"
    - "spec/**/*.rb"

# Sometimes, 1000 is better than 1_000. Especially, it's useful to run grep(1) for searching `1000`.
Style/NumericLiterals:
  Enabled: false

# foo.zero? don't care nil case, we should avoid undefined method risk
Style/NumericPredicate:
  Enabled: false

# We don't care which we use: #yield_self or #then
Style/ObjectThen:
  Enabled: false

# We don't care using ENV.fetch or ENV.[]
Style/FetchEnvVar:
  Enabled: false

# We don't care which style we should use.
Style/IfUnlessModifier:
  Enabled: false

##################### Layout ##################################
# We sometimes want to put multiple spaces before arguments.
Layout/SpaceBeforeFirstArg:
  Enabled: false
Layout/SpaceInLambdaLiteral:
  Enabled: false
Layout/HeredocIndentation:
  Enabled: false

# We want to allow various expression for arguments.
#
# foo(a,
#  b
# )
# foo(
#  a,
#  b)
# foo(
#  a,
#  b
# )
# foo(a,
#  b)
Layout/MultilineMethodCallBraceLayout:
  Enabled: false

# We don't want to enforce various method calls.
#
# while a
# .b
#   something
# end
# while a
#      .b
#   something
# end
# Thing.a
#    .b
#    .c
# while a
#   .b
#   something
# end
Layout/MultilineMethodCallIndentation:
  Enabled: false

# Sometimes, we want to write like this:
#
# aaa(bbb(
#   ccc
# ))
Layout/FirstParameterIndentation:
  Enabled: false

# We want to allow both of them:
#
#   aaa.
#     bb().
#     cc()
#   aaa
#     .bb()
#     .cc()
#
# The first one allows us to insert comments, and it would be convenient.
# The second one is the default of this Cop.
Layout/DotPosition:
  Enabled: false

# We want to consistently write fixture files with the rule: 1 line for 1 record.
# It's not intuitive with multiple lines.
Layout/ClosingParenthesisIndentation:
  Exclude:
    - "db/fixtures/*.rb"
Layout/ParameterAlignment:
  Exclude:
    - "db/fixtures/*.rb"

Layout/LineLength:
  Max: 160
  Exclude:
    - "db/migrate/*.rb"
    - "db/fixtures/*.rb" # Fixture files usually include long lines.

##################### Lint ##################################

# We sometimes want to put spaces before arguments.
Lint/ParenthesesAsGroupedExpression:
    Enabled: false

##################### Metrics ##################################

Metrics/AbcSize:
  Enabled: false

Metrics/BlockLength:
  Enabled: false

Metrics/ClassLength:
  Enabled: false

Metrics/CyclomaticComplexity:
  Enabled: false

Metrics/MethodLength:
  Enabled: false

Metrics/ModuleLength:
  Enabled: false

Metrics/ParameterLists:
  CountKeywordArgs: false

Metrics/PerceivedComplexity:
  Enabled: false

# Does the variable name for an exception object really matter?
# There are many things to think about before taking care of it.
Naming/RescuedExceptionsVariableName:
  Enabled: false

##################### RSpec ##################################

# disabling to not to show errors if there is no when/with/without in the context
RSpec/ContextWording:
  Enabled: false

# controller specs and request specs are hard to name reasonably
RSpec/NamedSubject:
  Enabled: false

# because response validation in `request spec` is expected to result in large rows
RSpec/ExampleLength:
  Enabled: false

# We don't care about whether using `described_class` or not.
RSpec/DescribedClass:
  Enabled: false