house_style 1.6.0 → 2.1.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 2b3c9f9f6267d9d7370a38d34f97b6277aa7a843209597965fc7ed71994f18a9
-  data.tar.gz: 2eb0151b69c20a8fc48fe0d1ed53cd46c4706a6a592a6afd98486cf3016fd0ef
+  metadata.gz: 60dd79ae28c2a1699e7484038e09b801a34f6a61d7657ab7ba64a7cbf70db40d
+  data.tar.gz: 78084875ed16986c3b8291e356498ec97607454d3364b0532e744d81074af377
 SHA512:
-  metadata.gz: 01ac391150f614674bcdf5036efce447e118b14e1845874a1acdd6c7ff3c15b6470078b67b73b8bfa6fd08a86fd3238b1be77d2951e9162284c8dfb03f825952
-  data.tar.gz: 31031e95a31e4d42029194e01265bf10d8ee2673b819cb5319c2323759fdb4bee7a95c8b479a0a5880d6c2bc891c86d541a96396418dd57820d35ad4484e3d69
+  metadata.gz: b3eeb1147984ef9a73d8ec6510dd6df3300e61abc487e66d88ec84c3cc27b00ee00c8bb9756e0283b2ebba06f85ab8fe95980ddae2b9f4fc0cfcf2d916217032
+  data.tar.gz: 7edfe2d0ebff0553ff58c1c582196e5ef17c80e32dbc9f7805f2fa43fa85d3e9efc6bab11675707218d5a4a4ff0d0b7f64e9c396a6fd95e5feab18de42755214

data/.rubocop.yml

@@ -3,7 +3,3 @@
 # See the [README](README.md) for instructions on using `house_style`
 # style settings from within your own projects
 inherit_from: ruby/rubocop.yml
-
-Style/IndentHeredoc:
-  Exclude:
-    - house_style.gemspec

data/.travis.yml

@@ -1,4 +1,7 @@
-language: ruby
+language:
+cache: bundler
 rvm:
-  - 2.2.2
-before_install: gem install bundler -v 1.10.6
+  - 2.6
+  - 2.7
+  - 3.0
+before_install: gem install bundler -v 2.2.18

data/CHANGELOG.md

@@ -1,5 +1,36 @@
 # CHANGELOG
 
+## 2.1.0
+- Enable NewCops by default
+- Support Rubocop up to 1.22.x and latest extensions
+
+## 2.0.2
+### Bug fixes
+- Exclude Guardfile from Ruby & Rails configurations
+
+## 2.0.1
+### Bug fixes
+- Ignore other database schemas in /db for Rails apps
+- Disable `RSpec/MultipleExpectations` cop.
+- Tweak `RSpec/MultipleMemoizedHelpers` to 10.
+
+## 2.0.0
+### New features
+
+- Add new rubocop extensions: rubocop-performance & rubocop-rake. Both enabled by default on Ruby configuration.
+
+### Bug fixes
+
+- Restrict extensions versions to avoid introducing new cops in the apps depending on house_style without previously configuing them here.
+- Exclude most Rails default configuration files for our cops to decrease the number of conflicts during Rails upgrades.
+
+### Changes
+
+- Update Travis CI configuration and add a default Rake task (rubocop)
+- Upgrade development dependencies: Bundler & Rake.
+- Upgrade rubocop to 1.15.0, rubocop-rails to 2.10.1 and rubocop-rspec to 2.3.0.
+- Remove old pre-1.0 post-install message.
+
 ## 1.6.0
 - [FIX] Require Rubocop higher than 0.49 to fix security issue: https://nvd.nist.gov/vuln/detail/CVE-2017-8418
 - [FIX] Rename `IndentHash` to `IndentFirstHashElement`, and require Rubocop 0.68 or higher because of this change.

data/Rakefile

@@ -1 +1,9 @@
 require 'bundler/gem_tasks'
+require 'rubocop/rake_task'
+
+desc 'Run rubocop'
+task :rubocop do
+  RuboCop::RakeTask.new
+end
+
+task default: :rubocop

data/house_style.gemspec

@@ -1,11 +1,9 @@
-# coding: utf-8
-
-lib = File.expand_path('../lib', __FILE__)
+lib = File.expand_path('lib', __dir__)
 $LOAD_PATH.unshift(lib) unless $LOAD_PATH.include?(lib)
 
 Gem::Specification.new do |spec|
   spec.name          = 'house_style'
