rubocop-scc 0.2.1 → 0.2.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 2c4469f298c5d79ec6f04eaf319a237f801ca2a19eb69d11ff22d21d3e3e10e4
-  data.tar.gz: a77f7f20ad7449b0b55c1af1757b5acef5cd0445cd580fe506f3c92c0cf9acc2
+  metadata.gz: 78494b0a918b525ad912cda5e333c3cda12f328b9a318a6c97929e563311db61
+  data.tar.gz: b369263e3fe3c0a7a03e59f62ff712726bca05cc9f3aa52c4c66810e50bba08a
 SHA512:
-  metadata.gz: c81e6504f2715173b40f7f9166a5bfaa363ef19525fc01221e1e4319496fe835c0749e42250566fb58fabdf3410e2f748bc2a9c062f86b4d103dcfbdf1f64e82
-  data.tar.gz: 872bb7814ccea0b6262cfe410c966d9e113b6aef86eb01363acf0282c81aac7801fe07176c4a449494ab8d32d3c7c1f803615e3d02e1bc02f002befb2834ff05
+  metadata.gz: b27de17b481f3d9a512919cfad59059b06ffde9ec0982e80c246b15816866f05196da4cb8d158d6ce87579b3cca94aa041d02d5be9a37927acb59b348ff774b6
+  data.tar.gz: 91d161c566f0a557b8d26aa13525b69f0cfe0faf0363b1c75f74c790e3b2a8676d361def26ae099112b76df07e85f3b741178aeba340be20382cd718b3f39816

data/config/default.yml

@@ -7,10 +7,10 @@ Bundler/DuplicatedGem:
   Enabled: true
 
 Bundler/InsecureProtocolSource:
-  Enabled: true
+  Enabled: false
 
 Bundler/OrderedGems:
-  Enabled: true
+  Enabled: false
 
 Layout/AccessModifierIndentation:
   Enabled: true
@@ -444,73 +444,73 @@ Style/BlockDelimiters:
   Enabled: true
   EnforcedStyle: line_count_based
   SupportedStyles:
-    # The `line_count_based` style enforces braces around single line blocks and
-    # do..end around multi-line blocks.
-    - line_count_based
-    # The `semantic` style enforces braces around functional blocks, where the
-    # primary purpose of the block is to return a value and do..end for
-    # procedural blocks, where the primary purpose of the block is its
-    # side-effects.
-    #
-    # This looks at the usage of a block's method to determine its type (e.g. is
-    # the result of a `map` assigned to a variable or passed to another
-    # method) but exceptions are permitted in the `ProceduralMethods`,
-    # `FunctionalMethods` and `IgnoredMethods` sections below.
-    - semantic
-    # The `braces_for_chaining` style enforces braces around single line blocks
-    # and do..end around multi-line blocks, except for multi-line blocks whose
-    # return value is being chained with another method (in which case braces
-    # are enforced).
-    - braces_for_chaining
+  # The `line_count_based` style enforces braces around single line blocks and
+  # do..end around multi-line blocks.
+  - line_count_based
+  # The `semantic` style enforces braces around functional blocks, where the
+  # primary purpose of the block is to return a value and do..end for
+  # procedural blocks, where the primary purpose of the block is its
+  # side-effects.
+  #
+  # This looks at the usage of a block's method to determine its type (e.g. is
+  # the result of a `map` assigned to a variable or passed to another
+  # method) but exceptions are permitted in the `ProceduralMethods`,
+  # `FunctionalMethods` and `IgnoredMethods` sections below.
+  - semantic
+  # The `braces_for_chaining` style enforces braces around single line blocks
+  # and do..end around multi-line blocks, except for multi-line blocks whose
+  # return value is being chained with another method (in which case braces
+  # are enforced).
+  - braces_for_chaining
   ProceduralMethods:
-    # Methods that are known to be procedural in nature but look functional from
-    # their usage, e.g.
-    #
-    #   time = Benchmark.realtime do
-    #     foo.bar
-    #   end
-    #
-    # Here, the return value of the block is discarded but the return value of
-    # `Benchmark.realtime` is used.
-    - benchmark
-    - bm
-    - bmbm
-    - create
-    - each_with_object
-    - measure
-    - new
-    - realtime
-    - tap
-    - with_object
+  # Methods that are known to be procedural in nature but look functional from
+  # their usage, e.g.
+  #
+  #   time = Benchmark.realtime do
+  #     foo.bar
+  #   end
+  #
+  # Here, the return value of the block is discarded but the return value of
+  # `Benchmark.realtime` is used.
+  - benchmark
+  - bm
+  - bmbm
+  - create
+  - each_with_object
+  - measure
+  - new
+  - realtime
+  - tap
+  - with_object
   FunctionalMethods:
-    # Methods that are known to be functional in nature but look procedural from
-    # their usage, e.g.
-    #
-    #   let(:foo) { Foo.new }
-    #
-    # Here, the return value of `Foo.new` is used to define a `foo` helper but
-    # doesn't appear to be used from the return value of `let`.
-    - let
-    - let!
-    - subject
-    - watch
+  # Methods that are known to be functional in nature but look procedural from
+  # their usage, e.g.
+  #
+  #   let(:foo) { Foo.new }
+  #
+  # Here, the return value of `Foo.new` is used to define a `foo` helper but
+  # doesn't appear to be used from the return value of `let`.
+  - let
+  - let!
+  - subject
+  - watch
   IgnoredMethods:
-    # Methods that can be either procedural or functional and cannot be
-    # categorised from their usage alone, e.g.
-    #
-    #   foo = lambda do |x|
-    #     puts "Hello, #{x}"
-    #   end
-    #
-    #   foo = lambda do |x|
-    #     x * 100
-    #   end
-    #
-    # Here, it is impossible to tell from the return value of `lambda` whether
-    # the inner block's return value is significant.
-    - lambda
-    - proc
-    - it
+  # Methods that can be either procedural or functional and cannot be
+  # categorised from their usage alone, e.g.
+  #
+  #   foo = lambda do |x|
+  #     puts "Hello, #{x}"
+  #   end
+  #
+  #   foo = lambda do |x|
+  #     x * 100
+  #   end
+  #
+  # Here, it is impossible to tell from the return value of `lambda` whether
+  # the inner block's return value is significant.
+  - lambda
+  - proc
+  - it
 
 Style/BracesAroundHashParameters:
   Enabled: true
@@ -587,7 +587,10 @@ Style/InfiniteLoop:
 Style/InlineComment:
   Enabled: true
 
-Style/MethodMissing:
+Style/MethodMissingSuper:
+  Enabled: true
+
+Style/MissingRespondToMissing:
   Enabled: true
 
 Style/MinMax:
@@ -677,4 +680,4 @@ Gemspec/DuplicatedAssignment:
   Enabled: true
 
 Gemspec/OrderedDependencies:
-  Enabled: true
+  Enabled: false

data/config/rails.yml

@@ -152,7 +152,7 @@ Sequioacap/SimpleModifierConditional:
 
 # Exclude Rails ERB files from incompatible cops
 
-Lint/BlockAlignment:
+Layout/BlockAlignment:
   Exclude:
     - 'app/views/**/*.erb'
 

data/lib/rubocop/cop/sequioacap/class_name.rb

@@ -16,9 +16,9 @@ module RuboCop
         # class name is a hardcoded string. If not, add an offense and return true.
         def on_send(node)
           association_statement =
-            node.command?(:has_many) ||
-            node.command?(:has_one) ||
-            node.command?(:belongs_to)
+              node.command?(:has_many) ||
+                  node.command?(:has_one) ||
+                  node.command?(:belongs_to)
 
           return unless association_statement
 
@@ -36,8 +36,8 @@ module RuboCop
         def class_name_node(node)
           node.descendants.detect do |e|
             e.is_a?(Parser::AST::Node) &&
-              e.pair_type? &&
-              e.children[0].children[0] == :class_name
+                e.pair_type? &&
+                e.children[0].children[0] == :class_name
           end
         end
 
@@ -48,4 +48,4 @@ module RuboCop
       end
     end
   end
-  end
+end

data/lib/rubocop/cop/sequioacap/default_scope.rb

@@ -21,4 +21,4 @@ module RuboCop
       end
     end
   end
-  end
+end

data/lib/rubocop/cop/sequioacap/factory_class_use_string.rb

@@ -28,8 +28,8 @@ module RuboCop
         def class_node(node)
           node.descendants.detect do |e|
             e.is_a?(Parser::AST::Node) &&
-              e.pair_type? &&
-              e.children[0].children[0] == :class
+                e.pair_type? &&
+                e.children[0].children[0] == :class
           end
         end
 
@@ -40,4 +40,4 @@ module RuboCop
       end
     end
   end
-  end
+end

data/lib/rubocop/cop/sequioacap/no_timeout.rb

@@ -7,7 +7,7 @@ module RuboCop
     module Sequioacap
       class NoTimeout < Cop
         MSG =
-          'Do not use Timeout.timeout. In combination with Rails autoloading, ' \
+            'Do not use Timeout.timeout. In combination with Rails autoloading, ' \
           'timeout can cause Segmentation Faults in version of Ruby we use. ' \
           'It can also cause logic errors since it can raise in ' \
           'any callee scope. Use client library timeouts and monitoring to ' \
@@ -20,4 +20,4 @@ module RuboCop
       end
     end
   end
-  end
+end

data/lib/rubocop/cop/sequioacap/simple_modifier_conditional.rb

@@ -22,4 +22,4 @@ module RuboCop
       end
     end
   end
-  end
+end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: rubocop-scc
 version: !ruby/object:Gem::Version
-  version: 0.2.1
+  version: 0.2.2
 platform: ruby
 authors:
 - borisding
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2018-03-13 00:00:00.000000000 Z
+date: 2018-12-11 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rubocop
@@ -16,28 +16,34 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.53'
+        version: 0.61.1
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.53'
+        version: 0.61.1
 - !ruby/object:Gem::Dependency
   name: actionview
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '5.1'
+        version: '5.2'
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 5.2.2
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '5.1'
+        version: '5.2'
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 5.2.2
 - !ruby/object:Gem::Dependency
   name: erubis
   requirement: !ruby/object:Gem::Requirement
@@ -59,6 +65,9 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '5.11'
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 5.11.3
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
@@ -66,6 +75,9 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '5.11'
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 5.11.3
 - !ruby/object:Gem::Dependency
   name: rake
   requirement: !ruby/object:Gem::Requirement
@@ -73,6 +85,9 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '12.3'
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 12.3.2
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
@@ -80,6 +95,9 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '12.3'
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 12.3.2
 description: Code style checking for Sequoia Capital China Ruby projects
 email: lding@sequoiacap.com
 executables: []
@@ -124,7 +142,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.7.3
+rubygems_version: 2.7.8
 signing_key: 
 specification_version: 4
 summary: RuboCop SCC