puppet 8.3.1-x86-mingw32 → 8.4.0-x86-mingw32

data/lib/puppet/file_system/windows.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 require_relative '../../puppet/file_system/posix'
 require_relative '../../puppet/util/windows'
 
@@ -11,6 +12,7 @@ class Puppet::FileSystem::Windows < Puppet::FileSystem::Posix
     # Ruby on Windows uses mode for setting file attributes like read-only and
     # archived, not for setting permissions like POSIX
     raise TypeError.new('mode must be specified as an Integer') if mode && !mode.is_a?(Numeric)
+
     ::File.open(path, options, nil, &block)
   end
 
@@ -60,6 +62,7 @@ class Puppet::FileSystem::Windows < Puppet::FileSystem::Posix
 
   def symlink?(path)
     return false if ! Puppet.features.manages_symlinks?
+
     Puppet::Util::Windows::File.symlink?(path)
   end
 
@@ -102,6 +105,7 @@ class Puppet::FileSystem::Windows < Puppet::FileSystem::Posix
     if ! Puppet.features.manages_symlinks?
       return Puppet::Util::Windows::File.stat(path)
     end
+
     Puppet::Util::Windows::File.lstat(path)
   end
 
@@ -144,7 +148,6 @@ class Puppet::FileSystem::Windows < Puppet::FileSystem::Posix
              raise ArgumentError, "#{mode} is invalid: Only modes 0644, 0640, 0660, and 0440 are allowed"
            end
 
-
     tempfile = Puppet::FileSystem::Uniquefile.new(Puppet::FileSystem.basename_string(path), Puppet::FileSystem.dir_string(path))
     begin
       tempdacl = Puppet::Util::Windows::AccessControlList.new
@@ -210,5 +213,4 @@ class Puppet::FileSystem::Windows < Puppet::FileSystem::Posix
       Puppet.warning _("The current user does not have the necessary permission to manage symlinks.")
     end
   end
-
 end

data/lib/puppet/file_system.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 module Puppet::FileSystem
   require_relative '../puppet/util'
   require_relative 'file_system/path_pattern'
@@ -9,8 +10,8 @@ module Puppet::FileSystem
 
   # create instance of the file system implementation to use for the current platform
   @impl = if Puppet::Util::Platform.windows?
-           require_relative 'file_system/windows'
-           Puppet::FileSystem::Windows
+            require_relative 'file_system/windows'
+            Puppet::FileSystem::Windows
           elsif Puppet::Util::Platform.jruby?
             require_relative 'file_system/jruby'
             Puppet::FileSystem::JRuby

data/lib/puppet/forge/cache.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 require 'uri'
 
 require_relative '../../puppet/forge'
@@ -8,7 +9,6 @@ class Puppet::Forge
   #
   # Provides methods for reading files from local cache, filesystem or network.
   class Cache
-
     # Instantiate new cache for the +repository+ instance.
     def initialize(repository, options = {})
       @repository = repository

data/lib/puppet/forge/errors.rb

@@ -1,11 +1,11 @@
 # frozen_string_literal: true
+
 require_relative '../../puppet/util/json'
 require_relative '../../puppet/error'
 require_relative '../../puppet/forge'
 
 # Puppet::Forge specific exceptions
 module Puppet::Forge::Errors
-
   # This exception is the parent for all Forge API errors
   class ForgeError < Puppet::Error
     # This is normally set by the child class, but if it is not this will
@@ -102,7 +102,6 @@ module Puppet::Forge::Errors
     #
     # @return [String] the multiline version of the error message
     def multiline
-
       message = []
       message << _('Request to Puppet Forge failed.')
       message << _('  The server being queried was %{uri}') % { uri: @uri }
@@ -111,5 +110,4 @@ module Puppet::Forge::Errors
       message.join("\n")
     end
   end
-
 end

data/lib/puppet/forge/repository.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 require_relative '../../puppet/ssl/openssl_loader'
 require 'digest/sha1'
 require 'uri'
