openbolt 5.0.0.rc1 → 5.1.0

data/lib/bolt/plugin/cache.rb

@@ -17,6 +17,7 @@ module Bolt
 
       def read_and_clean_cache
         return if ttl == 0
+
         validate
 
         # Luckily we don't need to use a serious hash algorithm
@@ -53,6 +54,7 @@ module Bolt
         # The default cache `plugin-cache` will be validated by the config
         # validator
         return if reference['_cache'].nil?
+
         r = reference['_cache']
         unless r.is_a?(Hash)
           raise Bolt::ValidationError,

data/lib/bolt/plugin/module.rb

@@ -67,6 +67,7 @@ module Bolt
 
       def process_schema(schema)
         raise InvalidPluginData.new('config specification is not an object', name) unless schema.is_a?(Hash)
+
         schema.each do |key, val|
           unless key =~ /\A[a-z][a-z0-9_]*\z/
             raise InvalidPluginData.new("config specification key, '#{key}',  is not allowed", name)
@@ -102,6 +103,7 @@ module Bolt
           unless spec['pcore_type'].instance?(val)
             raise Bolt::ValidationError, "#{name} plugin expects a #{spec['type']} for key #{key}, got: #{val}"
           end
+
           val.nil?
         end
         nil
@@ -177,8 +179,10 @@ module Bolt
         # Get the intersection of expected types (using Set)
         type_set = @hook_map.each_with_object({}) do |(_hook, task), acc|
           next unless (schema = task['task'].metadata['parameters'])
+
           schema.each do |param, scheme|
             next unless scheme['type'].is_a?(String)
+
             scheme['type'] = Set.new([scheme['type']])
             if acc.dig(param, 'type').is_a?(Set)
               scheme['type'].merge(acc[param]['type'])
@@ -189,6 +193,7 @@ module Bolt
         # Convert Set to string
         type_set.each do |_param, schema|
           next unless schema['type']
+
           schema['type'] = if schema['type'].size > 1
                              "Optional[Variant[#{schema['type'].to_a.join(', ')}]]"
                            else
@@ -210,6 +215,7 @@ module Bolt
                                          options).first
 
         raise Bolt::Error.new(result.error_hash['msg'], result.error_hash['kind']) unless result.ok
+
         result.value
       end
 
@@ -217,6 +223,7 @@ module Bolt
         hook = @hook_map[hook_name]
         # This shouldn't happen if the Plugin api is used
         raise PluginError::UnsupportedHook.new(name, hook_name) unless hook
+
         result = run_task(hook['task'], opts)
 
         if value

data/lib/bolt/plugin/puppet_connect_data.rb

@@ -38,6 +38,7 @@ module Bolt
               if current_value.is_a?(Hash) && current_value.key?('_plugin')
                 raise invalid_input_data_err("the #{key} key's value contains a plugin reference")
               end
+
               current_value
             end
           end

data/lib/bolt/plugin/task.rb

@@ -30,16 +30,19 @@ module Bolt
         options = { catch_errors: true }
 
         raise Bolt::ValidationError, "Task plugin requires that the 'task' is specified" unless opts['task']
+
         task = @context.get_validated_task(opts['task'], params)
 
         result = @context.run_local_task(task, params, options).first
 
         raise Bolt::Error.new(result.error_hash['msg'], result.error_hash['kind']) if result.error_hash
+
         result
       end
 
       def validate_options(opts)
         raise Bolt::ValidationError, "Task plugin requires that the 'task' is specified" unless opts['task']
+
         @context.get_validated_task(opts['task'], opts['parameters'] || {})
       end
       alias validate_resolve_reference validate_options

data/lib/bolt/plugin.rb

@@ -203,6 +203,7 @@ module Bolt
 
     def config_for_plugin(plugin_name)
       return {} unless @unresolved_plugin_configs.include?(plugin_name)
