rubocop-sketchup 0.5.0 → 0.6.0

data/lib/rubocop/sketchup/cop/requirements/language_handler_globals.rb

@@ -1,46 +1,46 @@
-# frozen_string_literal: true
-
-module RuboCop
-  module Cop
-    module SketchupRequirements
-      # Avoid using globals in general, but especially these which are known to
-      # be in use by other extensions made by SketchUp.
-      # They are still in use due to compatibility reasons.
-      class LanguageHandlerGlobals < SketchUp::Cop
-
-        include SketchUp::NoCommentDisable
-
-        MSG = 'Avoid globals in general, but especially these which are known '\
-              'to be in use.'.freeze
-
-        LH_GLOBALS = %i[
-          $dc_strings
-          $devl_strings
-          $exStrings
-          $fs_strings
-          $make_pano_string
-          $oceanStrings
-          $sn_strings
-          $ssf_strings
-          $suStrings
-          $tStrings
-          $unitsStrings
-          $uStrings
-          $wt_strings
-        ].freeze
-
-        def hl_global_var?(global_var)
-          LH_GLOBALS.include?(global_var)
-        end
-
-        def on_gvasgn(node)
-          global_var, = *node
-          return unless hl_global_var?(global_var)
-
-          add_offense(node, location: :name)
-        end
-
-      end
-    end
-  end
-end
+# frozen_string_literal: true
+
+module RuboCop
+  module Cop
+    module SketchupRequirements
+      # Avoid using globals in general, but especially these which are known to
+      # be in use by other extensions made by SketchUp.
+      # They are still in use due to compatibility reasons.
+      class LanguageHandlerGlobals < SketchUp::Cop
+
+        include SketchUp::NoCommentDisable
+
+        MSG = 'Avoid globals in general, but especially these which are known '\
+              'to be in use.'.freeze
+
+        LH_GLOBALS = %i[
+          $dc_strings
+          $devl_strings
+          $exStrings
+          $fs_strings
+          $make_pano_string
+          $oceanStrings
+          $sn_strings
+          $ssf_strings
+          $suStrings
+          $tStrings
+          $unitsStrings
+          $uStrings
+          $wt_strings
+        ].freeze
+
+        def hl_global_var?(global_var)
+          LH_GLOBALS.include?(global_var)
+        end
+
+        def on_gvasgn(node)
+          global_var, = *node
+          return unless hl_global_var?(global_var)
+
+          add_offense(node, location: :name)
+        end
+
+      end
+    end
+  end
+end

data/lib/rubocop/sketchup/cop/requirements/load_path.rb

