rubocop-sketchup 0.9.0 → 0.10.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 8f41cde091e8436d91f8401f489bda6daed828f7dbef89f645620f325b6a188d
-  data.tar.gz: 8f13f01611e868c0b4bb3ad904d5679de3c4aa7c38869d42e38cddab05971b8f
+  metadata.gz: f191f3c6933aecafd8276d46cf582094e4562b7d1a9d7e2c62031351d83c7318
+  data.tar.gz: '0854e6c51bfec535cf5c626fdf5f44d335ef392daa979e56bfdff9fc0deb2a8e'
 SHA512:
-  metadata.gz: c7dd283a2ff010bafbea36e15c57c58f3ce72e17787da27e5b4e80e7dc0b545e81ae55edecad18d91be0e50d54130c90bf8bdeef03834168627608ff96368a53
-  data.tar.gz: 5007bf166f68996827c4f9c0782ca0214cfc36a669e03c3573a8c15912a331ed088e759e434b89a0223897c0134d0e042923395e7dcfa51cd39cb142d329fba4
+  metadata.gz: 4f2755cbefe2667ef8d2888c269f16561109b6c848bb7ce089cbe0d03273e801bc68862f773e5e95be170734bc5c28b71b9848d602a9ce89fe6bd0cc98b74abf
+  data.tar.gz: 62f91f9d6adf7da13fdd3da9ab0f1f75145c7d8172d6c5cc21217352df3d3a875e6e450a825ac46632a594a4cdaa200771475f5c968a25f08df58c21dec1462d

data/Gemfile

@@ -12,9 +12,9 @@ end
 
 group :development do
   gem 'bump', '~> 0.5', require: false
-  gem 'nokogiri', '>= 1.8', '< 1.10', require: false # 1.10 require Ruby 2.3
+  gem 'rubocop-performance', require: false
   gem 'simplecov', '~> 0.10', require: false
-  gem 'solargraph', '~> 0.31', require: false
+  gem 'solargraph', '~> 0.34', require: false
   gem 'thor', '~> 0.20', require: false
   gem 'yard', '~> 0.9', require: false
 end

data/lib/rubocop-sketchup.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'rubocop'
 require 'rubocop/sketchup'
 require 'rubocop/sketchup/version'

data/lib/rubocop/sketchup.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'pathname'
 
 module RuboCop

data/lib/rubocop/sketchup/cop/bugs/material_name.rb

@@ -79,7 +79,7 @@ module RuboCop
         PATTERN
 
         MSG_SET_NAME = '`material.name=` might add duplicate materials in '\
-            'SU2017 and older versions.'.freeze
+            'SU2017 and older versions.'
 
         def on_send(node)
           return unless valid_for_target_sketchup_version?

data/lib/rubocop/sketchup/cop/bugs/render_mode.rb

@@ -43,10 +43,10 @@ module RuboCop
         PATTERN
 
         MSG_OBSOLETE = 'Obsolete render mode will crash in SU2017 and '\
-            'newer versions.'.freeze
+            'newer versions.'
 
         MSG_INVALID = 'Invalid render mode will crash in SU2017 and '\
-            'newer versions.'.freeze
+            'newer versions.'
 
         def on_send(node)
           return unless valid_for_target_sketchup_version?

data/lib/rubocop/sketchup/cop/bugs/uniform_scaling.rb

@@ -21,7 +21,7 @@ module RuboCop
         PATTERN
 
         MSG = 'Resulting transformation matrix might yield unexpected '\
-            'results.'.freeze
+            'results.'
 
         def on_send(node)
           return unless valid_for_target_sketchup_version?

data/lib/rubocop/sketchup/cop/deprecations/add_separator_to_menu.rb

@@ -7,7 +7,7 @@ module RuboCop
       # a sub-menu instead.
       class AddSeparatorToMenu < SketchUp::Cop
 
-        MSG = 'Method is deprecated.'.freeze
+        MSG = 'Method is deprecated.'
 
         def_node_matcher :add_separator_to_menu?, <<-PATTERN
           (send nil? :add_separator_to_menu _)

data/lib/rubocop/sketchup/cop/deprecations/operation_next_transparent.rb

@@ -13,7 +13,7 @@ module RuboCop
       # Only use this flag if you have no other option, for instance to work
       # around bug in how `Sketchup::Model#place_component` starts operations.
       class OperationNextTransparent < SketchUp::Cop