-  spec.version       = '1.6.0'
+  spec.version       = '2.1.0'
   spec.authors       = ['Altmetric', 'Scott Matthewman']
   spec.email         = ['engineering@altmetric.com', 'scott.matthewman@gmail.com']
 
@@ -14,20 +12,15 @@ Gem::Specification.new do |spec|
   spec.homepage      = 'https://github.com/altmetric/house_style'
   spec.license       = 'MIT'
 
-  spec.post_install_message = <<-ENDOFMESSAGE
-If you are updating house_style from version 1.0.0 or older, note that you can
-now remove any reference to TargetRubyVersion in your project's .rubocop.yml
-file. You should specify a .ruby-version file for your entire project, and that
-will be taken into account by rubocop also.
-  ENDOFMESSAGE
-
   spec.files         = `git ls-files -z`.split("\x0").reject { |f| f.match(%r{^(test|spec|features)/}) }
   spec.require_paths = ['lib']
 
-  spec.add_dependency 'rubocop', '>= 0.68', '< 0.75'
-  spec.add_dependency 'rubocop-rails', '~> 2.3'
-  spec.add_dependency 'rubocop-rspec', '~> 1.15'
+  spec.add_dependency 'rubocop', '>= 1.0', '< 1.23'
+  spec.add_dependency 'rubocop-performance', '~> 1.11.3'
+  spec.add_dependency 'rubocop-rails', '~> 2.12.3'
+  spec.add_dependency 'rubocop-rake', '< 1.0'
+  spec.add_dependency 'rubocop-rspec', '~> 2.5.0'
 
-  spec.add_development_dependency 'bundler', '~> 1.10'
-  spec.add_development_dependency 'rake', '~> 11.0'
+  spec.add_development_dependency 'bundler', '~> 2.0'
+  spec.add_development_dependency 'rake', '~> 13.0'
 end

data/performance/default_rules.yml

@@ -0,0 +1,32 @@
+Performance/AncestorsInclude:
+  Enabled: true
+Performance/BigDecimalWithNumericArgument:
+  Enabled: true
+Performance/BlockGivenWithExplicitBlock:
+  Enabled: true
+Performance/CollectionLiteralInLoop:
+  Enabled: true
+Performance/ConstantRegexp:
+  Enabled: true
+Performance/MapCompact:
+  Enabled: true
+Performance/MethodObjectAsBlock:
+  Enabled: true
+Performance/RedundantEqualityComparisonBlock:
+  Enabled: true
+Performance/RedundantSortBlock:
+  Enabled: true
+Performance/RedundantSplitRegexpArgument:
+  Enabled: true
+Performance/RedundantStringChars:
+  Enabled: true
+Performance/ReverseFirst:
+  Enabled: true
+Performance/SortReverse:
+  Enabled: true
+Performance/Squeeze:
+  Enabled: true
+Performance/StringInclude:
+  Enabled: true
+Performance/Sum:
+  Enabled: true

data/performance/rubocop.yml

@@ -0,0 +1,4 @@
+inherit_from:
+  - ./default_rules.yml
+
+require: rubocop-performance

data/rails/db_migrate.yml

@@ -1,6 +1,10 @@
-Metrics/ClassLength:
+Layout/EmptyLineBetweenDefs:
+  Enabled: false
+Layout/LineLength:
+  Enabled: false
+Metrics/AbcSize:
   Enabled: false
-Metrics/LineLength:
+Metrics/ClassLength:
   Enabled: false
 Metrics/MethodLength:
   Enabled: false
@@ -8,9 +12,5 @@ Style/BlockDelimiters:
   Enabled: false
 Style/Documentation:
   Enabled: false
-Layout/EmptyLineBetweenDefs:
-  Enabled: false
 Style/StringLiterals:
   Enabled: false
-Metrics/AbcSize:
-  Enabled: false

data/rails/default_rules.yml