@@ -30,6 +31,7 @@ class Puppet::Forge
     # Return a Net::HTTPResponse read for this +path+.
     def make_http_request(path, io = nil)
       raise ArgumentError, "Path must start with forward slash" unless path.start_with?('/')
+
       begin
         str = @uri.to_s
         str.chomp!('/')

data/lib/puppet/forge.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 require_relative '../puppet/vendor'
 Puppet::Vendor.load_vendored
 
@@ -195,6 +196,7 @@ class Puppet::Forge < SemanticPuppet::Dependency::Source
     # Obtain a suitable temporary path for unpacking tarballs
     #
     # @return [Pathname] path to temporary unpacking location
+    # rubocop:disable Naming/MemoizedInstanceVariableName
     def tmpdir
       @dir ||= Dir.mktmpdir(name, Puppet::Forge::Cache.base_path)
     end
@@ -204,6 +206,7 @@ class Puppet::Forge < SemanticPuppet::Dependency::Source
         f.binmode
       end
     end
+    # rubocop:enable Naming/MemoizedInstanceVariableName
 
     def download(uri, destination)
       response = @source.make_http_request(uri, destination)

data/lib/puppet/functions/abs.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 # Returns the absolute value of a Numeric value, for example -34.56 becomes
 # 34.56. Takes a single `Integer` or `Float` value as an argument.
 #
@@ -46,7 +47,7 @@ Puppet::Functions.create_function(:abs) do
 
   def on_string(x)
     Puppet.warn_once('deprecations', 'abs_function_numeric_coerce_string',
-      _("The abs() function's auto conversion of String to Numeric is deprecated - change to convert input before calling"))
+                     _("The abs() function's auto conversion of String to Numeric is deprecated - change to convert input before calling"))
 
     # These patterns for conversion are backwards compatible with the stdlib
     # version of this function.

data/lib/puppet/functions/alert.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 # Logs a message on the server at level `alert`.
 Puppet::Functions.create_function(:alert, Puppet::Functions::InternalFunction) do
   # @param values The values to log.

data/lib/puppet/functions/all.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 # Runs a [lambda](https://puppet.com/docs/puppet/latest/lang_lambdas.html)
 # repeatedly using each value in a data structure until the lambda returns a non "truthy" value which
 # makes the function return `false`, or if the end of the iteration is reached, `true` is returned.

data/lib/puppet/functions/annotate.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 # Handles annotations on objects. The function can be used in four different ways.
 #
 # With two arguments, an `Annotation` type and an object, the function returns the annotation

data/lib/puppet/functions/any.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 # Runs a [lambda](https://puppet.com/docs/puppet/latest/lang_lambdas.html)
 # repeatedly using each value in a data structure until the lambda returns a "truthy" value which
 # makes the function return `true`, or if the end of the iteration is reached, false is returned.

data/lib/puppet/functions/assert_type.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 # Returns the given value if it is of the given
 # [data type](https://puppet.com/docs/puppet/latest/lang_data.html), or
 # otherwise either raises an error or executes an optional two-parameter

data/lib/puppet/functions/binary_file.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 # Loads a binary file from a module or file system and returns its contents as a `Binary`.
 # The argument to this function should be a `<MODULE NAME>/<FILE>`
 # reference, which will load `<FILE>` from a module's `files`