+
       if @resolution_stack.include?(plugin_name)
         msg = "Configuration for plugin '#{plugin_name}' depends on the plugin itself"
         raise PluginError.new(msg, 'bolt/plugin-error')
@@ -225,6 +226,7 @@ module Bolt
       plugin = by_name(plugin_name)
       raise PluginError::Unknown, plugin_name unless plugin
       raise PluginError::UnsupportedHook.new(plugin_name, hook) unless plugin.hooks.include?(hook)
+
       @analytics.report_bundled_content("Plugin #{hook}", plugin_name)
 
       plugin.method(hook)
@@ -260,6 +262,7 @@ module Bolt
         when Bolt::Plugin::Module
           plugin.hook_map.each do |hook, spec|
             next unless hooks.include?(hook)
+
             hooks[hook][name] = spec['task'].description
           end
         else
@@ -277,6 +280,7 @@ module Bolt
     private def load_all_plugins
       modules.each do |name, mod|
         next unless mod.plugin?
+
         by_name(name)
       end
 

data/lib/bolt/project.rb

@@ -55,7 +55,7 @@ module Bolt
         rescue StandardError
           Bolt::Logger.warn(
             "non_writeable_project",
-            "Could not create default project at #{path}. Continuing without a writeable project. "\
+            "Could not create default project at #{path}. Continuing without a writeable project. " \
             "Log and rerun files will not be written."
           )
         end
@@ -67,7 +67,7 @@ module Bolt
 
       if !Bolt::Util.windows? && type != 'environment' && fullpath.world_writable?
         raise Bolt::Error.new(
-          "Project directory '#{fullpath}' is world-writable which poses a security risk. Set "\
+          "Project directory '#{fullpath}' is world-writable which poses a security risk. Set " \
           "BOLT_PROJECT='#{fullpath}' to force the use of this project directory.",
           "bolt/world-writable-error"
         )
@@ -203,7 +203,7 @@ module Bolt
           and can include lowercase letters, numbers, and underscores.
           ERROR_STRING
         elsif Dir.children(Bolt::Config::Modulepath::BOLTLIB_PATH).include?(name)
-          raise Bolt::ValidationError, "The project '#{name}' will not be loaded. The project name conflicts "\
+          raise Bolt::ValidationError, "The project '#{name}' will not be loaded. The project name conflicts " \
             "with a built-in Bolt module of the same name."
         end
       elsif name.nil? &&

data/lib/bolt/project_manager/config_migrator.rb

@@ -41,7 +41,7 @@ module Bolt
 
           begin
             @outputter.print_action_step(
-              "Moving transportation configuration options '#{transport_data.keys.join(', ')}' "\
+              "Moving transportation configuration options '#{transport_data.keys.join(', ')}' " \
               "from bolt.yaml to inventory.yaml"
             )
 
@@ -57,8 +57,8 @@ module Bolt
 
         command = Bolt::Util.powershell? ? 'Get-Help about_bolt_project' : 'bolt guide project'
         @outputter.print_action_step(
-          "Successfully migrated config. Please add a 'name' key to bolt-project.yaml "\
-          "to use project-level tasks and plans. Learn more about projects by running "\
+          "Successfully migrated config. Please add a 'name' key to bolt-project.yaml " \
+          "to use project-level tasks and plans. Learn more about projects by running " \
           "'#{command}'."
         )
 

data/lib/bolt/project_manager/inventory_migrator.rb

@@ -34,7 +34,7 @@ module Bolt
           true
         rescue StandardError => e
           raise Bolt::FileError.new(
-            "Unable to write to #{inventory_file}: #{e.message}. See "\
+            "Unable to write to #{inventory_file}: #{e.message}. See " \
             "http://pup.pt/bolt-inventory to manually update.",
             inventory_file
           )

data/lib/bolt/project_manager/module_migrator.rb