-        MSG = 'Third argument is deprecated.'.freeze
+        MSG = 'Third argument is deprecated.'
 
         def on_send(node)
           _, method_name, *args = *node

data/lib/rubocop/sketchup/cop/deprecations/require_all.rb

@@ -9,7 +9,7 @@ module RuboCop
       class RequireAll < SketchUp::Cop
 
         MSG = 'Method is deprecated because it adds the given path '\
-              'to $LOAD_PATH.'.freeze
+              'to $LOAD_PATH.'
 
         def_node_matcher :require_all?, <<-PATTERN
           (send nil? :require_all _)

data/lib/rubocop/sketchup/cop/deprecations/set_texture_projection.rb

@@ -8,7 +8,7 @@ module RuboCop
       # repaired. Once repaired the UV mapping will visually change.
       class SetTextureProjection < SketchUp::Cop
 
-        MSG = 'Method is deprecated. It can create invalid UV mapping.'.freeze
+        MSG = 'Method is deprecated. It can create invalid UV mapping.'
 
         def_node_matcher :set_texture_projection?, <<-PATTERN
           (send _ :set_texture_projection ...)

data/lib/rubocop/sketchup/cop/deprecations/show_ruby_panel.rb

@@ -7,7 +7,7 @@ module RuboCop
       class ShowRubyPanel < SketchUp::Cop
 
         MSG = 'Method is deprecated. Use `SKETCHUP_CONSOLE.show` '\
-               'instead.'.freeze
+               'instead.'
 
         def_node_matcher :show_ruby_panel?, <<-PATTERN
           (send nil? :show_ruby_panel)

data/lib/rubocop/sketchup/cop/deprecations/sketchup_set.rb

@@ -12,7 +12,7 @@ module RuboCop
       # and less versatile.
       class SketchupSet < SketchUp::Cop
 
-        MSG = 'Class is deprecated.'.freeze
+        MSG = 'Class is deprecated.'
 
         def_node_matcher :sketchup_set?, <<-PATTERN
           (const (const nil? :Sketchup) :Set)

data/lib/rubocop/sketchup/cop/performance/openssl.rb

@@ -16,7 +16,7 @@ module RuboCop
       class OpenSSL < SketchUp::Cop
 
         MSG = 'Avoid use of OpenSSL within SketchUp due to severe performance '\
-              'issues.'.freeze
+              'issues.'
 
         # http://www.rubydoc.info/gems/rubocop/RuboCop/NodePattern
         # https://rubocop.readthedocs.io/en/latest/node_pattern/

data/lib/rubocop/sketchup/cop/performance/operation_disable_ui.rb

@@ -11,7 +11,7 @@ module RuboCop
       #   # <model changes>
       #   model.commit_operation
       class OperationDisableUI < SketchUp::Cop
-        MSG = 'Operations should disable the UI for performance gain.'.freeze
+        MSG = 'Operations should disable the UI for performance gain.'
 
         def on_send(node)
           _, method_name, *args = *node

data/lib/rubocop/sketchup/cop/performance/selection_bulk.rb

@@ -29,7 +29,7 @@ module RuboCop
         include RangeHelp
 
         MSG = 'Prefer changing selection in bulk instead of modifying '\
-              'selection within loops.'.freeze
+              'selection within loops.'
 
         # http://www.rubydoc.info/gems/rubocop/RuboCop/NodePattern
         # https://rubocop.readthedocs.io/en/latest/node_pattern/

data/lib/rubocop/sketchup/cop/performance/type_check.rb

@@ -31,7 +31,7 @@ module RuboCop
         include RangeHelp
 
         MSG = 'String comparisons are very slow, prefer `.is_a?` '\