@@ -0,0 +1,34 @@
+Rails/ActiveRecordCallbacksOrder:
+  Enabled: true
+Rails/AfterCommitOverride:
+  Enabled: true
+Rails/AttributeDefaultBlockValue:
+  Enabled: true
+Rails/FindById:
+  Enabled: true
+Rails/Inquiry:
+  Enabled: true
+Rails/MailerName:
+  Enabled: true
+Rails/MatchRoute:
+  Enabled: true
+Rails/NegateInclude:
+  Enabled: true
+Rails/Pluck:
+  Enabled: true
+Rails/PluckInWhere:
+  Enabled: true
+Rails/RenderInline:
+  Enabled: true
+Rails/RenderPlainText:
+  Enabled: true
+Rails/SquishedSQLHeredocs:
+  Enabled: true
+Rails/TimeZoneAssignment:
+  Enabled: true
+Rails/WhereEquals:
+  Enabled: true
+Rails/WhereExists:
+  Enabled: true
+Rails/WhereNot:
+  Enabled: true

data/rails/rubocop.yml

@@ -1,15 +1,25 @@
-inherit_from: ../ruby/rubocop.yml
+inherit_from:
+  - ../ruby/rubocop.yml
+  - ./default_rules.yml
+  - ./style.yml
+
 AllCops:
   Exclude:
     - bin/**/*
     - vendor/**/*
-    - db/schema.rb
+    - db/*schema.rb
     - lib/tasks/cucumber.rake
+    - Guardfile
 Rails:
   Enabled: true
 Rails/InverseOf:
   Enabled: false
 Rails/HasManyOrHasOneDependent:
   Enabled: false
+Rails/EnvironmentVariableAccess:
+  Enabled: true
+  AllowReads: true
+Rails/ShortI18n:
+  Enabled: false
 
 require: rubocop-rails

data/rails/style.yml

@@ -0,0 +1,31 @@
+# Exclude Rails default configuration files that offend our cops to avoid
+# conflicts during Rails upgrades
+Layout/SpaceInsideArrayLiteralBrackets:
+  Exclude:
+    - config/environments/production.rb
+
+Rails/FilePath:
+  Exclude:
+    - config/environments/development.rb
+
+Style/GlobalStdStream:
+  Exclude:
+    - config/environments/production.rb
+
+Style/StringLiterals:
+  Exclude:
+    - config/application.rb
+    - config/boot.rb
+    - config/environment.rb
+    - config/environments/development.rb
+    - config/environments/production.rb
+    - config/environments/test.rb
+    - config/initializers/**
+    - config/puma.rb
+    - config.ru
+    - Rakefile
+
+Style/SymbolArray:
+  Exclude:
+    - config/initializers/filter_parameter_logging.rb
+

data/rake/rubocop.yml

@@ -0,0 +1 @@
+require: rubocop-rake

data/rspec/rubocop.yml

@@ -2,14 +2,26 @@ inherit_from: ../ruby/rubocop.yml
 
 require: rubocop-rspec
 
-Metrics/BlockLength:
+Layout/LineLength:
+  Enabled: false
+Layout/SpaceInsideHashLiteralBraces:
   Enabled: false
-Metrics/LineLength:
+Lint/AmbiguousBlockAssociation:
+  Exclude:
+    - spec/**/*
+Metrics/BlockLength:
   Enabled: false
 Metrics/ModuleLength:
   Enabled: false
+RSpec/ContextWording:
+  Exclude:
+    - spec/support/shared_contexts/rake.rb
 RSpec/ExampleLength:
   Enabled: false
+RSpec/MultipleExpectations:
+  Enabled: false
+RSpec/MultipleMemoizedHelpers:
+  Max: 10
 RSpec/NotToNot:
   EnforcedStyle: to_not
 Style/BlockDelimiters:
@@ -17,16 +29,8 @@ Style/BlockDelimiters:
     - spec/factories/**/*.rb
 Style/Documentation:
   Enabled: false
-Layout/SpaceInsideHashLiteralBraces:
-  Enabled: false
 Style/SymbolProc:
   Exclude:
     - spec/factories/**/*.rb
 Style/WordArray:
   Enabled: false
-Lint/AmbiguousBlockAssociation:
-  Exclude:
-    - spec/**/*
-RSpec/ContextWording:
-  Exclude:
-    - spec/support/shared_contexts/rake.rb

data/ruby/configuration.yml

@@ -3,7 +3,12 @@ AllCops:
   DisplayStyleGuide: true
   ExtraDetails: true
   UseCache: true
+  NewCops: enable
 
   Exclude:
-    - 'vendor/**/*'
     - 'bin/**/*'
+    - 'vendor/**/*'
+    - 'Guardfile'
+
+Gemspec/RequiredRubyVersion:
+  Enabled: false

data/ruby/default_rules.yml

@@ -0,0 +1,60 @@
+Gemspec/DateAssignment:
+  Enabled: true
+Layout/SpaceBeforeBrackets:
+  Enabled: true
+Lint/AmbiguousAssignment:
+  Enabled: true
+Lint/DeprecatedConstants:
+  Enabled: true
+Lint/DuplicateBranch:
+  Enabled: true
+Lint/DuplicateRegexpCharacterClassElement:
+  Enabled: true
+Lint/EmptyBlock:
+  Enabled: true
+Lint/EmptyClass:
+  Enabled: true
+Lint/LambdaWithoutLiteralBlock:
+  Enabled: true
+Lint/NoReturnInBeginEndBlocks:
+  Enabled: true
+Lint/NumberedParameterAssignment:
+  Enabled: true
+Lint/OrAssignmentToConstant:
+  Enabled: true
+Lint/RedundantDirGlobSort:
+  Enabled: true
+Lint/SymbolConversion:
+  Enabled: true
+Lint/ToEnumArguments:
+  Enabled: true
+Lint/TripleQuotes:
+  Enabled: true
+Lint/UnexpectedBlockArity:
+  Enabled: true
+Lint/UnmodifiedReduceAccumulator:
+  Enabled: true
+Style/ArgumentsForwarding:
+  Enabled: true
+Style/CollectionCompact:
+  Enabled: true
+Style/DocumentDynamicEvalDefinition:
+  Enabled: true
+Style/EndlessMethod:
+  Enabled: true
+Style/HashConversion:
+  Enabled: true
+Style/HashExcept:
+  Enabled: true
+Style/IfWithBooleanLiteralBranches:
+  Enabled: true
+Style/NegatedIfElseCondition:
+  Enabled: true
+Style/NilLambda:
+  Enabled: true
+Style/RedundantArgument:
+  Enabled: true
+Style/StringChars:
+  Enabled: true
+Style/SwapValues:
+  Enabled: true

data/ruby/metrics.yml

@@ -1,10 +1,2 @@
-Metrics/LineLength:
-  Max: 120
-  Details: >-
-    If lines are too short, methods become hard to read because you must
-    constantly jump from line to line. If lines are too long, following
-    the flow can be hard because it becomes harder to scan back and
-    locate the start of the next line. 80 characters is generally a bit
-    too short; 120 characters is a good compromise.
 Metrics/MethodLength:
   Max: 10

data/ruby/rubocop.yml

@@ -2,3 +2,6 @@ inherit_from:
   - ./configuration.yml
   - ./metrics.yml
   - ./style.yml
+  - ./default_rules.yml
+  - ../rake/rubocop.yml
+  - ../performance/rubocop.yml

data/ruby/style.yml

@@ -1,3 +1,23 @@
+Layout/EmptyLineBetweenDefs:
+  AllowAdjacentOneLineDefs: true
+Layout/FirstArrayElementIndentation:
+  EnforcedStyle: consistent
+Layout/FirstHashElementIndentation:
+  EnforcedStyle: consistent
+Layout/MultilineOperationIndentation:
+  Enabled: false
+Layout/MultilineMethodCallIndentation:
+  EnforcedStyle: indented
+Layout/LineLength:
+  Max: 120
+  Details: >-
+    If lines are too short, methods become hard to read because you must
+    constantly jump from line to line. If lines are too long, following
+    the flow can be hard because it becomes harder to scan back and
+    locate the start of the next line. 80 characters is generally a bit
+    too short; 120 characters is a good compromise.
+Style/AccessModifierDeclarations:
+  Enabled: false
 Style/BlockDelimiters:
   EnforcedStyle: semantic
   FunctionalMethods:
@@ -17,12 +37,6 @@ Style/BlockDelimiters:
     - let!
     - subject
     - watch
-Style/BracesAroundHashParameters:
-  EnforcedStyle: context_dependent
-  Details: >-
-    When the last (or only) argument to a method is a Hash,
-    omit the { } braces. The exception is if the preceding
-    argument is also a Hash.
 Style/CollectionMethods:
   Enabled: true
 Style/CommandLiteral:
@@ -31,24 +45,14 @@ Style/Copyright:
   Enabled: false
 Style/Documentation:
   Enabled: false
-Layout/EmptyLineBetweenDefs:
-  AllowAdjacentOneLineDefs: true
 Style/FrozenStringLiteralComment:
   Enabled: false
 Style/GuardClause:
   MinBodyLength: 3
 Style/HashSyntax:
   EnforcedStyle: ruby19_no_mixed_keys
-Layout/IndentFirstArrayElement:
-  EnforcedStyle: consistent
-Layout/IndentFirstHashElement:
-  EnforcedStyle: consistent
 Style/MethodCalledOnDoEndBlock:
   Enabled: true
-Layout/MultilineOperationIndentation:
-  Enabled: false
-Layout/MultilineMethodCallIndentation:
-  EnforcedStyle: indented
 Style/NonNilCheck:
   IncludeSemanticChanges: true
 Style/NumericLiterals:
@@ -60,5 +64,3 @@ Style/StringLiterals:
   Enabled: true
 Style/StringMethods:
   Enabled: true
-Style/AccessModifierDeclarations:
-  Enabled: false

metadata

@@ -1,15 +1,15 @@
 --- !ruby/object:Gem::Specification
 name: house_style
 version: !ruby/object:Gem::Version
-  version: 1.6.0
+  version: 2.1.0
 platform: ruby
 authors:
 - Altmetric
 - Scott Matthewman
-autorequire: 
+autorequire:
 bindir: bin
 cert_chain: []
-date: 2019-09-19 00:00:00.000000000 Z
+date: 2021-10-08 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rubocop
@@ -17,76 +17,104 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '0.68'
+        version: '1.0'
     - - "<"
       - !ruby/object:Gem::Version
-        version: '0.75'
+        version: '1.23'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '0.68'
+        version: '1.0'
     - - "<"
       - !ruby/object:Gem::Version
-        version: '0.75'
+        version: '1.23'
+- !ruby/object:Gem::Dependency
+  name: rubocop-performance
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 1.11.3
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 1.11.3
 - !ruby/object:Gem::Dependency
   name: rubocop-rails
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '2.3'
+        version: 2.12.3
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '2.3'
+        version: 2.12.3
+- !ruby/object:Gem::Dependency
+  name: rubocop-rake
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "<"
+      - !ruby/object:Gem::Version
+        version: '1.0'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "<"
+      - !ruby/object:Gem::Version
+        version: '1.0'
 - !ruby/object:Gem::Dependency
   name: rubocop-rspec
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.15'
+        version: 2.5.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.15'
+        version: 2.5.0
 - !ruby/object:Gem::Dependency
   name: bundler
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.10'
+        version: '2.0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.10'
+        version: '2.0'
 - !ruby/object:Gem::Dependency
   name: rake
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '11.0'
+        version: '13.0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '11.0'
+        version: '13.0'
 description: Configuration files and other snippets to help you apply standards across
   multiple projects
 email:
@@ -114,10 +142,16 @@ files:
 - lib/generators/house_style/templates/db_migrate_rubocop.yml
 - lib/generators/house_style/templates/rspec-rubocop.yml
 - lib/generators/house_style/templates/rubocop.yml
+- performance/default_rules.yml
+- performance/rubocop.yml
 - rails/db_migrate.yml
+- rails/default_rules.yml
 - rails/rubocop.yml
+- rails/style.yml
+- rake/rubocop.yml
 - rspec/rubocop.yml
 - ruby/configuration.yml
+- ruby/default_rules.yml
 - ruby/metrics.yml
 - ruby/rubocop.yml
 - ruby/style.yml
@@ -125,11 +159,7 @@ homepage: https://github.com/altmetric/house_style
 licenses:
 - MIT
 metadata: {}
-post_install_message: |
-  If you are updating house_style from version 1.0.0 or older, note that you can
-  now remove any reference to TargetRubyVersion in your project's .rubocop.yml
-  file. You should specify a .ruby-version file for your entire project, and that
-  will be taken into account by rubocop also.
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -144,8 +174,8 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.0.3
-signing_key: 
+rubygems_version: 3.2.22
+signing_key:
 specification_version: 4
 summary: A centralised store of house style rules
 test_files: []