@@ -1,83 +1,83 @@
-# frozen_string_literal: true
-
-module RuboCop
-  module Cop
-    module SketchupRequirements
-      # Do not modify the load path. Modifying `$LOAD_PATH` is bad practice
-      # because it can cause extensions to inadvertently load the wrong file.
-      class LoadPath < SketchUp::Cop
-
-        include SketchUp::NoCommentDisable
-
-        MSG = 'Do not modify the load path.'.freeze
-
-        def_node_matcher :load_path_mutator?, <<-PATTERN
-          (send
-            (gvar #load_path?) $#mutator?
-            ...
-          )
-        PATTERN
-
-        LOAD_PATH_ALIASES = %i[
-          $: $LOAD_PATH
-        ].freeze
-
-        MUTATORS = %i[
-          <<
-          []=
-          clear
-          collect!
-          compact!
-          concat
-          delete
-          delete_at
-          delete_if
-          drop
-          drop_while
-          fill
-          flatten!
-          insert
-          keep_if
-          map!
-          pop
-          push
-          reject!
-          replace
-          reverse!
-          rotate!
-          select!
-          shift
-          shuffle!
-          slice!
-          sort!
-          sort_by!
-          uniq!
-          unshift
-        ].freeze
-
-        def load_path?(sym)
-          LOAD_PATH_ALIASES.include?(sym)
-        end
-
-        def mutator?(sym)
-          MUTATORS.include?(sym)
-        end
-
-        def on_gvasgn(node)
-          global_var, = *node
-          return unless load_path?(global_var)
-
-          add_offense(node, location: :expression)
-        end
-
-        def on_send(node)
-          method_name = load_path_mutator?(node)
-          return unless method_name
-
-          add_offense(node, location: :expression)
-        end
-
-      end
-    end
-  end
-end
+# frozen_string_literal: true
+
+module RuboCop
+  module Cop
+    module SketchupRequirements
+      # Do not modify the load path. Modifying `$LOAD_PATH` is bad practice
+      # because it can cause extensions to inadvertently load the wrong file.
+      class LoadPath < SketchUp::Cop
+
+        include SketchUp::NoCommentDisable
+
+        MSG = 'Do not modify the load path.'.freeze
+
+        def_node_matcher :load_path_mutator?, <<-PATTERN
+          (send
+            (gvar #load_path?) $#mutator?
+            ...
+          )
+        PATTERN
+
+        LOAD_PATH_ALIASES = %i[
+          $: $LOAD_PATH
+        ].freeze
+
+        MUTATORS = %i[
+          <<
+          []=
+          clear
+          collect!
+          compact!
+          concat
+          delete
+          delete_at
+          delete_if
+          drop
+          drop_while
+          fill
+          flatten!
+          insert
+          keep_if
+          map!
+          pop
+          push
+          reject!
+          replace
+          reverse!
+          rotate!
+          select!
+          shift
+          shuffle!
+          slice!
+          sort!
+          sort_by!
+          uniq!
+          unshift
+        ].freeze
+
+        def load_path?(sym)
+          LOAD_PATH_ALIASES.include?(sym)
+        end
+
+        def mutator?(sym)
+          MUTATORS.include?(sym)
+        end
+
+        def on_gvasgn(node)
+          global_var, = *node
+          return unless load_path?(global_var)
+
+          add_offense(node, location: :expression)
+        end
+
+        def on_send(node)
+          method_name = load_path_mutator?(node)
+          return unless method_name
+
+          add_offense(node, location: :expression)
+        end
+
+      end
+    end
+  end
+end

data/lib/rubocop/sketchup/cop/requirements/minimal_registration.rb

@@ -1,73 +1,73 @@
-# frozen_string_literal: true
-
-require 'pathname'
-
-module RuboCop
-  module Cop
-    module SketchupRequirements
-      # Don't load extension files in the root file registering the extension.
-      # Extensions should not load additional files when it's disabled.
-      #
-      # @example Bad - Extension will always load.
-      #   module Example
-      #     require 'example/main' # BAD! This will load even when extension
-      #                            #      is disabled.
-      #     unless file_loaded?(__FILE__)
-      #       extension = SketchupExtension.new('Hello World', 'example/main')
-      #       Sketchup.register_extension(extension, true)
-      #       file_loaded(__FILE__)
-      #     end
-      #   end
-      #
-      # @example Good - Extension doesn't load anything unless its enabled.
-      #   module Example
-      #     unless file_loaded?(__FILE__)
-      #       extension = SketchupExtension.new('Hello World', 'example/main')
-      #       Sketchup.register_extension(extension, true)
-      #       file_loaded(__FILE__)
-      #     end
-      #   end
-      class MinimalRegistration < SketchUp::Cop
-
-        include SketchUp::NoCommentDisable
-        include SketchUp::ExtensionProject
-
-        MSG = "Don't load extension files in the root file registering the "\
-              'extension.'.freeze
-
-        # Reference: http://rubocop.readthedocs.io/en/latest/node_pattern/
-        def_node_matcher :require_filename, <<-PATTERN
-          {
-            (send {(const nil? :Sketchup) nil?} {:require :load} (str $_))
-            (send nil? :require_relative (str $_))
-          }
-        PATTERN
-
-        def investigate(processed_source)
-          if root_file?(processed_source)
-            filename = processed_source.buffer.name
-            @extension_basename = File.basename(filename, '.*')
-          end
-        end
-
-        def extension_file?(filename)
-          return false unless filename.include?('/')
-
-          first_directory = filename.split('/').first
-          @extension_basename.casecmp(first_directory) == 0
-        end
-
-        def on_send(node)
-          return unless @extension_basename
-
-          filename = require_filename(node)
-          return if filename.nil?
-          return unless extension_file?(filename)
-
-          add_offense(node)
-        end
-
-      end
-    end
-  end
-end
+# frozen_string_literal: true
+
+require 'pathname'
+
+module RuboCop
+  module Cop
+    module SketchupRequirements
+      # Don't load extension files in the root file registering the extension.
+      # Extensions should not load additional files when it's disabled.
+      #
+      # @example Bad - Extension will always load.
+      #   module Example
+      #     require 'example/main' # BAD! This will load even when extension
+      #                            #      is disabled.
+      #     unless file_loaded?(__FILE__)
+      #       extension = SketchupExtension.new('Hello World', 'example/main')
+      #       Sketchup.register_extension(extension, true)
+      #       file_loaded(__FILE__)
+      #     end
+      #   end
+      #
+      # @example Good - Extension doesn't load anything unless its enabled.
+      #   module Example
+      #     unless file_loaded?(__FILE__)
+      #       extension = SketchupExtension.new('Hello World', 'example/main')
+      #       Sketchup.register_extension(extension, true)
+      #       file_loaded(__FILE__)
+      #     end
+      #   end
+      class MinimalRegistration < SketchUp::Cop
+
+        include SketchUp::NoCommentDisable
+        include SketchUp::ExtensionProject
+
+        MSG = "Don't load extension files in the root file registering the "\
+              'extension.'.freeze
+
+        # Reference: http://rubocop.readthedocs.io/en/latest/node_pattern/
+        def_node_matcher :require_filename, <<-PATTERN
+          {
+            (send {(const nil? :Sketchup) nil?} {:require :load} (str $_))
+            (send nil? :require_relative (str $_))
+          }
+        PATTERN
+
+        def investigate(processed_source)
+          if root_file?(processed_source)
+            filename = processed_source.buffer.name
+            @extension_basename = File.basename(filename, '.*')
+          end
+        end
+
+        def extension_file?(filename)
+          return false unless filename.include?('/')
+
+          first_directory = filename.split('/').first
+          @extension_basename.casecmp(first_directory) == 0
+        end
+
+        def on_send(node)
+          return unless @extension_basename
+
+          filename = require_filename(node)
+          return if filename.nil?
+          return unless extension_file?(filename)
+
+          add_offense(node)
+        end
+
+      end
+    end
+  end
+end