-              'instead.'.freeze
+              'instead.'
 
         def_node_matcher :string_class_compare?, <<-PATTERN
           (send

data/lib/rubocop/sketchup/cop/performance/typename.rb

@@ -8,7 +8,7 @@ module RuboCop
       # `entity.typename == 'Face'` is slow because it performs a string
       # comparison. `is_a?` is much faster because it's a simple type check.
       class Typename < SketchUp::Cop
-        MSG = '`.typename` is very slow, prefer `.is_a?` instead.'.freeze
+        MSG = '`.typename` is very slow, prefer `.is_a?` instead.'
 
         def on_send(node)
           _, method_name = *node

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

@@ -14,7 +14,7 @@ module RuboCop
         include SketchUp::NoCommentDisable
         include SketchUp::NamespaceChecker
 
-        MSG = 'Do not modify the SketchUp API.'.freeze
+        MSG = 'Do not modify the SketchUp API.'
 
         NAMESPACES = %w[
           Geom Layout Sketchup SketchupExtension UI

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

@@ -14,7 +14,7 @@ module RuboCop
         include SketchUp::NoCommentDisable
 
         MSG = '`exit` attempts to kill the Ruby interpreter. Use `return`, '\
-              '`next`, `break` or `raise` instead.'.freeze
+              '`next`, `break` or `raise` instead.'
 
         # Reference: http://rubocop.readthedocs.io/en/latest/development/
         def_node_matcher :exit?, <<-PATTERN

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

@@ -88,7 +88,7 @@ module RuboCop
           namespace = Namespace.new(node.defined_module_name).from_root
           format('Use a single root namespace. '\
                  '(Found `%<found>s`; Previously found `%<expected>s`)',
-              found: namespace, expected: @@namespace)
+                 found: namespace, expected: @@namespace)
         end
 
         def exempted?(namespace)

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

@@ -40,8 +40,8 @@ module RuboCop
             msg = 'Extensions must have exactly one root Ruby (.rb) file. '\
                   'Found: %d'
             add_offense(nil,
-                location: range,
-                message: format(msg, root_ruby_files.size))
+                        location: range,
+                        message: format(msg, root_ruby_files.size))
             return
           end
 
@@ -57,8 +57,8 @@ module RuboCop
           if sub_folders.size != 1
             msg = 'Extensions must have exactly one support directory. Found %d'
             add_offense(nil,
-                location: range,
-                message: format(msg, sub_folders.size))
+                        location: range,
+                        message: format(msg, sub_folders.size))
             return
           end
 
@@ -69,8 +69,8 @@ module RuboCop
                   'of the root Ruby file. Expected %s, found %s'
             msg = format(msg, extension_basename, support_directory.basename)
             add_offense(nil,
-                location: range,
-                message: msg)
+                        location: range,
+                        message: msg)
           end
         end
 

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

@@ -26,7 +26,7 @@ module RuboCop
         include SketchUp::NoCommentDisable
 
         MSG = '`Gem.install` is unreliable in SketchUp, and can cause '\
-              'extensions to clash.'.freeze
+              'extensions to clash.'
 
         # Reference: http://rubocop.readthedocs.io/en/latest/development/
         def_node_matcher :gem_install?, <<-PATTERN

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

@@ -14,12 +14,12 @@ module RuboCop
         include RangeHelp
         include SketchUp::NoCommentDisable
 
-        MSG_INVALID = 'Invalid extension GUID'.freeze
+        MSG_INVALID = 'Invalid extension GUID'
 
         MSG_WRONG_TYPE = 'Only pass in extension GUID from local string '\
-                         'literals.'.freeze
+                         'literals.'
 
