.rubocop.yml

inherit_from: .rubocop_todo.yml

inherit_mode:
  merge:
    - Exclude

require:
  - rubocop-thread_safety
  - rubocop-rspec
  - rubocop-rails
  - rubocop-rake
  - rubocop-performance

AllCops:
  NewCops: enable
  Exclude:
    - db/schema.rb # autogenerated
    - vendor/**/*  # not our code
    - bin/**/*     # autogenerated
    - node_modules/**/*

# we allow exiting from rake tasks
Rails/Exit:
  Exclude:
    - lib/tasks/**/*

# Include: app/**/*.rb, config/**/*.rb, db/**/*.rb, lib/**/*.rb
Rails/Output:
  Exclude:
    - lib/tasks/**/* # rake tasks need output
    - app/jobs/**/*  # heroku scheduler support output# Offense count: 4

RSpec/MessageSpies:
  EnforcedStyle: receive

Metrics/BlockLength:
  Exclude:
    - spec/**/*
    - config/**/*
    - db/migrate/*
    - lib/tasks/**/*

# perceived complexity cop forces us to avoid inlining code, even though
# inlining code makes code easier to reason with and less error prone
Metrics/PerceivedComplexity:
  Enabled: false

# cyclomatic complexity cop forces us to avoid inlining code, even though
# inlining code makes code easier to reason with and less error prone
Metrics/CyclomaticComplexity:
  Enabled: false

# we will never change existing migrations, but we want rubocop
# to fail for old migrations. So this isn't in the .todo file but thel
Rails/CreateTableWithTimestamps:
  Exclude:
    - 'db/migrate/20151210035820_init_schema.rb'
    - 'db/migrate/20160211061631_add_chapter_leaderships_again.rb'
    - 'db/migrate/20160312224628_create_organization_leaderships.rb'

# explicit arguments are easier to understand / read without needing to read the docs
Style/RedundantArgument:
  Enabled: false

Style/Documentation:
  Exclude:
    - spec/**/*