Layout/MultilineMethodDefinitionBraceLayout

This commit enables the Rubocop
Layout/MultilineMethodDefinitionBraceLayout cop and addresses all
offenses.

Author: mhashizume

.rubocop_todo.yml

@@ -22,13 +22,6 @@ I18n/GetText/DecorateStringFormattingUsingPercent:
 I18n/RailsI18n/DecorateString:
   Enabled: false
 
-# This cop supports safe auto-correction (--auto-correct).
-# Configuration parameters: EnforcedStyle.
-# SupportedStyles: symmetrical, new_line, same_line
-Layout/MultilineMethodDefinitionBraceLayout:
-  Exclude:
-    - 'lib/puppet/pal/pal_impl.rb'
-
 # This cop supports safe auto-correction (--auto-correct).
 # Configuration parameters: EnforcedStyle, IndentationWidth.
 # SupportedStyles: aligned, indented

lib/puppet/pal/pal_impl.rb

@@ -236,8 +236,7 @@ def self.in_tmp_environment(env_name,
       settings_hash: {},
       facts:         nil,
       variables:     {},
-      &block
-  )
+      &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')
@@ -288,8 +287,7 @@ def self.in_environment(env_name,
       envpath:       nil,
       facts:         nil,
       variables:     {},
-      &block
-  )
+      &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)