arcadia_cops 4.0.2 → 4.0.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 6651da173543ee1ffb9d8e31b0dfda75c35ef5454f9d1ff054160ae5d621041a
-  data.tar.gz: d7e2497155d064da862d66b65a1163f5abe01ef1a0afee89df5e879ac46ea281
+  metadata.gz: 6e254a8851f6048dabca6101e69ab0160d19a6155636f8fd46eb46e8970ed17f
+  data.tar.gz: b2d36f4a92357caf5468c31deab4e1af9894220da245ea51e7a4bae4f84d946c
 SHA512:
-  metadata.gz: dc99132ea7385e8d1e9cd2943b130fd2736dfef8f6a024b8b341fc4748ee519588f290c8af65c5806ef8460427d72ba21a4532a97bc9b36a54f7db12cb1cef59
-  data.tar.gz: '03087562410c086336d1d94f59431675a5b10bef7ad631acd27857e63eb15653c0bbc1fd5357a34f1d9e380f578c11ccee7963a650eb6b10f341a2a48356006f'
+  metadata.gz: a4a552a043a4889583cb7e2be39c74663b9b66906265b069efa674185a6e4ef7c82c55e4040e28305aec637b2290677f4921e962e41efdc420a8f7826cbcff6d
+  data.tar.gz: 6f26d7596c968cc0559b3d47190bf1d09755bd507d02705654b2c8476dcdaa040bdc282af30704ad41cb904c1f4e81435d150b51bff5d775dfeadd66909b0f14

data/README.md

@@ -18,6 +18,8 @@ Add a new cop to the `enabled.yml`, bump the version, and put in a PR for review
 
 To see all cops including those that aren't enabled run `bundle exec rubocop --show-cops`.
 
+WHen developing custom cops, make sure to add specs and run run `bundle exec rspec` before releasing.
+
 ## Release
 
 Ensure you have bumped the version and run `rake release` to release to rubygems.org.

data/config/config.yml

@@ -4,6 +4,7 @@
 require:
   - rubocop-rails
   - rubocop-rspec
+  - ../lib/arcadia_cops.rb
 
 # Common configuration.
 AllCops:

data/config/enabled.yml

@@ -687,3 +687,9 @@ Style/Not:
 
 Style/NegatedUnless:
   Enabled: true
+
+ArcadiaCops/SimpleModifierConditional:
+  Enabled: true
+
+ArcadiaCops/SimpleUnless:
+  Enabled: true

data/lib/arcadia_cops/simple_modifier_conditional.rb

@@ -0,0 +1,21 @@
+# frozen_string_literal: true
+
+module ArcadiaCops
+  # Cop to tackle prevent more complicated modifier if/unless statements
+  # https://github.com/airbnb/ruby/blob/12435e8136d2adf710de999bc0f6bef01215df2c/rubocop-airbnb/lib/rubocop/cop/airbnb/simple_modifier_conditional.rb
+  class SimpleModifierConditional < RuboCop::Cop::Cop
+    MSG = 'Modifier if/unless usage is okay when the body is simple, ' \
+      'the condition is simple, and the whole thing fits on one line. ' \
+      'Otherwise, avoid modifier if/unless.'.freeze
+
+    def_node_matcher :multiple_conditionals?, '(if ({and or :^} ...) ...)'
+
+    def on_if(node)
+      return unless node.modifier_form?
+
+      if multiple_conditionals?(node) || node.multiline?
+        add_offense(node)
+      end
+    end
+  end
+end

data/lib/arcadia_cops/simple_unless.rb

@@ -0,0 +1,17 @@
+# frozen_string_literal: true
+
+module ArcadiaCops
+  # Cop to tackle prevent unless statements with multiple conditions
+  # https://github.com/airbnb/ruby/blob/12435e8136d2adf710de999bc0f6bef01215df2c/rubocop-airbnb/lib/rubocop/cop/airbnb/simple_unless.rb
+  class SimpleUnless < RuboCop::Cop::Cop
+    MSG = 'Unless usage is okay when there is only one conditional'.freeze
+
+    def_node_matcher :multiple_conditionals?, '(if ({and or :^} ...) ...)'
+
+    def on_if(node)
+      return unless node.unless?
+
+      add_offense(node) if multiple_conditionals?(node)
+    end
+  end
+end

data/lib/arcadia_cops.rb

@@ -0,0 +1,4 @@
+# frozen_string_literal: true
+
+require 'rubocop'
+Dir.glob(File.expand_path('arcadia_cops/*.rb', File.dirname(__FILE__))).map(&method(:require))

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: arcadia_cops
 version: !ruby/object:Gem::Version
-  version: 4.0.2
+  version: 4.0.4
 platform: ruby
 authors:
 - engineering
-autorequire: 
+autorequire:
 bindir: bin
 cert_chain: []
-date: 2022-06-07 00:00:00.000000000 Z
+date: 2023-03-21 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rubocop
@@ -16,14 +16,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 1.29.1
+        version: '1.29'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 1.29.1
+        version: '1.29'
 - !ruby/object:Gem::Dependency
   name: rubocop-rails
   requirement: !ruby/object:Gem::Requirement
@@ -66,6 +66,20 @@ dependencies:
     - - ">"
       - !ruby/object:Gem::Version
         version: 2.0.2
+- !ruby/object:Gem::Dependency
+  name: rspec
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: rake
   requirement: !ruby/object:Gem::Requirement
@@ -80,7 +94,7 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
-description: Contains enabled rubocops for arcadia power ruby repos.
+description: Contains enabled rubocops for Arcadia ruby repos.
 email:
 - engineering@arcadia.com
 executables: []
@@ -90,11 +104,14 @@ files:
 - README.md
 - config/config.yml
 - config/enabled.yml
+- lib/arcadia_cops.rb
+- lib/arcadia_cops/simple_modifier_conditional.rb
+- lib/arcadia_cops/simple_unless.rb
 homepage: https://github.com/ArcadiaPower/arcadia_cops/
 licenses:
 - MIT
 metadata: {}
-post_install_message: 
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -109,8 +126,8 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.1.6
-signing_key: 
+rubygems_version: 3.4.9
+signing_key:
 specification_version: 4
-summary: Arcadia Power Style Cops
+summary: Arcadia Style Cops
 test_files: []