@@ -21,8 +21,8 @@ module Bolt
         # Notify user to manually migrate modules if using non-default modulepath
         if configured_modulepath != new_modulepath && configured_modulepath != old_modulepath
           @outputter.print_action_step(
-            "Project has a non-default configured modulepath, unable to automatically "\
-            "migrate project modules. To migrate project modules manually, see "\
+            "Project has a non-default configured modulepath, unable to automatically " \
+            "migrate project modules. To migrate project modules manually, see " \
             "http://pup.pt/bolt-modules"
           )
           true
@@ -106,7 +106,7 @@ module Bolt
           Bolt::ModuleInstaller::Installer.new.install(puppetfile_path, managed_moduledir)
         else
           @outputter.print_action_step(
-            "Project does not include any managed modules, deleting Puppetfile "\
+            "Project does not include any managed modules, deleting Puppetfile " \
             "at #{puppetfile_path}"
           )
           FileUtils.rm(puppetfile_path)
@@ -118,9 +118,9 @@ module Bolt
       #
       private def select_modules(modules)
         @outputter.print_action_step(
-          "Select modules that are direct dependencies of your project. Bolt will "\
-          "automatically manage dependencies for each module selected, so do not "\
-          "select a module's dependencies unless you use content from it directly "\
+          "Select modules that are direct dependencies of your project. Bolt will " \
+          "automatically manage dependencies for each module selected, so do not " \
+          "select a module's dependencies unless you use content from it directly " \
           "in your project."
         )
 
@@ -161,6 +161,7 @@ module Bolt
           source.each_child do |mod|
             next unless mod.directory?
             next if (moduledir + mod.basename).directory?
+
             FileUtils.mv(mod, moduledir)
           end
 

data/lib/bolt/project_manager.rb

@@ -70,14 +70,14 @@ module Bolt
         if modules
           command = Bolt::Util.powershell? ? 'Add-BoltModule -Module' : 'bolt module add'
           raise Bolt::Error.new(
-            "Found existing project directory with #{config.basename} at #{project}, "\
-            "unable to initialize project with modules. To add modules to the project, "\
+            "Found existing project directory with #{config.basename} at #{project}, " \
+            "unable to initialize project with modules. To add modules to the project, " \
             "run '#{command} <module>' instead.",
             'bolt/existing-project-error'
           )
         else
           raise Bolt::Error.new(
-            "Found existing project directory with #{config.basename} at #{project}, "\
+            "Found existing project directory with #{config.basename} at #{project}, " \
             "unable to initialize project.",
             'bolt/existing-project-error'
           )
@@ -85,27 +85,27 @@ module Bolt
       elsif old_config.exist?
         command = Bolt::Util.powershell? ? 'Update-BoltProject' : 'bolt project migrate'
         raise Bolt::Error.new(
-          "Found existing project directory with #{old_config.basename} at #{project}, "\
-          "unable to initialize project. #{old_config.basename} is deprecated. To "\
+          "Found existing project directory with #{old_config.basename} at #{project}, " \
+          "unable to initialize project. #{old_config.basename} is deprecated. To " \
           "update the project to current best practices, run '#{command}'.",
           'bolt/existing-project-error'
         )
       elsif modules && puppetfile.exist?
         raise Bolt::Error.new(
-          "Found existing Puppetfile at #{puppetfile}, unable to initialize project "\
+          "Found existing Puppetfile at #{puppetfile}, unable to initialize project " \
           "with modules.",
           'bolt/existing-puppetfile-error'
         )
       elsif project_name !~ Bolt::Module::MODULE_NAME_REGEX
         if name
           raise Bolt::ValidationError,
-                "The provided project name '#{project_name}' is invalid; project name must "\
-                "begin with a lowercase letter and can include lowercase letters, "\
+                "The provided project name '#{project_name}' is invalid; project name must " \
+                "begin with a lowercase letter and can include lowercase letters, " \
                 "numbers, and underscores."
         else
           command = Bolt::Util.powershell? ? 'New-BoltProject -Name' : 'bolt project init'
           raise Bolt::ValidationError,