-        MSG_TRAILING_SPACE = 'Extra space in extension GUID'.freeze
+        MSG_TRAILING_SPACE = 'Extra space in extension GUID'
 
         def_node_matcher :get_extension_license, <<-PATTERN
           (send

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

@@ -14,7 +14,7 @@ module RuboCop
         include SketchUp::NoCommentDisable
         include SketchUp
 
-        MSG = 'Do not introduce global constants.'.freeze
+        MSG = 'Do not introduce global constants.'
 
         def_node_matcher :namespaced_constant?, <<-PATTERN
           (casgn

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

@@ -14,7 +14,7 @@ module RuboCop
         include SketchUp::NoCommentDisable
         include SketchUp
 
-        MSG = 'Do not include into global namespace.'.freeze
+        MSG = 'Do not include into global namespace.'
 
         def_node_matcher :is_include?, <<-PATTERN
           (send nil? :include ...)

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

@@ -14,7 +14,7 @@ module RuboCop
         include SketchUp::NoCommentDisable
         include SketchUp
 
-        MSG = 'Do not introduce global methods.'.freeze
+        MSG = 'Do not introduce global methods.'
 
         # Reference: http://www.rubydoc.info/gems/rubocop/RuboCop/NodePattern
         #

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

@@ -21,7 +21,7 @@ module RuboCop
         include SketchUp::NoCommentDisable
         include SketchUp::DynamicComponentGlobals
 
-        MSG = 'Do not introduce global variables.'.freeze
+        MSG = 'Do not introduce global variables.'
 
         # predefined global variables their English aliases
         # http://www.zenspider.com/Languages/Ruby/QuickRef.html

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

@@ -11,7 +11,7 @@ module RuboCop
         include SketchUp::NoCommentDisable
 
         MSG = 'Avoid globals in general, but especially these which are known '\
-              'to be in use.'.freeze
+              'to be in use.'
 
         LH_GLOBALS = %i[
           $dc_strings

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

@@ -9,7 +9,7 @@ module RuboCop
 
         include SketchUp::NoCommentDisable
 
-        MSG = 'Do not modify the load path.'.freeze
+        MSG = 'Do not modify the load path.'
 
         def_node_matcher :load_path_mutator?, <<-PATTERN
           (send

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

@@ -33,7 +33,7 @@ module RuboCop
         include SketchUp::ExtensionProject
 
         MSG = "Don't load extension files in the root file registering the "\
-              'extension.'.freeze
+              'extension.'
 
         # Reference: http://rubocop.readthedocs.io/en/latest/node_pattern/
         def_node_matcher :require_filename, <<-PATTERN

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

@@ -32,7 +32,7 @@ module RuboCop
         include SketchUp::NoCommentDisable
         include RangeHelp
 
-        MSG = 'Observers should create transparent operations.'.freeze
+        MSG = 'Observers should create transparent operations.'
 
         def_node_search :start_operation, <<-PATTERN
           (send

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

@@ -18,7 +18,7 @@ module RuboCop
 
         include SketchUp::NoCommentDisable
 
-        MSG = 'Always register extensions to load by default.'.freeze
+        MSG = 'Always register extensions to load by default.'
 
         def_node_search :sketchup_register_extension, <<-PATTERN
           (send

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

@@ -14,7 +14,7 @@ module RuboCop
         include SketchUp::NoCommentDisable
         include SketchUp::NamespaceChecker
 
-        MSG = 'Do not modify Ruby core functionality.'.freeze
+        MSG = 'Do not modify Ruby core functionality.'
 
         # We check only against the top level namespaces. The core define more
         # objects, but they are under one of the top level namespaces listed.

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

@@ -14,7 +14,7 @@ module RuboCop
         include SketchUp::NoCommentDisable
         include SketchUp::NamespaceChecker
 
-        MSG = 'Do not modify Ruby stdlib functionality.'.freeze
+        MSG = 'Do not modify Ruby stdlib functionality.'
 
         # We check only against the top level namespaces. The stdlib define more
         # objects, but they are under one of the top level namespaces listed.

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

@@ -9,7 +9,7 @@ module RuboCop
         include SketchUp::NoCommentDisable
         include SketchUp::NamespaceChecker
 
-        MSG = 'Do not modify shipped extensions.'.freeze
+        MSG = 'Do not modify shipped extensions.'
 
         # We check only against the top level namespaces. The core define more
         # objects, but they are under one of the top level namespaces listed.

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

@@ -21,11 +21,11 @@ module RuboCop
         include RangeHelp
 
         # rubocop:disable Metrics/LineLength
-        MSG = 'Create and register one SketchupExtension instance per extension.'.freeze
-        MSG_CREATE_ONE = 'Create only SketchupExtension instance per extension.'.freeze
-        MSG_CREATE_MISSING = 'SketchupExtension.new not found.'.freeze
-        MSG_REGISTER_ONE = 'Only register one SketchupExtension instance per extension.'.freeze
-        MSG_REGISTER_MISSING = 'Registration of SketchupExtension not found. Expected %s'.freeze
+        MSG = 'Create and register one SketchupExtension instance per extension.'
+        MSG_CREATE_ONE = 'Create only SketchupExtension instance per extension.'
+        MSG_CREATE_MISSING = 'SketchupExtension.new not found.'
+        MSG_REGISTER_ONE = 'Only register one SketchupExtension instance per extension.'
+        MSG_REGISTER_MISSING = 'Registration of SketchupExtension not found. Expected %s'
         # rubocop:enable Metrics/LineLength
 
         # Reference: http://rubocop.readthedocs.io/en/latest/node_pattern/
@@ -53,14 +53,14 @@ module RuboCop
 
           # Threat instances not assigned to anything as non-existing.
           extension_nodes.select! { |node|
-            node.parent && node.parent.assignment?
+            node.parent&.assignment?
           }
 
           # There should not be multiple instances.
           if extension_nodes.size > 1
             add_offense(nil,
-                location: range,
-                message: MSG_CREATE_ONE)
+                        location: range,
+                        message: MSG_CREATE_ONE)
             return
           end
 
@@ -68,8 +68,8 @@ module RuboCop
           extension_node = extension_nodes.first
           if extension_node.nil?
             add_offense(nil,
-                location: range,
-                message: MSG_CREATE_MISSING)
+                        location: range,
+                        message: MSG_CREATE_MISSING)
             return
           end
 
@@ -81,7 +81,7 @@ module RuboCop
                         'Missing required name arguments'
                       end
             add_offense(extension_node,
-                message: message)
+                        message: message)
             return
           end
 
@@ -100,7 +100,7 @@ module RuboCop
           # Make sure there is only one call to `register_extension`.
           if registered_vars.size > 1
             add_offense(registered_vars[1],
-                message: MSG_REGISTER_ONE)
+                        message: MSG_REGISTER_ONE)
             return
           end
 
@@ -108,8 +108,8 @@ module RuboCop
           unless registered_var == extension_var
             msg = MSG_REGISTER_MISSING % extension_var.to_s
             add_offense(nil,
-                location: range,
-                message: msg)
+                        location: range,
+                        message: msg)
           end
         end
 

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

@@ -28,19 +28,19 @@ module RuboCop
         include RangeHelp
 
         MSG_SKETCHUP_REQUIRE_EXT_NAME = 'Do not hard code file extensions '\
-                                        'with `Sketchup.require`.'.freeze
+                                        'with `Sketchup.require`.'
 
         MSG_EXTENSION_NEW_EXT_NAME = 'Do not hard code file extensions '\
-                                      'with `SketchupExtension.new`.'.freeze
+                                      'with `SketchupExtension.new`.'
 
         MSG_REQUIRE_FOR_BINARY = 'Use `require` instead of `Sketchup.require` '\
-                                 'to load binary Ruby libraries.'.freeze
+                                 'to load binary Ruby libraries.'
 
         MSG_OMIT_BINARY_EXT = 'Do not hard code .so/.bundle file '\
-                              'extensions'.freeze
+                              'extensions'
 
         MSG_REQUIRE_ENCRYPTED = 'Use `Sketchup.require` when loading Ruby '\
-                                'files for encrypted extensions.'.freeze
+                                'files for encrypted extensions.'
 
         def_node_matcher :ruby_require, <<-PATTERN
           (send nil? :require (str $_))

data/lib/rubocop/sketchup/cop/suggestions/add_group.rb

@@ -30,7 +30,7 @@ module RuboCop
 
         include RangeHelp
 
-        MSG = 'Avoid creating groups out of existing entities.'.freeze
+        MSG = 'Avoid creating groups out of existing entities.'
 
         def_node_matcher :add_group?, <<-PATTERN
           (send _ :add_group ...)

data/lib/rubocop/sketchup/cop/suggestions/compatibility.rb

@@ -16,7 +16,7 @@ module RuboCop
         include SketchUp::Features
         include SketchUp
 
-        MSG = 'Incompatible feature with target SketchUp version'.freeze
+        MSG = 'Incompatible feature with target SketchUp version'
 
         def on_def(node)
           return unless observer_method?(node)
@@ -43,7 +43,7 @@ module RuboCop
         end
 
         def on_const(node)
-          if node.parent && node.parent.module_definition?
+          if node.parent&.module_definition?
             # This catches definition of classes and modules.
             namespace = Namespace.new(node.parent_module_name)
             return unless namespace.top_level?
@@ -103,7 +103,7 @@ module RuboCop
         end
 
         def module_method?(node)
-          node.receiver && node.receiver.const_type?
+          node.receiver&.const_type?
         end
 
         def instance_method?(feature_name)

data/lib/rubocop/sketchup/cop/suggestions/dc_internals.rb

@@ -11,7 +11,7 @@ module RuboCop
 
         include SketchUp::DynamicComponentGlobals
 
-        MSG = 'Avoid relying on internal logic of Dynamic Components.'.freeze
+        MSG = 'Avoid relying on internal logic of Dynamic Components.'
 
         def on_gvar(node)
           check_global(node)

data/lib/rubocop/sketchup/cop/suggestions/file_encoding.rb

@@ -17,7 +17,7 @@ module RuboCop
       #   basename = File.basename(file, '.*')
       class FileEncoding < SketchUp::Cop
 
-        MSG = 'Beware encoding bug with `__FILE__` and `__dir__`.'.freeze
+        MSG = 'Beware encoding bug with `__FILE__` and `__dir__`.'
 
         def_node_matcher :file_loaded?, <<-PATTERN
           (send nil? {:file_loaded? :file_loaded} ...)

data/lib/rubocop/sketchup/cop/suggestions/model_entities.rb

@@ -11,7 +11,7 @@ module RuboCop
       # have an explicit reason to work in the root model context.
       class ModelEntities < SketchUp::Cop
 
-        MSG = 'Prefer `model.active_entities` over `model.entities`.'.freeze
+        MSG = 'Prefer `model.active_entities` over `model.entities`.'
 
         # Reference: http://www.rubydoc.info/gems/rubocop/RuboCop/NodePattern
         def_node_matcher :active_model_entities?, <<-PATTERN

data/lib/rubocop/sketchup/cop/suggestions/monkey_patched_api.rb

@@ -14,7 +14,7 @@ module RuboCop
 
         def on_send(node)
           # Only check instance methods.
-          return if node.receiver && node.receiver.const_type?
+          return if node.receiver&.const_type?
 
           name = node.method_name
 
@@ -22,7 +22,7 @@ module RuboCop
           return unless dc_method
 
           if dc_method.key?(:variables)
-            return unless node.receiver && node.receiver.variable?
+            return unless node.receiver&.variable?
 
             receiver_name = node.receiver.children.first
             # Account for instance and class variables.
@@ -34,9 +34,9 @@ module RuboCop
           message = "#{path}##{name} is not part of the official API. "\
                     "It's a monkey-patched addition by Dynamic Components."
           add_offense(node,
-              location: :selector,
-              severity: :warning,
-              message: message)
+                      location: :selector,
+                      severity: :warning,
+                      message: message)
         end
 
       end

