.rubocop.yml

inherit_from: .rubocop_todo.yml

plugins:
  - rubocop-performance
  - rubocop-rails
  - rubocop-rspec

require:
  - rubocop-capybara
  - rubocop-factory_bot
  - rubocop-rspec_rails

AllCops:
  NewCops: enable
  Exclude:
    - "db/schema.rb"
    - "db/migrate/20230628052836_init_schema.rb"
    - "vendor/**/*"

Layout/HashAlignment:
  EnforcedHashRocketStyle: table
  EnforcedColonStyle: table

Layout/LineLength:
  Exclude:
    - "db/**/*"

# This is too aggressive with our use of Seattle.rb style lack of parens on method calls
# E.g., return redirect_to [:articles]
# Rubocop thinks that `redirect_to` is a collection and `:articles` is a hash key
Layout/SpaceBeforeBrackets:
  Enabled: false

Metrics/AbcSize:
  Max: 80

Metrics/BlockLength:
  Max: 99
  Exclude:
    - "config/routes.rb"

Metrics/ClassLength:
  Max: 180

Metrics/CyclomaticComplexity:
  Max: 13

Metrics/MethodLength:
  Max: 40
  Exclude:
    - "db/migrate/*"

Metrics/ModuleLength:
  Max: 121

Metrics/ParameterLists:
  Max: 6

Metrics/PerceivedComplexity:
  Max: 15

Naming/VariableNumber:
  EnforcedStyle: snake_case

# TODO: https://docs.rubocop.org/rubocop-rails/cops_rails.html#railsenvironmentvariableaccess
#       Consider using `Rails.application.config.foo` instead of `ENV['FOO']`
Rails/EnvironmentVariableAccess:
  Enabled: false

# Using `puts` is allowed in these files
Rails/Output:
  Exclude:
    - "app/lib/code_archiver.rb"
    - "db/seeds.rb"

Rails/TimeZone:
  Exclude:
    - "script/*"

RSpec/DescribeClass:
  Exclude:
    - "spec/system/*"

RSpec/ExampleLength:
  Max: 14
  Exclude:
    - "spec/system/*"

RSpec/MessageSpies:
  Enabled: false

RSpec/MultipleExpectations:
  Max: 5
  Exclude:
    - "spec/system/*"

Style/AsciiComments:
  Enabled: false

Style/Documentation:
  Enabled: false

Style/FrozenStringLiteralComment:
  Enabled: false

Style/MethodDefParentheses:
  EnforcedStyle: require_no_parentheses

Style/MixinUsage:
  Exclude:
    - "script/*"

Style/RedundantFetchBlock:
  Enabled: false

Style/OpenStructUse:
  Enabled: false

# For now we disable this, since explicit hash value sounds better to use
Style/HashSyntax:
  Enabled: false

# For now we disable this, since I prefer explicit block name
Naming/BlockForwarding:
  Enabled: false

# We don't translate admin flash message for now
Rails/I18nLocaleTexts:
  Enabled: false

# Remove once we won,t have commented tests
RSpec/NoExpectationExample:
  Enabled: false

# If we don,t care about explicit type, then remove this and fix errors
RSpecRails/InferredSpecType:
  Enabled: false