-                "The current directory name '#{project_name}' is an invalid project name. "\
+                "The current directory name '#{project_name}' is an invalid project name. " \
                 "Please specify a name using '#{command} <name>'."
         end
       end
@@ -162,8 +162,8 @@ module Bolt
       @outputter.print_message("Migrating project #{@config.project.path}\n\n")
 
       @outputter.print_action_step(
-        "Migrating a Bolt project might make irreversible changes to the project's "\
-        "configuration and inventory files. Before continuing, make sure the "\
+        "Migrating a Bolt project might make irreversible changes to the project's " \
+        "configuration and inventory files. Before continuing, make sure the " \
         "project has a backup or uses a version control system."
       )
 

data/lib/bolt/puppetdb/config.rb

@@ -61,6 +61,7 @@ module Bolt
 
       def token
         return @token if @token_computed
+
         # Allow nil in config to skip loading a token
         if @settings.include?('token')
           if @settings['token']
@@ -81,6 +82,7 @@ module Bolt
       def expand_paths(project_path)
         %w[cacert cert key token].each do |file|
           next unless @settings[file]
+
           @settings[file] = File.expand_path(@settings[file], project_path)
         end
       end
@@ -89,6 +91,7 @@ module Bolt
         if @settings[file] && !File.exist?(@settings[file])
           raise Bolt::PuppetDBError, "#{file} file #{@settings[file]} does not exist"
         end
+
         true
       end
 
@@ -107,6 +110,7 @@ module Bolt
 
       def uri
         return @uri if @uri
+
         require 'addressable/uri'
 
         uri = case @settings['server_urls']

data/lib/bolt/puppetdb/instance.rb

@@ -105,6 +105,7 @@ module Bolt
 
       def http_client
         return @http if @http
+
         # lazy-load expensive gem code
         require 'httpclient'
         @logger.trace("Creating HTTP Client")

data/lib/bolt/rerun.rb

@@ -16,6 +16,7 @@ module Bolt
       unless @data.is_a?(Array) && @data.all? { |r| r['target'] && r['status'] }
         raise Bolt::FileError.new("Missing data in rerun file: #{@path}", @path)
       end
+
       @data
     end
 

data/lib/bolt/resource_instance.rb

@@ -14,7 +14,7 @@ module Bolt
 
     # Needed by Puppet to serialize with _pcore_init_hash instead of the object's attributes
     def self._pcore_init_from_hash(_init_hash)
-      raise "ResourceInstance shouldn't be instantiated from a pcore_init class method. "\
+      raise "ResourceInstance shouldn't be instantiated from a pcore_init class method. " \
             "How did this get called?"
     end
 

data/lib/bolt/result.rb