data/lib/rubocop/sketchup/cop/suggestions/operation_name.rb

@@ -48,8 +48,8 @@ module RuboCop
 
         include RangeHelp
 
-        MSG = 'Operation name should be a short capitalized description.'.freeze
-        MSG_MAX = 'Operation names should not be short and concise. [%d/%d]'.freeze
+        MSG = 'Operation name should be a short capitalized description.'
+        MSG_MAX = 'Operation names should not be short and concise. [%d/%d]'
 
         def on_send(node)
           _, method_name, *args = *node
@@ -72,8 +72,8 @@ module RuboCop
           unless operation_name.size <= max_operation_name_length
             message = format(MSG_MAX, operation_name.size, max_operation_name_length)
             add_offense(args.first,
-                location: excess_range(args.first, operation_name),
-                message: message)
+                        location: excess_range(args.first, operation_name),
+                        message: message)
           end
           # Ensure operation name is not empty.
           if operation_name.empty?

data/lib/rubocop/sketchup/cop/suggestions/sketchup_find_support_file.rb

@@ -16,7 +16,7 @@ module RuboCop
       class SketchupFindSupportFile < SketchUp::Cop
 
         MSG = 'Avoid `Sketchup.find_support_file` to find your '\
-              "extension's files.".freeze
+              "extension's files."
 
         # http://www.rubydoc.info/gems/rubocop/RuboCop/NodePattern
         # https://rubocop.readthedocs.io/en/latest/node_pattern/

