Layout/ClosingParenthesisIndentation

This commit enables the Rubocop Layout/ClosingParenthesisIndentation
cop.

Author: mhashizume

.rubocop_todo.yml

@@ -22,10 +22,6 @@ I18n/GetText/DecorateStringFormattingUsingPercent:
 I18n/RailsI18n/DecorateString:
   Enabled: false
 
-# This cop supports safe auto-correction (--auto-correct).
-Layout/ClosingParenthesisIndentation:
-  Enabled: false
-
 # This cop supports safe auto-correction (--auto-correct).
 # Configuration parameters: AllowForAlignment.
 Layout/CommentIndentation:

lib/puppet/application.rb

@@ -364,7 +364,7 @@ def initialize(command_line = Puppet::Util::CommandLine.new)
   def app_defaults
     Puppet::Settings.app_defaults_for_run_mode(self.class.run_mode).merge(
         :name => name
-    )
+      )
   end
 
   # Initialize application defaults. It's usually not necessary to override this method.

lib/puppet/defaults.rb

@@ -117,7 +117,7 @@ def self.initialize_default_settings!(settings)
         :desc     => "The name of the application, if we are running as one.  The
           default is essentially $0 without the path or `.rb`.",
     }
-  )
+    )
 
   settings.define_settings(:main,
     :logdir => {
@@ -1298,7 +1298,7 @@ def self.initialize_default_settings!(settings)
         :default    => nil,
         :desc       => "The address the agent should use to initiate requests.",
       },
-  )
+    )
 
   settings.define_settings(:environment,
     :manifest => {
@@ -2135,7 +2135,7 @@ def self.initialize_default_settings!(settings)
 
           For more info, see [the ENC documentation](https://puppet.com/docs/puppet/latest/nodes_external.html).",
     }
-    )
+  )
 
         settings.define_settings(
         :ldap,
@@ -2205,7 +2205,7 @@ def self.initialize_default_settings!(settings)
           have one already set.  Generally, it should be the 'ou=Hosts'
           branch under your main directory.",
     }
-  )
+      )
 
   settings.define_settings(:server,
     :storeconfigs => {

lib/puppet/ffi/windows/constants.rb

@@ -203,7 +203,7 @@ module Constants
         :TokenSecurityAttributes,
         :TokenIsRestricted,
         :MaxTokenInfoClass
-    )
+      )
 
     # Service error codes
     # https://docs.microsoft.com/en-us/windows/desktop/debug/system-error-codes--1000-1299-

lib/puppet/functions/dig.rb

@@ -46,7 +46,7 @@ def dig(data, *args)
             'SLICE_ERROR',
             {'walked_path' => walked_path, 'value_type' => t},
             'EXPECTED_COLLECTION'
-        )
+          )
         raise Puppet::ErrorWithData.new(error_data, msg)
       end
 
@@ -59,7 +59,7 @@ def dig(data, *args)
             'SLICE_ERROR',
             {'walked_path' => walked_path, 'index_type' => t},
             'EXPECTED_INTEGER_INDEX'
-        )
+          )
         raise Puppet::ErrorWithData.new(error_data, msg)
       end
       d[k]

lib/puppet/module_tool/applications/installer.rb

@@ -187,7 +187,7 @@ def run
                     :requested_name => name,
                     :requested_version => options[:version] || graph.dependencies[name].max.version.to_s,
                     :unsatisfied => unsatisfied
-            )
+                  )
           end
 
           unless forced?

lib/puppet/pal/pal_impl.rb

@@ -235,7 +235,7 @@ def self.in_tmp_environment(env_name,
       facts:         nil,
       variables:     {},
       &block
-    )
+  )
     assert_non_empty_string(env_name, _("temporary environment name"))
     # TRANSLATORS: do not translate variable name string in these assertions
     assert_optionally_empty_array(modulepath, 'modulepath')
@@ -249,7 +249,7 @@ def self.in_tmp_environment(env_name,
     in_environment_context(
       Puppet::Environments::Static.new(env), # The tmp env is the only known env
       env, facts, variables, &block
-      )
+    )
   end
 
   # Defines the context in which to perform puppet operations (evaluation, etc)
@@ -287,7 +287,7 @@ def self.in_environment(env_name,
       facts:         nil,
       variables:     {},
       &block
-    )
+  )
     # TRANSLATORS terms in the assertions below are names of terms in code
     assert_non_empty_string(env_name, 'env_name')
     assert_optionally_empty_array(modulepath, 'modulepath', true)

lib/puppet/parser/ast/resource.rb

@@ -52,7 +52,7 @@ def evaluate(scope)
           :source => scope.source,
           :scope => scope,
           :strict => true
-          )
+          ) # rubocop:disable Layout/ClosingParenthesisIndentation
 
           if resource.resource_type.is_a? Puppet::Resource::Type
             resource.resource_type.instantiate_resource(scope, resource)

lib/puppet/parser/functions/create_resources.rb

@@ -106,6 +106,6 @@
           :add    => false
         )
       end.compact
-      )
+    )
   end.flatten.compact
 end

lib/puppet/parser/functions/fail.rb

@@ -6,7 +6,7 @@
 Fail with a parse error. Any parameters will be stringified,
 concatenated, and passed to the exception-handler.
 DOC
-) do |vals|
+  ) do |vals|
     vals = vals.collect { |s| s.to_s }.join(" ") if vals.is_a? Array
     raise Puppet::ParseError, vals.to_s
 end