@@ -25,9 +26,10 @@ Puppet::Functions.create_function(:binary_file, Puppet::Functions::InternalFunct
   def binary_file(scope, unresolved_path)
     path = Puppet::Parser::Files.find_file(unresolved_path, scope.compiler.environment)
     unless path && Puppet::FileSystem.exist?(path)
-      #TRANSLATORS the string "binary_file()" should not be translated
+      # TRANSLATORS the string "binary_file()" should not be translated
       raise Puppet::ParseError, _("binary_file(): The given file '%{unresolved_path}' does not exist") % { unresolved_path: unresolved_path }
     end
+
     Puppet::Pops::Types::PBinaryType::Binary.from_binary_string(Puppet::FileSystem.binread(path))
   end
 end

data/lib/puppet/functions/break.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 # Breaks an innermost iteration as if it encountered an end of input.
 # This function does not return to the caller.
 #
@@ -25,7 +26,7 @@
 # }
 # $data = [1,2,3]
 # notice $data.map |$x| { break_if_even($x); $x*10 }
-#```
+# ```
 # Would notice the value `[10]`
 #
 # * Also see functions `next` and `return`

data/lib/puppet/functions/call.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 # Calls an arbitrary Puppet function by name.
 #
 # This function takes one mandatory argument and one or more optional arguments:
@@ -77,5 +78,4 @@ Puppet::Functions.create_function(:call, Puppet::Functions::InternalFunction) do
   def call_deferred(scope, deferred)
     Puppet::Pops::Evaluator::DeferredResolver.resolve(deferred, scope.compiler)
   end
-
 end

data/lib/puppet/functions/camelcase.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 # Creates a Camel Case version of a String
 #
 # This function is compatible with the stdlib function with the same name.
@@ -30,7 +31,6 @@
 # Would both result in `['AbcDef', 'BcdXyz']`
 #
 Puppet::Functions.create_function(:camelcase) do
-
   dispatch :on_numeric do
     param 'Numeric', :arg
   end

data/lib/puppet/functions/capitalize.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 # Capitalizes the first character of a String, or the first character of every String in an Iterable value (such as an Array).
 #
 # This function is compatible with the stdlib function with the same name.
@@ -29,7 +30,6 @@
 # Would both result in `['Abc', 'Bcd']`
 #
 Puppet::Functions.create_function(:capitalize) do
-
   dispatch :on_numeric do
     param 'Numeric', :arg
   end

data/lib/puppet/functions/ceiling.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 # Returns the smallest `Integer` greater or equal to the argument.
 # Takes a single numeric value as an argument.
 #
@@ -25,7 +26,7 @@ Puppet::Functions.create_function(:ceiling) do
 
   def on_string(x)
     Puppet.warn_once('deprecations', 'ceiling_function_numeric_coerce_string',
-      _("The ceiling() function's auto conversion of String to Float is deprecated - change to convert input before calling"))
+                     _("The ceiling() function's auto conversion of String to Float is deprecated - change to convert input before calling"))
 
     begin
       Float(x).ceil
@@ -34,5 +35,4 @@ Puppet::Functions.create_function(:ceiling) do
       raise(ArgumentError, _('ceiling(): cannot convert given value to a floating point value.'))
     end
   end
-
 end

data/lib/puppet/functions/chomp.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 # Returns a new string with the record separator character(s) removed.
 # The record separator is the line ending characters `\r` and `\n`.
 #
@@ -25,7 +26,6 @@
 # Would both result in `['hello', 'hi']`
 #
 Puppet::Functions.create_function(:chomp) do
-
   dispatch :on_numeric do
     param 'Numeric', :arg
   end

data/lib/puppet/functions/chop.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 # Returns a new string with the last character removed.
 # If the string ends with `\r\n`, both characters are removed. Applying chop to an empty
 # string returns an empty string. If you wish to merely remove record
@@ -35,7 +36,6 @@
 # Would both result in `['hello', 'hi']`
 #
 Puppet::Functions.create_function(:chop) do
-
   dispatch :on_numeric do
     param 'Numeric', :arg
   end

data/lib/puppet/functions/compare.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 # Compares two values and returns -1, 0 or 1 if first value is smaller, equal or larger than the second value.
 # The compare function accepts arguments of the data types `String`, `Numeric`, `Timespan`, `Timestamp`, and `Semver`,
 # such that:

data/lib/puppet/functions/contain.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 # Makes one or more classes be contained inside the current class.
 # If any of these classes are undeclared, they will be declared as if
 # there were declared with the `include` function.

data/lib/puppet/functions/convert_to.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 # The `convert_to(value, type)` is a convenience function that does the same as `new(type, value)`.
 # The difference in the argument ordering allows it to be used in chained style for
 # improved readability "left to right".

data/lib/puppet/functions/crit.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 # Logs a message on the server at level `crit`.
 Puppet::Functions.create_function(:crit, Puppet::Functions::InternalFunction) do
   # @param values The values to log.