data/lib/rubocop/sketchup/cop/suggestions/tool_drawing_bounds.rb

@@ -29,7 +29,7 @@ module RuboCop
         include SketchUp::ToolChecker
 
         MSG_MISSING_GET_EXTENTS = 'When drawing to the viewport implement '\
-            '`getExtents` so drawn geometry is not clipped.'.freeze
+            '`getExtents` so drawn geometry is not clipped.'
 
         def on_tool_class(class_node, body_methods)
           return unless find_method(body_methods, :draw)

data/lib/rubocop/sketchup/cop/suggestions/tool_invalidate.rb

@@ -31,10 +31,10 @@ module RuboCop
         include SketchUp::ToolChecker
 
         MSG_MISSING_INVALIDATE_METHOD = 'When drawing to the viewport, make '\
-            'sure to `suspend` and `deactivate` calls `view.invalidate`.'.freeze
+            'sure to `suspend` and `deactivate` calls `view.invalidate`.'
 
         MSG_MISSING_INVALIDATE = 'When drawing to the viewport, make sure to '\
-            'call `view.invalidate` when the tool becomes inactive.'.freeze
+            'call `view.invalidate` when the tool becomes inactive.'
 
         def_node_search :view_invalidate?, <<-PATTERN
           (send (lvar :view) :invalidate ...)