@@ -81,7 +81,7 @@ module Bolt
         unless value['_error'].is_a?(Hash) && value['_error'].key?('msg')
           details['original_error'] = value['_error']
           value['_error'] = {
-            'msg'     => "Invalid error returned from task #{task}: #{value['_error'].inspect}. Error "\
+            'msg'     => "Invalid error returned from task #{task}: #{value['_error'].inspect}. Error " \
                          "must be an object with a msg key.",
             'kind'    => 'bolt/invalid-task-error',
             'details' => details
@@ -148,6 +148,7 @@ module Bolt
       if error && !error.is_a?(Hash)
         raise "TODO: how did we get a string error"
       end
+
       @value['_error'] = error if error
       @value['_output'] = message if message
     end

data/lib/bolt/shell/bash.rb

@@ -301,6 +301,7 @@ module Bolt
         if result.exit_code != 0
           raise Bolt::Node::FileError.new("Could not make tmpdir: #{result.stderr.string}", 'TMPDIR_ERROR')
         end
+
         path = tmppath || result.stdout.string.chomp
         Bolt::Shell::Bash::Tmpdir.new(self, path)
       end
@@ -482,7 +483,7 @@ module Bolt
         when 0
           @logger.trace { "Command `#{command_str}` returned successfully" }
         when 126
-          msg = "\n\nThis might be caused by the default tmpdir being mounted "\
+          msg = "\n\nThis might be caused by the default tmpdir being mounted " \
             "using 'noexec'. See http://pup.pt/task-failure for details and workarounds."
           result_output.stderr        << msg
           result_output.merged_output << msg

data/lib/bolt/shell/powershell.rb

@@ -22,8 +22,8 @@ module Bolt
         if !version.empty? && version.to_i < 3
           # This lets us know how many targets have Powershell 2, and lets the
           # user know how many targets they have with PS2
-          msg = "Detected PowerShell 2 on one or more targets.\nPowerShell 2 "\
-            "is unsupported. See bolt-debug.log or run with '--log-level debug' to see the full "\
+          msg = "Detected PowerShell 2 on one or more targets.\nPowerShell 2 " \
+            "is unsupported. See bolt-debug.log or run with '--log-level debug' to see the full " \
             "list of targets with PowerShell 2."
 
           Bolt::Logger.deprecate_once("powershell_2", msg)
@@ -45,7 +45,7 @@ module Bolt
 
       def validate_extensions(ext)
         unless @extensions.include?(ext)
-          raise Bolt::Node::FileError.new("File extension #{ext} is not enabled, "\
+          raise Bolt::Node::FileError.new("File extension #{ext} is not enabled, " \
                                           "to run it please add to 'winrm: extensions'", 'FILETYPE_ERROR')
         end
       end
@@ -143,6 +143,7 @@ module Bolt
         if result.exit_code != 0
           raise Bolt::Node::FileError.new("Could not make tmpdir: #{result.stderr.string}", 'TMPDIR_ERROR')
         end
+
         result.stdout.string.chomp
       end
 

data/lib/bolt/shell.rb

@@ -11,7 +11,7 @@ module Bolt
 
       if Bolt::Logger.stream
         Bolt::Logger.warn_once("stream_experimental",
-                               "The 'stream' option is experimental, and might "\
+                               "The 'stream' option is experimental, and might " \
                                "include breaking changes between minor versions.")
         @stream_logger = Bolt::Logger.logger(:stream)
         # Don't send stream messages to the parent logger

data/lib/bolt/task/run.rb

@@ -47,6 +47,7 @@ module Bolt
           if !result.ok && !options[:catch_errors]
             raise Bolt::RunFailure.new(result, 'run_task', task.name)
           end
+
           result
         end
       end

data/lib/bolt/task.rb

@@ -99,12 +99,14 @@ module Bolt
                  Set.new(imp['requirements']).subset?(available_features) && !!remote_impl == @remote
                end
                raise NoImplementationError.new(target, self) unless impl
+
                impl = impl.dup
                impl['path'] = file_path(impl['name'])
                impl.delete('requirements')
                impl
              else
                raise NoImplementationError.new(target, self) unless !!metadata['remote'] == @remote
+
                name = files.first['name']
                { 'name' => name, 'path' => file_path(name) }
              end
@@ -117,6 +119,7 @@ module Bolt
       impl['files'] = filenames.map do |file|
         path = file_path(file)
         raise "No file found for reference #{file}" if path.nil?
+
         { 'name' => file, 'path' => path }
       end
 

data/lib/bolt/transport/docker/connection.rb

@@ -11,6 +11,7 @@ module Bolt
 
         def initialize(target)
           raise Bolt::ValidationError, "Target #{target.safe_name} does not have a host" unless target.host
+
           @target = target
           @user = ENV['USER'] || Etc.getlogin
           @logger = Bolt::Logger.logger(target.safe_name)
@@ -53,6 +54,7 @@ module Bolt
           output = execute_local_json_command('ps', ['--no-trunc'])
           index = output.find_index { |item| item["ID"].start_with?(target.host) || item["Names"] == target.host }
           raise "Could not find a container with name or ID matching '#{target.host}'" if index.nil?
+
           # Now find the indepth container information
           output = execute_local_json_command('inspect', [output[index]["ID"]])
           # Store the container information for later

data/lib/bolt/transport/jail/connection.rb

@@ -11,6 +11,7 @@ module Bolt
 
         def initialize(target)
           raise Bolt::ValidationError, "Target #{target.safe_name} does not have a host" unless target.host
+
           @target = target
           @user = @target.user || ENV['USER'] || Etc.getlogin
           @logger = Bolt::Logger.logger(target.safe_name)
@@ -38,6 +39,7 @@ module Bolt
           output = JSON.parse(`jls --libxo=json`)
           @jail_info = output['jail-information']['jail'].select { |jail| jail['hostname'] == target.host }.first
           raise "Could not find a jail with name matching #{target.host}" if @jail_info.nil?
+
           @logger.trace { "Opened session" }
           true
         rescue StandardError => e

data/lib/bolt/transport/lxd/connection.rb

@@ -36,10 +36,12 @@ module Bolt
           unless status.exitstatus.zero?
             raise "Error listing available containers: #{err}"
           end
+
           containers = JSON.parse(out)
           if containers.empty?
             raise "Could not find a container with name or ID matching '#{container_id}'"
           end
+
           @logger.trace("Opened session")
           true
         rescue StandardError => e

data/lib/bolt/transport/lxd.rb

@@ -13,7 +13,7 @@ module Bolt
 
       def with_connection(target, options = {})
         Bolt::Logger.warn_once("lxd_experimental",
-                               "The LXD transport is experimental, and might "\
+                               "The LXD transport is experimental, and might " \
                                "include breaking changes between minor versions.")
         conn = Connection.new(target, options)
         conn.connect

data/lib/bolt/transport/podman/connection.rb

@@ -11,6 +11,7 @@ module Bolt
 
         def initialize(target)
           raise Bolt::ValidationError, "Target #{target.safe_name} does not have a host" unless target.host
+
           @target = target
           @user = ENV['USER'] || Etc.getlogin
           @logger = Bolt::Logger.logger(target.safe_name)
@@ -44,6 +45,7 @@ module Bolt
               Array(item["Names"]).include?(@target.host)
           }
           raise "Could not find a container with name or ID matching '#{@target.host}'" if container.nil?
+
           # Now find the indepth container information
           id = container["ID"] || container["Id"]
           output = execute_local_json_command('inspect', [id])

data/lib/bolt/transport/remote.rb

@@ -16,6 +16,7 @@ module Bolt
       def get_proxy(target)
         inventory = target.inventory
         raise "Target was created without inventory? Not get_targets?" unless inventory
+
         proxy = inventory.get_targets(target.options['run-on'] || 'localhost').first
 
         if proxy.transport == 'remote'

data/lib/bolt/transport/ssh/connection.rb

@@ -227,7 +227,7 @@ module Bolt
           end
           [in_wr, out_rd, err_rd, th]
         rescue Errno::EMFILE => e
-          msg = "#{e.message}. This might be resolved by increasing your user limit "\
+          msg = "#{e.message}. This might be resolved by increasing your user limit " \
             "with 'ulimit -n 1024'. See https://puppet.com/docs/bolt/latest/bolt_known_issues.html for details."
           raise Bolt::Error.new(msg, 'bolt/too-many-files')
         end

data/lib/bolt/transport/winrm/connection.rb

@@ -11,6 +11,7 @@ module Bolt
 
         def initialize(target, transport_logger)
           raise Bolt::ValidationError, "Target #{target.safe_name} does not have a host" unless target.host
+
           @target = target
 
           default_port = target.options['ssl'] ? HTTPS_PORT : HTTP_PORT
@@ -81,8 +82,8 @@ module Bolt
             theres_your_problem = "\nAre you using SSL to connect to a non-SSL port?"
           end
           if target.options['ssl-verify'] && e.message.include?('certificate verify failed')
-            theres_your_problem = "\nIs the remote host using a self-signed SSL "\
-                                  "certificate? Use --no-ssl-verify to disable "\
+            theres_your_problem = "\nIs the remote host using a self-signed SSL " \
+                                  "certificate? Use --no-ssl-verify to disable " \
                                   "remote host SSL verification."
           end
           raise Bolt::Node::ConnectError.new(
@@ -137,7 +138,7 @@ module Bolt
 
           [inp, out_rd, err_rd, th]
         rescue Errno::EMFILE => e
-          msg = "#{e.message}. This might be resolved by increasing your user limit "\
+          msg = "#{e.message}. This might be resolved by increasing your user limit " \
             "with 'ulimit -n 1024'. See https://puppet.com/docs/bolt/latest/bolt_known_issues.html for details."
           raise Bolt::Error.new(msg, 'bolt/too-many-files')
         rescue StandardError

data/lib/bolt/util/format.rb

@@ -56,6 +56,7 @@ module Bolt
           logs = result.resource_logs&.map do |log|
             # Omit low-level info/debug messages
             next if %w[info debug].include?(log['level'])
+
             indent(2, format_log(log))
           end
           hash = result.to_data

data/lib/bolt/util.rb

@@ -52,7 +52,7 @@ module Bolt
         content = File.open(path, "r:UTF-8") { |f| YAML.safe_load(f.read) } || {}
         unless content.is_a?(Hash)
           raise Bolt::FileError.new(
-            "Invalid content for #{file_name} file at #{path}\nContent should be a Hash or empty, "\
+            "Invalid content for #{file_name} file at #{path}\nContent should be a Hash or empty, " \
             "not #{content.class}",
             path
           )
@@ -62,11 +62,11 @@ module Bolt
       rescue Errno::ENOENT
         raise Bolt::FileError.new("Could not read #{file_name} file at #{path}", path)
       rescue Psych::SyntaxError => e
-        raise Bolt::FileError.new("Could not parse #{file_name} file at #{path}, line #{e.line}, "\
+        raise Bolt::FileError.new("Could not parse #{file_name} file at #{path}, line #{e.line}, " \
                                   "column #{e.column}\n#{e.problem}",
                                   path)
       rescue Psych::BadAlias => e
-        raise Bolt::FileError.new('Bolt does not support the use of aliases in YAML files. Alias '\
+        raise Bolt::FileError.new('Bolt does not support the use of aliases in YAML files. Alias ' \
                                   "detected in #{file_name} file at #{path}\n#{e.message}", path)
       rescue Psych::Exception => e
         raise Bolt::FileError.new("Could not parse #{file_name} file at #{path}\n#{e.message}",
@@ -147,7 +147,7 @@ module Bolt
 
         # Only accept paths with '/plans/' or '/tasks/'
         unless path.match?(regex)
-          msg = "Could not determine module from #{path}. "\
+          msg = "Could not determine module from #{path}. " \
             "The path must include 'plans' or 'tasks' directory"
           raise Bolt::Error.new(msg, 'bolt/modulepath-error')
         end
@@ -361,6 +361,7 @@ module Bolt
         elsif stat.directory?
           Dir.foreach(path) do |file|
             next if %w[. ..].include?(file)
+
             validate_file(type, File.join(path, file), allow_dir)
           end
         end
@@ -431,11 +432,13 @@ module Bolt
 
       def unix_basename(path)
         raise Bolt::ValidationError, "path must be a String, received #{path.class} #{path}" unless path.is_a?(String)
+
         path.split('/').last
       end
 
       def windows_basename(path)
         raise Bolt::ValidationError, "path must be a String, received #{path.class} #{path}" unless path.is_a?(String)
+
         path.split(%r{[/\\]}).last
       end