data/lib/puppet/functions/debug.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 # Logs a message on the server at level `debug`.
 Puppet::Functions.create_function(:debug, Puppet::Functions::InternalFunction) do
   # @param values The values to log.

data/lib/puppet/functions/defined.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 # Determines whether a given class or resource type is defined and returns a Boolean
 # value. You can also use `defined` to determine whether a specific resource is defined,
 # or whether a variable has a value (including `undef`, as opposed to the variable never
@@ -101,13 +102,12 @@
 # @since 4.0.0
 #
 Puppet::Functions.create_function(:'defined', Puppet::Functions::InternalFunction) do
-
   dispatch :is_defined do
     scope_param
     required_repeated_param 'Variant[String, Type[CatalogEntry], Type[Type[CatalogEntry]]]', :vals
   end
 
-  def is_defined(scope, *vals)
+  def is_defined(scope, *vals) # rubocop:disable Naming/PredicateName
     vals.any? do |val|
       case val
       when String
@@ -131,11 +131,13 @@ Puppet::Functions.create_function(:'defined', Puppet::Functions::InternalFunctio
 
       when Puppet::Pops::Types::PResourceType
         raise ArgumentError, _('The given resource type is a reference to all kind of types') if val.type_name.nil?
+
         type = Puppet::Pops::Evaluator::Runtime3ResourceSupport.find_resource_type(scope, val.type_name)
         val.title.nil? ? type : scope.compiler.findresource(type, val.title)
 
       when Puppet::Pops::Types::PClassType
-        raise  ArgumentError, _('The given class type is a reference to all classes') if val.class_name.nil?
+        raise ArgumentError, _('The given class type is a reference to all classes') if val.class_name.nil?
+
         scope.compiler.findresource(:class, val.class_name)
 
       when Puppet::Pops::Types::PTypeType
@@ -149,7 +151,8 @@ Puppet::Functions.create_function(:'defined', Puppet::Functions::InternalFunctio
           # Interpreted as asking if a class (and nothing else) is defined without having to be included in the catalog
           # (this is the same as asking for just the class' name, but with the added certainty that it cannot be a defined type.
           #
-          raise  ArgumentError, _('The given class type is a reference to all classes') if val.type.class_name.nil?
+          raise ArgumentError, _('The given class type is a reference to all classes') if val.type.class_name.nil?
+
           Puppet::Pops::Evaluator::Runtime3ResourceSupport.find_hostclass(scope, val.type.class_name)
         end
       else

data/lib/puppet/functions/dig.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 # Returns a value for a sequence of given keys/indexes into a structure, such as
 # an array or hash.
 #
@@ -38,14 +39,15 @@ Puppet::Functions.create_function(:dig) do
     walked_path = []
     args.reduce(data) do | d, k |
       return nil if d.nil? || k.nil?
+
       if !(d.is_a?(Array) || d.is_a?(Hash))
         t = Puppet::Pops::Types::TypeCalculator.infer(d)
         msg = _("The given data does not contain a Collection at %{walked_path}, got '%{type}'") % { walked_path: walked_path, type: t }
         error_data = Puppet::DataTypes::Error.new(
-            msg,
-            'SLICE_ERROR',
-            {'walked_path' => walked_path, 'value_type' => t},
-            'EXPECTED_COLLECTION'
+          msg,
+          'SLICE_ERROR',
+          {'walked_path' => walked_path, 'value_type' => t},
+          'EXPECTED_COLLECTION'
         )
         raise Puppet::ErrorWithData.new(error_data, msg)
       end
@@ -55,10 +57,10 @@ Puppet::Functions.create_function(:dig) do
         t = Puppet::Pops::Types::TypeCalculator.infer(k)
         msg = _("The given data requires an Integer index at %{walked_path}, got '%{type}'") % { walked_path: walked_path, type: t }
         error_data = Puppet::DataTypes::Error.new(
-            msg,
-            'SLICE_ERROR',
-            {'walked_path' => walked_path, 'index_type' => t},
-            'EXPECTED_INTEGER_INDEX'
+          msg,
+          'SLICE_ERROR',
+          {'walked_path' => walked_path, 'index_type' => t},
+          'EXPECTED_INTEGER_INDEX'
         )
         raise Puppet::ErrorWithData.new(error_data, msg)
       end

data/lib/puppet/functions/downcase.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 # Converts a String, Array or Hash (recursively) into lower case.
 #
 # This function is compatible with the stdlib function with the same name.

data/lib/puppet/functions/each.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 # Runs a [lambda](https://puppet.com/docs/puppet/latest/lang_lambdas.html)
 # repeatedly using each value in a data structure, then returns the values unchanged.
 #

data/lib/puppet/functions/emerg.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 # Logs a message on the server at level `emerg`.
 Puppet::Functions.create_function(:emerg, Puppet::Functions::InternalFunction) do
   # @param values The values to log.

data/lib/puppet/functions/empty.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 # Returns `true` if the given argument is an empty collection of values.
 #
 # This function can answer if one of the following is empty:

data/lib/puppet/functions/epp.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 # Evaluates an Embedded Puppet (EPP) template file and returns the rendered text
 # result as a String.
 #
@@ -36,7 +37,6 @@
 # @since 4.0.0
 #
 Puppet::Functions.create_function(:epp, Puppet::Functions::InternalFunction) do
-
   dispatch :epp do
     scope_param
     param 'String', :path

data/lib/puppet/functions/err.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 # Logs a message on the server at level `err`.
 Puppet::Functions.create_function(:err, Puppet::Functions::InternalFunction) do
   # @param values The values to log.

data/lib/puppet/functions/eyaml_lookup_key.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 # The `eyaml_lookup_key` is a hiera 5 `lookup_key` data provider function.
 # See [the configuration guide documentation](https://puppet.com/docs/puppet/latest/hiera_config_yaml_5.html#configuring-a-hierarchy-level-hiera-eyaml) for
 # how to use this function.
@@ -27,10 +28,10 @@ Puppet::Functions.create_function(:eyaml_lookup_key) do
     # Can't do this with an argument_mismatch dispatcher since there is no way to declare a struct that at least
     # contains some keys but may contain other arbitrary keys.
     unless options.include?('path')
-      #TRANSLATORS 'eyaml_lookup_key':, 'path', 'paths' 'glob', 'globs', 'mapped_paths', and lookup_key should not be translated
+      # TRANSLATORS 'eyaml_lookup_key':, 'path', 'paths' 'glob', 'globs', 'mapped_paths', and lookup_key should not be translated
       raise ArgumentError,
-        _("'eyaml_lookup_key': one of 'path', 'paths' 'glob', 'globs' or 'mapped_paths' must be declared in hiera.yaml"\
-              " when using this lookup_key function")
+            _("'eyaml_lookup_key': one of 'path', 'paths' 'glob', 'globs' or 'mapped_paths' must be declared in hiera.yaml"\
+                  " when using this lookup_key function")
     end
 
     # nil key is used to indicate that the cache contains the raw content of the eyaml file
@@ -53,6 +54,7 @@ Puppet::Functions.create_function(:eyaml_lookup_key) do
         else
           msg = _("%{path}: file does not contain a valid yaml hash") % { path: path }
           raise Puppet::DataBinding::LookupError, msg if Puppet[:strict] == :error && data != false
+
           Puppet.warning(msg)
           {}
         end
@@ -91,7 +93,7 @@ Puppet::Functions.create_function(:eyaml_lookup_key) do
         data = tokens.map(&:to_plain_text).join.chomp
       rescue StandardError => ex
         raise Puppet::DataBinding::LookupError,
-          _("hiera-eyaml backend error decrypting %{data} when looking up %{key} in %{path}. Error was %{message}") % { data: data, key: key, path: options['path'], message: ex.message }
+              _("hiera-eyaml backend error decrypting %{data} when looking up %{key} in %{path}. Error was %{message}") % { data: data, key: key, path: options['path'], message: ex.message }
       end
     end
     context.interpolate(data)

data/lib/puppet/functions/filter.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 # Applies a [lambda](https://puppet.com/docs/puppet/latest/lang_lambdas.html)
 # to every value in a data structure and returns an array or hash containing any elements
 # for which the lambda evaluates to a truthy value (not `false` or `undef`).

data/lib/puppet/functions/find_file.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 # Finds an existing file from a module and returns its path.
 #
 # This function accepts an argument that is a String as a `<MODULE NAME>/<FILE>`

data/lib/puppet/functions/find_template.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 # Finds an existing template from a module and returns its path.
 #
 # This function accepts an argument that is a String as a `<MODULE NAME>/<TEMPLATE>`

data/lib/puppet/functions/flatten.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 # Returns a flat Array produced from its possibly deeply nested given arguments.
 #
 # One or more arguments of any data type can be given to this function.

data/lib/puppet/functions/floor.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 # Returns the largest `Integer` less or equal to the argument.
 # Takes a single numeric value as an argument.
 #
@@ -25,7 +26,7 @@ Puppet::Functions.create_function(:floor) do
 
   def on_string(x)
     Puppet.warn_once('deprecations', 'floor_function_numeric_coerce_string',
-      _("The floor() function's auto conversion of String to Float is deprecated - change to convert input before calling"))
+                     _("The floor() function's auto conversion of String to Float is deprecated - change to convert input before calling"))
 
     begin
       Float(x).floor
@@ -34,5 +35,4 @@ Puppet::Functions.create_function(:floor) do
       raise(ArgumentError, _('floor(): cannot convert given value to a floating point value.'))
     end
   end
-
 end

data/lib/puppet/functions/get.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 # Digs into a value with dot notation to get a value from within a structure.
 #
 # **To dig into a given value**, call the function with (at least) two arguments:
@@ -132,7 +133,7 @@ Puppet::Functions.create_function(:get, Puppet::Functions::InternalFunction) do
     # like '0.1' the wanted result is `[0,1]`, not `["0", 1]`. The workaround here is to
     # prefix the navigation with `"x."` thus giving split_key a first segment that is a string.
     # The fake segment is then dropped.
-    segments = split_key("x." + navigation) {|err| _("Syntax error in dotted-navigation string")}
+    segments = split_key("x." + navigation) {|_err| _("Syntax error in dotted-navigation string")}
     segments.shift
 
     begin

data/lib/puppet/functions/getvar.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 # Digs into a variable with dot notation to get a value from a structure.
 #
 # **To get the value from a variable** (that may or may not exist), call the function with
@@ -46,7 +47,7 @@
 Puppet::Functions.create_function(:getvar, Puppet::Functions::InternalFunction) do
   dispatch :get_from_navigation do
     scope_param
-     param 'Pattern[/\A(?:::)?(?:[a-z]\w*::)*[a-z_]\w*(?:\.|\Z)/]', :get_string
+    param 'Pattern[/\A(?:::)?(?:[a-z]\w*::)*[a-z_]\w*(?:\.|\Z)/]', :get_string
     optional_param 'Any', :default_value
     optional_block_param 'Callable[1,1]', :block
   end

data/lib/puppet/functions/hiera.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 require 'hiera/puppet_function'
 # Performs a standard priority lookup of the hierarchy and returns the most specific value
 # for a given key. The returned value can be any type of data.

data/lib/puppet/functions/hiera_array.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 require 'hiera/puppet_function'
 
 # Finds all matches of a key throughout the hierarchy and returns them as a single flattened

data/lib/puppet/functions/hiera_hash.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 require 'hiera/puppet_function'
 
 # Finds all matches of a key throughout the hierarchy and returns them in a merged hash.

data/lib/puppet/functions/hiera_include.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 require 'hiera/puppet_function'
 
 # Assigns classes to a node using an
@@ -100,6 +101,7 @@ Puppet::Functions.create_function(:hiera_include, Hiera::PuppetFunction) do
 
   def post_lookup(scope, key, value)
     raise Puppet::ParseError, _("Could not find data item %{key}") % { key: key } if value.nil?
+
     call_function_with_scope(scope, 'include', value) unless value.empty?
   end
 end