data/lib/rubocop/sketchup/cop/suggestions/tool_user_input.rb

@@ -24,7 +24,7 @@ module RuboCop
         include SketchUp::ToolChecker
 
         MSG_MISSING_ENABLE_VCB = 'When accepting user input, make sure to '\
-            'define `enableVCB?`.'.freeze
+            'define `enableVCB?`.'
 
         def on_tool_class(class_node, body_methods)
           return unless find_method(body_methods, :onUserText)

data/lib/rubocop/sketchup/cop/suggestions/toolbar_timer.rb

@@ -25,7 +25,7 @@ module RuboCop
         include RangeHelp
 
         MSG = 'Wrapping `toolbar.restore` in `UI.start_timer` is '\
-              'redundant.'.freeze
+              'redundant.'
 
         def_node_matcher :toolbar_new?, <<-PATTERN
           (send (const (const nil? :UI) :Toolbar) :new _)
@@ -50,7 +50,7 @@ module RuboCop
           toolbar_variable_name = assignment_node.children.first
 
           receiver = ui_start_timer_restore(assignment_node.parent).first
-          return unless receiver && receiver.variable?
+          return unless receiver&.variable?
 
           receiver_variable_name = receiver.children.first
 

data/lib/rubocop/sketchup/formatter/extension_review.rb

@@ -10,7 +10,7 @@ module RuboCop
   module Formatter
     # This formatter saves the output as an html file.
     class ExtensionReviewFormatter < BaseFormatter
-      ELLIPSES = '<span class="extra-code">...</span>'.freeze
+      ELLIPSES = '<span class="extra-code">...</span>'
       TEMPLATE_PATH =
           File.expand_path('../../../../assets/output.html.erb', __dir__)
 

data/lib/rubocop/sketchup/namespace.rb

@@ -6,7 +6,7 @@ module RuboCop
 
       attr_reader :namespace
 
-      SEPARATOR = '::'.freeze
+      SEPARATOR = '::'
 
       # @param [String] namespace
       def initialize(namespace)

data/lib/rubocop/sketchup/range_help.rb

@@ -33,7 +33,7 @@ module RuboCop
       def conditional_range(node)
         if node.modifier_form?
           range_between(node.loc.keyword.begin_pos,
-              node.loc.expression.end_pos)
+                        node.loc.expression.end_pos)
         else
           :expression
         end

data/lib/rubocop/sketchup/version.rb

@@ -1,5 +1,7 @@
+# frozen_string_literal: true
+
 module RuboCop
   module SketchUp
-    VERSION = '0.9.0'.freeze
+    VERSION = '0.10.0'
   end
 end

data/rubocop-sketchup.gemspec

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 $LOAD_PATH.unshift File.expand_path('lib', __dir__)
 require 'rubocop/sketchup/version'
 
@@ -11,7 +13,7 @@ Gem::Specification.new do |spec|
 
   spec.version = RuboCop::SketchUp::VERSION
   spec.platform = Gem::Platform::RUBY
-  spec.required_ruby_version = '>= 2.2.0'
+  spec.required_ruby_version = '>= 2.3.0'
 
   spec.require_paths = ['lib']
   spec.files = Dir[
@@ -22,6 +24,6 @@ Gem::Specification.new do |spec|
       'Gemfile'
   ]
 
-  spec.add_dependency 'rubocop', '~> 0.63.0'
+  spec.add_dependency 'rubocop', '~> 0.69.0'
   spec.add_development_dependency 'bundler', '>= 1.13', '< 3.0'
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: rubocop-sketchup
 version: !ruby/object:Gem::Version
-  version: 0.9.0
+  version: 0.10.0
 platform: ruby
 authors:
 - Trimble Inc, SketchUp Team
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2019-08-15 00:00:00.000000000 Z
+date: 2019-08-28 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rubocop
@@ -16,14 +16,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.63.0
+        version: 0.69.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.63.0
+        version: 0.69.0
 - !ruby/object:Gem::Dependency
   name: bundler
   requirement: !ruby/object:Gem::Requirement
@@ -131,7 +131,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: 2.2.0
+      version: 2.3.0
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="