pdk-akerl 1.8.0.1 → 1.9.1.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 8880f732882ed9e6aa16f553af49b852fcfad47ec2750a10dbe5f2b98e309dd4
-  data.tar.gz: 2c823dfa985db272e0f033f1c0352fe84b0cad792e19b7145e4b43045c2ca896
+  metadata.gz: 39f58183065dc5b1f230bcffb0d5b9cb889b17c3a60185c824c34d2ab48e8787
+  data.tar.gz: 32a54051654794a36a5de1a34980a469a5e0b11e9a6d1c5ad18407b764dc7728
 SHA512:
-  metadata.gz: 5b8e5db0050d37a5ec5120c272e814d40ad060142fa124fc6393200c82338eed67b35bccd2274cceb1ffd1a780b071702954d39eb54a07fe0eef6e9846e57408
-  data.tar.gz: d4d83f4722b2d8c94bee7aa5a202bcf82bebe141462e82ac40616777543d0e84dd5697630901e10497ffdf46ea8c3570760c267999b20cf9201b5ddc8c1a0a68
+  metadata.gz: a5d3c027319d74685372a715a7ae858bc3ee219be81e4d26d31d60e8a89d68a9952f6547c4cd45c8ef23683dbf399a892cb75f11e41a755afaaf36deb9407cce
+  data.tar.gz: 45a778d76f7d8ec4ff09082c0710c2a0b291dce8162287ca4ef7f705e4e8f34e5ad534664d3a23ca25ab0a1986b834f654f2c833559e3d8a36d824bba06877d2

data/CHANGELOG.md

@@ -4,7 +4,54 @@ All changes to this repo will be documented in this file.
 See the [release notes](https://puppet.com/docs/pdk/latest/release_notes.html) for a high-level summary.
 
 
-## [v1.8.0](https://github.com/puppetlabs/pdk/tree/v1.8.0) (2018-11-27)
+## [v1.9.1](https://github.com/puppetlabs/pdk/tree/v1.9.1) (2019-03-01)
+[Full Changelog](https://github.com/puppetlabs/pdk/compare/v1.9.0...v1.9.1)
+
+**Fixed bugs:**
+
+- \(IMAGES-1037\) Make sure our paths are used [\#630](https://github.com/puppetlabs/pdk/pull/630) ([mihaibuzgau](https://github.com/mihaibuzgau))
+- \(PDK-1266\) Clear modulepath value when validating manifest syntax [\#629](https://github.com/puppetlabs/pdk/pull/629) ([rodjek](https://github.com/rodjek))
+- \(PDK-1272\) Convert user/module module names to user-module [\#626](https://github.com/puppetlabs/pdk/pull/626) ([rodjek](https://github.com/rodjek))
+- \(PDK-1276\) Skip non-file YAML validator targets [\#625](https://github.com/puppetlabs/pdk/pull/625) ([rodjek](https://github.com/rodjek))
+- \(PDK-1273\) Whitelist Ruby symbols in YAML validator [\#624](https://github.com/puppetlabs/pdk/pull/624) ([rodjek](https://github.com/rodjek))
+
+**Merged pull requests:**
+
+- \(maint\) Pin parallel gem to 1.13.0 [\#631](https://github.com/puppetlabs/pdk/pull/631) ([rodjek](https://github.com/rodjek))
+- \(PDK-1260\) Bump to 1.10.0.pre for new dev work [\#621](https://github.com/puppetlabs/pdk/pull/621) ([bmjen](https://github.com/bmjen))
+
+## [v1.9.0](https://github.com/puppetlabs/pdk/tree/v1.9.0) (2019-01-29)
+[Full Changelog](https://github.com/puppetlabs/pdk/compare/v1.8.0...v1.9.0)
+
+**Implemented enhancements:**
+
+- \(PDK-735\) Implement a YAML validator [\#612](https://github.com/puppetlabs/pdk/pull/612) ([rodjek](https://github.com/rodjek))
+
+**Fixed bugs:**
+
+- \(PDK-914\) Adjust default\_template\_url validation to accept local dirs [\#606](https://github.com/puppetlabs/pdk/pull/606) ([rodjek](https://github.com/rodjek))
+- \(PDK-1204\) pdk bundle execs in the context of the pwd [\#603](https://github.com/puppetlabs/pdk/pull/603) ([rodjek](https://github.com/rodjek))
+
+**Closed issues:**
+
+- After today upgrade rubygems-update the check for ruby 2.1.9 is completely broken [\#609](https://github.com/puppetlabs/pdk/issues/609)
+- 'unknown type yumrepo' during 'pdk test unit' [\#607](https://github.com/puppetlabs/pdk/issues/607)
+
+**Merged pull requests:**
+
+- \(maint\) Fix pin for nokogiri in package tests [\#620](https://github.com/puppetlabs/pdk/pull/620) ([bmjen](https://github.com/bmjen))
+- \(PDK-1240\) Update nokogiri to minimum of 1.8.5 [\#619](https://github.com/puppetlabs/pdk/pull/619) ([bmjen](https://github.com/bmjen))
+- \(maint\) Update hitimes pin to 1.3.0 for r2.1 compat [\#617](https://github.com/puppetlabs/pdk/pull/617) ([bmjen](https://github.com/bmjen))
+- Release 1.9.0 [\#616](https://github.com/puppetlabs/pdk/pull/616) ([bmjen](https://github.com/bmjen))
+- \(MAINT\) Configure Slack notifications for Travis [\#614](https://github.com/puppetlabs/pdk/pull/614) ([scotje](https://github.com/scotje))
+- \(maint\) Fix package tests to remove hardcoding [\#613](https://github.com/puppetlabs/pdk/pull/613) ([bmjen](https://github.com/bmjen))
+- \(MAINT\) Fix package acceptance tests to pass with any Ruby 2.4.x [\#611](https://github.com/puppetlabs/pdk/pull/611) ([scotje](https://github.com/scotje))
+- \(MAINT\) Bump default packaged ruby version to 2.4.5 [\#608](https://github.com/puppetlabs/pdk/pull/608) ([scotje](https://github.com/scotje))
+- \(PDK-1202\) Pass TemplateDir object through to TemplateFile [\#605](https://github.com/puppetlabs/pdk/pull/605) ([rodjek](https://github.com/rodjek))
+- \(PDK-1231\) Update version for new dev cycle. [\#604](https://github.com/puppetlabs/pdk/pull/604) ([bmjen](https://github.com/bmjen))
+- \(PDK-1001\) Chdir before execing git rather than "git -C" [\#602](https://github.com/puppetlabs/pdk/pull/602) ([rodjek](https://github.com/rodjek))
+
+## [v1.8.0](https://github.com/puppetlabs/pdk/tree/v1.8.0) (2018-11-28)
 [Full Changelog](https://github.com/puppetlabs/pdk/compare/v1.7.1...v1.8.0)
 
 **Implemented enhancements:**
@@ -34,6 +81,7 @@ See the [release notes](https://puppet.com/docs/pdk/latest/release_notes.html) f
 
 **Merged pull requests:**
 
+- Release 1.8.0 [\#601](https://github.com/puppetlabs/pdk/pull/601) ([bmjen](https://github.com/bmjen))
 - \(maint\) Update package tests to add task name validation [\#600](https://github.com/puppetlabs/pdk/pull/600) ([bmjen](https://github.com/bmjen))
 - \(maint\) Fix package tests [\#596](https://github.com/puppetlabs/pdk/pull/596) ([rodjek](https://github.com/rodjek))
 - \(maint\) Add --skip-bundle-install to `pdk new module` [\#595](https://github.com/puppetlabs/pdk/pull/595) ([rodjek](https://github.com/rodjek))

data/lib/pdk/cli/bundle.rb

@@ -27,7 +27,7 @@ EOF
       gemfile_env = PDK::Util::Bundler::BundleHelper.gemfile_env(puppet_env[:gemset])
 
       command = PDK::CLI::Exec::Command.new(PDK::CLI::Exec.bundle_bin, *args).tap do |c|
-        c.context = :module
+        c.context = :pwd
         c.update_environment(gemfile_env)
       end
 

data/lib/pdk/cli/exec.rb

@@ -94,7 +94,7 @@ module PDK
         end
 
         def context=(new_context)
-          unless [:system, :module].include?(new_context)
+          unless [:system, :module, :pwd].include?(new_context)
             raise ArgumentError, _("Expected execution context to be :system or :module but got '%{context}'.") % { context: new_context }
           end
 
@@ -132,7 +132,7 @@ module PDK
 
           @process.environment['BUNDLE_IGNORE_CONFIG'] = '1'
 
-          if context == :module
+          if [:module, :pwd].include?(context)
             @process.environment['GEM_HOME'] = PDK::Util::RubyVersion.gem_home
             @process.environment['GEM_PATH'] = PDK::Util::RubyVersion.gem_path
 
@@ -143,8 +143,8 @@ module PDK
               File.join(@process.environment['GEM_HOME'], 'bin'),
               PDK::Util::RubyVersion.gem_paths_raw.map { |gem_path| File.join(gem_path, 'bin') },
               package_binpath,
-              ENV['PATH'],
               PDK::Util.package_install? ? PDK::Util::Git.git_paths : nil,
+              ENV['PATH'],
             ].compact.flatten.join(File::PATH_SEPARATOR)
 
             mod_root = PDK::Util.module_root
@@ -155,7 +155,7 @@ module PDK
               raise PDK::CLI::FatalError, _('Current working directory is not part of a module. (No metadata.json was found.)')
             end
 
-            if Dir.pwd == mod_root
+            if Dir.pwd == mod_root || context == :pwd
               run_process_in_clean_env!
             else
               Dir.chdir(mod_root) do

data/lib/pdk/cli/validate.rb

@@ -6,7 +6,7 @@ module PDK::CLI
     usage _('validate [validators] [options] [targets]')
     summary _('Run static analysis tests.')
     description _(
-      "Run metadata, Puppet, Ruby, or Tasks validation.\n\n" \
+      "Run metadata, YAML, Puppet, Ruby, or Tasks validation.\n\n" \
       '[validators] is an optional comma-separated list of validators to use. ' \
       'If not specified, all validators are used. ' \
       "Note that when using PowerShell, the list of validators must be enclosed in single quotes.\n\n" \

data/lib/pdk/module/metadata.rb

@@ -163,7 +163,8 @@ module PDK
       # Do basic validation and parsing of the name parameter.
       def process_name(data)
         validate_name(data['name'])
-        author, _modname = data['name'].split(%r{[-/]}, 2)
+        author, modname = data['name'].split(%r{[-/]}, 2)
+        data['name'] = [author, modname].join('-')
 
         data['author'] ||= author if @data['author'] == DEFAULTS['author']
       end

data/lib/pdk/module/templatedir.rb

@@ -117,7 +117,7 @@ module PDK
             dest_status = :delete
           else
             begin
-              dest_content = PDK::TemplateFile.new(File.join(template_loc, template_file), configs: config).render
+              dest_content = PDK::TemplateFile.new(File.join(template_loc, template_file), configs: config, template_dir: self).render
             rescue => e
               error_msg = _(
                 "Failed to render template '%{template}'\n" \
@@ -269,8 +269,14 @@ module PDK
         if File.file?(loc) && File.readable?(loc)
           begin
             YAML.safe_load(File.read(loc), [], [], true)
-          rescue StandardError => e
-            PDK.logger.warn(_("'%{file}' is not a valid YAML file: %{message}") % { file: loc, message: e.message })
+          rescue Psych::SyntaxError => e
+            PDK.logger.warn _("'%{file}' is not a valid YAML file: %{problem} %{context} at line %{line} column %{column}") % {
+              file:    loc,
+              problem: e.problem,
+              context: e.context,
+              line:    e.line,
+              column:  e.column,
+            }
             {}
           end
         else
@@ -294,11 +300,13 @@ module PDK
         clone_result = PDK::Util::Git.git('clone', origin_repo, temp_dir)
 
         if clone_result[:exit_code].zero?
-          reset_result = PDK::Util::Git.git('-C', temp_dir, 'reset', '--hard', git_ref)
-          unless reset_result[:exit_code].zero?
-            PDK.logger.error reset_result[:stdout]
-            PDK.logger.error reset_result[:stderr]
-            raise PDK::CLI::FatalError, _("Unable to set HEAD of git repository at '%{repo}' to ref:'%{ref}'.") % { repo: temp_dir, ref: git_ref }
+          Dir.chdir(temp_dir) do
+            reset_result = PDK::Util::Git.git('reset', '--hard', git_ref)
+            unless reset_result[:exit_code].zero?
+              PDK.logger.error reset_result[:stdout]
+              PDK.logger.error reset_result[:stderr]
+              raise PDK::CLI::FatalError, _("Unable to set HEAD of git repository at '%{repo}' to ref:'%{ref}'.") % { repo: temp_dir, ref: git_ref }
+            end
           end
         else
           PDK.logger.error clone_result[:stdout]

data/lib/pdk/module.rb

@@ -4,7 +4,6 @@ module PDK
   module Module
     DEFAULT_IGNORED = [
       '/pkg/',
-      '.*',
       '~*',
       '/coverage',
       '/checksums.json',
@@ -13,8 +12,10 @@ module PDK
       '/vendor/',
     ].freeze
 
-    def default_ignored_pathspec
-      @default_ignored_pathspec ||= PathSpec.new(DEFAULT_IGNORED)
+    def default_ignored_pathspec(ignore_dotfiles = true)
+      PathSpec.new(DEFAULT_IGNORED).tap do |ps|
+        ps.add('.*') if ignore_dotfiles
+      end
     end
     module_function :default_ignored_pathspec
   end

data/lib/pdk/template_file.rb

@@ -44,6 +44,12 @@ module PDK
       end
     end
 
+    def config_for(path)
+      return nil unless respond_to?(:template_dir)
+
+      template_dir.config_for(path)
+    end
+
     private
 
     # Reads the content of the template file into memory.

data/lib/pdk/util/ruby_version.rb

@@ -45,8 +45,9 @@ module PDK
         end
 
         def default_ruby_version
-          # For now, the packaged versions will be using default of 2.4.4.
-          return '2.4.4' if PDK::Util.package_install?
+          # For now, the packaged versions will be using default of 2.4.5
+          # FIXME: make this dynamic
+          return '2.4.5' if PDK::Util.package_install?
 
           # TODO: may not be a safe assumption that highest available version should be default
           latest_ruby_version

data/lib/pdk/util.rb

@@ -206,13 +206,25 @@ module PDK
       return puppetlabs_template_url if answer_file_url == 'https://github.com/puppetlabs/pdk-module-template'
       return puppetlabs_template_url if answer_file_url == puppetlabs_template_url
 
-      unless PDK::Util::Git.repo?(answer_file_url)
-        PDK.logger.warn(_("Unable to access the previously used template '%{template}', using the default template instead.") % { template: answer_file_url })
-        PDK.answers.update!('template-url' => nil)
-        return puppetlabs_template_url
+      if File.directory?(answer_file_url)
+        # Instantiating a new TemplateDir object pointing to the directory
+        # will cause the directory contents to be validated, raising
+        # ArgumentError if it does not appear to be a valid template.
+        PDK::Module::TemplateDir.new(answer_file_url) {}
+        return answer_file_url
       end
 
+      raise ArgumentError unless PDK::Util::Git.repo?(answer_file_url)
+
       answer_file_url
+    rescue ArgumentError
+      PDK.logger.warn(
+        _("Unable to access the previously used template '%{template}', using the default template instead.") % {
+          template: answer_file_url,
+        },
+      )
+      PDK.answers.update!('template-url' => nil)
+      return puppetlabs_template_url
     end
     module_function :default_template_url
 

data/lib/pdk/validate/base_validator.rb

@@ -21,6 +21,8 @@ module PDK
       #   false - PDK will pass the globbed targets to the validator command.
       ALLOW_EMPTY_TARGETS = false
 
+      IGNORE_DOTFILES = true
+
       def self.cmd_path
         File.join(PDK::Util.module_root, 'bin', cmd)
       end
@@ -49,10 +51,10 @@ module PDK
           if respond_to?(:pattern)
             if File.directory?(target)
               target_root = PDK::Util.module_root
-              pattern_glob = Array(pattern).map { |p| Dir.glob(File.join(target_root, p)) }
+              pattern_glob = Array(pattern).map { |p| Dir.glob(File.join(target_root, p), File::FNM_DOTMATCH) }
 
               target_list = pattern_glob.flatten.map do |file|
-                if File.fnmatch(File.join(File.expand_path(target), '*'), file)
+                if File.fnmatch(File.join(File.expand_path(target), '*'), file, File::FNM_DOTMATCH)
                   Pathname.new(file).relative_path_from(Pathname.new(PDK::Util.module_root)).to_s
                 end
               end
@@ -65,7 +67,7 @@ module PDK
             elsif File.file?(target)
               if Array(pattern).include? target
                 target
-              elsif Array(pattern).any? { |p| File.fnmatch(File.expand_path(p), File.expand_path(target)) }
+              elsif Array(pattern).any? { |p| File.fnmatch(File.expand_path(p), File.expand_path(target), File::FNM_DOTMATCH) }
                 target
               else
                 skipped << target
@@ -83,7 +85,7 @@ module PDK
       end
 
       def self.ignore_pathspec
-        ignore_pathspec = PDK::Module.default_ignored_pathspec.dup
+        ignore_pathspec = PDK::Module.default_ignored_pathspec(ignore_dotfiles?)
 
         if respond_to?(:pattern_ignore)
           Array(pattern_ignore).each do |pattern|
@@ -94,6 +96,10 @@ module PDK
         ignore_pathspec
       end
 
+      def self.ignore_dotfiles?
+        self::IGNORE_DOTFILES
+      end
+
       def self.parse_options(_options, targets)
         targets
       end

data/lib/pdk/validate/puppet/puppet_syntax.rb

@@ -1,6 +1,8 @@
 require 'pdk'
 require 'pdk/cli/exec'
 require 'pdk/validate/base_validator'
+require 'fileutils'
+require 'tmpdir'
 
 module PDK
   module Validate
@@ -46,7 +48,30 @@ module PDK
       end
 
       def self.parse_options(_options, targets)
-        ['parser', 'validate', '--config', null_file].concat(targets)
+        # Due to PDK-1266 we need to run `puppet parser validate` with an empty
+        # modulepath. On *nix, Ruby treats `/dev/null` as an empty directory
+        # however it doesn't do so with `NUL` on Windows. The workaround for
+        # this to ensure consistent behaviour is to create an empty temporary
+        # directory and use that as the modulepath.
+        ['parser', 'validate', '--config', null_file, '--modulepath', validate_tmpdir].concat(targets)
+      end
+
+      def self.invoke(report, options)
+        super
+      ensure
+        remove_validate_tmpdir
+      end
+
+      def self.validate_tmpdir
+        @validate_tmpdir ||= Dir.mktmpdir('puppet-parser-validate')
+      end
+
+      def self.remove_validate_tmpdir
+        return unless @validate_tmpdir
+        return unless File.directory?(@validate_tmpdir)
+
+        FileUtils.remove_entry_secure(@validate_tmpdir)
+        @validate_tmpdir = nil
       end
 
       def self.null_file

data/lib/pdk/validate/yaml/syntax.rb

@@ -0,0 +1,123 @@
+require 'pdk'
+require 'pdk/cli/exec'
+require 'pdk/validate/base_validator'
+require 'pdk/util'
+require 'pathname'
+
+module PDK
+  module Validate
+    class YAML
+      class Syntax < BaseValidator
+        IGNORE_DOTFILES = false
+        YAML_WHITELISTED_CLASSES = [Symbol].freeze
+
+        def self.name
+          'yaml-syntax'
+        end
+
+        def self.pattern
+          [
+            '**/*.yaml',
+            '*.yaml',
+            '**/*.yml',
+            '*.yml',
+          ]
+        end
+
+        def self.spinner_text(_targets = [])
+          _('Checking YAML syntax (%{targets}).') % {
+            targets: pattern,
+          }
+        end
+
+        def self.create_spinner(targets = [], options = {})
+          return unless PDK::CLI::Util.interactive?
+          options = options.merge(PDK::CLI::Util.spinner_opts_for_platform)
+
+          exec_group = options[:exec_group]
+          @spinner = if exec_group
+                       exec_group.add_spinner(spinner_text(targets), options)
+                     else
+                       TTY::Spinner.new("[:spinner] #{spinner_text(targets)}", options)
+                     end
+          @spinner.auto_spin
+        end
+
+        def self.stop_spinner(exit_code)
+          if exit_code.zero? && @spinner
+            @spinner.success
+          elsif @spinner
+            @spinner.error
+          end
+        end
+
+        def self.invoke(report, options = {})
+          targets, skipped, invalid = parse_targets(options)
+
+          process_skipped(report, skipped)
+          process_invalid(report, invalid)
+
+          return 0 if targets.empty?
+
+          return_val = 0
+          create_spinner(targets, options)
+
+          targets.each do |target|
+            next unless File.file?(target)
+
+            unless File.readable?(target)
+              report.add_event(
+                file:     target,
+                source:   name,
+                state:    :failure,
+                severity: 'error',
+                message:  _('Could not be read.'),
+              )
+              return_val = 1
+              next
+            end
+
+            begin
+              ::YAML.safe_load(File.read(target), YAML_WHITELISTED_CLASSES, [], true)
+
+              report.add_event(
+                file:     target,
+                source:   name,
+                state:    :passed,
+                severity: 'ok',
+              )
+            rescue Psych::SyntaxError => e
+              report.add_event(
+                file:     target,
+                source:   name,
+                state:    :failure,
+                severity: 'error',
+                line:     e.line,
+                column:   e.column,
+                message:  _('%{problem} %{context}') % {
+                  problem: e.problem,
+                  context: e.context,
+                },
+              )
+              return_val = 1
+            rescue Psych::DisallowedClass => e
+              report.add_event(
+                file:     target,
+                source:   name,
+                state:    :failure,
+                severity: 'error',
+                message:  _('Unsupported class: %{message}') % {
+                  message: e.message,
+                },
+              )
+              return_val = 1
+            end
+          end
+
+          stop_spinner(return_val)
+          return_val
+        end
+      end
+    end
+  end
+end

data/lib/pdk/validate/yaml_validator.rb

@@ -0,0 +1,31 @@
+require 'pdk'
+require 'pdk/cli/exec'
+require 'pdk/validate/base_validator'
+require 'pdk/validate/yaml/syntax'
+
+module PDK
+  module Validate
+    class YAMLValidator < BaseValidator
+      def self.name
+        'yaml'
+      end
+
+      def self.validators
+        [
+          PDK::Validate::YAML::Syntax,
+        ]
+      end
+
+      def self.invoke(report, options = {})
+        exit_code = 0
+
+        validators.each do |validator|
+          exit_code = validator.invoke(report, options)
+          break if exit_code != 0
+        end
+
+        exit_code
+      end
+    end
+  end
+end

data/lib/pdk/validate.rb

@@ -2,11 +2,18 @@ require 'pdk/validate/metadata_validator'
 require 'pdk/validate/puppet_validator'
 require 'pdk/validate/ruby_validator'
 require 'pdk/validate/tasks_validator'
+require 'pdk/validate/yaml_validator'
 
 module PDK
   module Validate
     def self.validators
-      @validators ||= [MetadataValidator, PuppetValidator, RubyValidator, TasksValidator].freeze
+      @validators ||= [
+        MetadataValidator,
+        YAMLValidator,
+        PuppetValidator,
+        RubyValidator,
+        TasksValidator,
+      ].freeze
     end
 
     class ParseOutputError < StandardError; end

data/lib/pdk/version.rb

@@ -1,4 +1,4 @@
 module PDK
-  VERSION = '1.8.0'.freeze
+  VERSION = '1.9.1'.freeze
   TEMPLATE_REF = VERSION
 end

data/locales/pdk.pot

@@ -1,16 +1,16 @@
 # SOME DESCRIPTIVE TITLE.
-# Copyright (C) 2018 Puppet, Inc.
+# Copyright (C) 2019 Puppet, Inc.
 # This file is distributed under the same license as the puppet development kit package.
-# FIRST AUTHOR <EMAIL@ADDRESS>, 2018.
+# FIRST AUTHOR <EMAIL@ADDRESS>, 2019.
 #
 #, fuzzy
 msgid ""
 msgstr ""
-"Project-Id-Version: puppet development kit v1.7.1-49-g6159e02fac\n"
+"Project-Id-Version: puppet development kit v1.9.0-15-g264d3c07ab\n"
 "\n"
 "Report-Msgid-Bugs-To: docs@puppet.com\n"
-"POT-Creation-Date: 2018-11-27 17:00-0700\n"
-"PO-Revision-Date: 2018-11-27 17:00-0700\n"
+"POT-Creation-Date: 2019-03-01 11:43-0700\n"
+"PO-Revision-Date: 2019-03-01 11:43-0700\n"
 "Last-Translator: FULL NAME <EMAIL@ADDRESS>\n"
 "Language-Team: LANGUAGE <LL@li.org>\n"
 "Language: \n"
@@ -533,7 +533,7 @@ msgstr ""
 
 #: ../lib/pdk/cli/validate.rb:8
 msgid ""
-"Run metadata, Puppet, Ruby, or Tasks validation.\n"
+"Run metadata, YAML, Puppet, Ruby, or Tasks validation.\n"
 "\n"
 "[validators] is an optional comma-separated list of validators to use. If not specified, all validators are used. Note that when using PowerShell, the list of validators must be enclosed in single quotes.\n"
 "\n"
@@ -695,11 +695,11 @@ msgid "If there is a public issue tracker for this module, enter its URL here."
 msgstr ""
 
 #: ../lib/pdk/generate/module.rb:258
-msgid "create"
+msgid "update"
 msgstr ""
 
 #: ../lib/pdk/generate/module.rb:258
-msgid "update"
+msgid "create"
 msgstr ""
 
 #: ../lib/pdk/generate/module.rb:259
@@ -878,23 +878,23 @@ msgstr ""
 msgid "The \"puppet\" requirement in module metadata does not specify a \"version_requirement\"."
 msgstr ""
 
-#: ../lib/pdk/module/metadata.rb:180
+#: ../lib/pdk/module/metadata.rb:181
 msgid "Field must be a dash-separated user name and module name."
 msgstr ""
 
-#: ../lib/pdk/module/metadata.rb:182
+#: ../lib/pdk/module/metadata.rb:183
 msgid "Module name must contain only alphanumeric or underscore characters."
 msgstr ""
 
-#: ../lib/pdk/module/metadata.rb:184
+#: ../lib/pdk/module/metadata.rb:185
 msgid "Module name must begin with a letter."
 msgstr ""
 
-#: ../lib/pdk/module/metadata.rb:186
+#: ../lib/pdk/module/metadata.rb:187
 msgid "Namespace must contain only alphanumeric characters."
 msgstr ""
 
-#: ../lib/pdk/module/metadata.rb:189
+#: ../lib/pdk/module/metadata.rb:190
 msgid "Invalid 'name' field in metadata.json: %{err}"
 msgstr ""
 
@@ -933,14 +933,14 @@ msgid "The directory '%{dir}' doesn't exist"
 msgstr ""
 
 #: ../lib/pdk/module/templatedir.rb:273
-msgid "'%{file}' is not a valid YAML file: %{message}"
+msgid "'%{file}' is not a valid YAML file: %{problem} %{context} at line %{line} column %{column}"
 msgstr ""
 
-#: ../lib/pdk/module/templatedir.rb:301
+#: ../lib/pdk/module/templatedir.rb:308
 msgid "Unable to set HEAD of git repository at '%{repo}' to ref:'%{ref}'."
 msgstr ""
 
-#: ../lib/pdk/module/templatedir.rb:306
+#: ../lib/pdk/module/templatedir.rb:314
 msgid "Unable to clone git repository at '%{repo}' into '%{dest}'."
 msgstr ""
 
@@ -1024,7 +1024,7 @@ msgstr ""
 msgid "Trace must be an Array of stack trace lines."
 msgstr ""
 
-#: ../lib/pdk/template_file.rb:62
+#: ../lib/pdk/template_file.rb:68
 msgid "'%{template}' is not a readable file"
 msgstr ""
 
@@ -1084,7 +1084,7 @@ msgstr ""
 msgid "Package basedir requested for non-package install."
 msgstr ""
 
-#: ../lib/pdk/util.rb:210
+#: ../lib/pdk/util.rb:222
 msgid "Unable to access the previously used template '%{template}', using the default template instead."
 msgstr ""
 
@@ -1214,23 +1214,23 @@ msgstr ""
 msgid "Failed to call GetLongPathName"
 msgstr ""
 
-#: ../lib/pdk/validate/base_validator.rb:102
+#: ../lib/pdk/validate/base_validator.rb:108
 msgid "Invoking %{cmd}"
 msgstr ""
 
-#: ../lib/pdk/validate/base_validator.rb:107
+#: ../lib/pdk/validate/base_validator.rb:113
 msgid "%{validator}: Skipped '%{target}'. Target does not contain any files to validate (%{pattern})."
 msgstr ""
 
-#: ../lib/pdk/validate/base_validator.rb:111
+#: ../lib/pdk/validate/base_validator.rb:117
 msgid "Target does not contain any files to validate (%{pattern})."
 msgstr ""
 
-#: ../lib/pdk/validate/base_validator.rb:120
+#: ../lib/pdk/validate/base_validator.rb:126
 msgid "%{validator}: Skipped '%{target}'. Target file not found."
 msgstr ""
 
-#: ../lib/pdk/validate/base_validator.rb:124
+#: ../lib/pdk/validate/base_validator.rb:130
 msgid "File does not exist."
 msgstr ""
 
@@ -1246,7 +1246,7 @@ msgstr ""
 msgid "Checking metadata syntax (%{targets})."
 msgstr ""
 
-#: ../lib/pdk/validate/metadata/metadata_syntax.rb:69 ../lib/pdk/validate/tasks/metadata_lint.rb:77
+#: ../lib/pdk/validate/metadata/metadata_syntax.rb:69 ../lib/pdk/validate/tasks/metadata_lint.rb:77 ../lib/pdk/validate/yaml/syntax.rb:74
 msgid "Could not be read."
 msgstr ""
 
@@ -1254,7 +1254,7 @@ msgstr ""
 msgid "Checking Puppet manifest style (%{pattern})."
 msgstr ""
 
-#: ../lib/pdk/validate/puppet/puppet_syntax.rb:45
+#: ../lib/pdk/validate/puppet/puppet_syntax.rb:47
 msgid "Checking Puppet manifest syntax (%{pattern})."
 msgstr ""
 
@@ -1281,3 +1281,15 @@ msgstr ""
 #: ../lib/pdk/validate/tasks/name.rb:23
 msgid "Checking task names (%{targets})."
 msgstr ""
+
+#: ../lib/pdk/validate/yaml/syntax.rb:28
+msgid "Checking YAML syntax (%{targets})."
+msgstr ""
+
+#: ../lib/pdk/validate/yaml/syntax.rb:97
+msgid "%{problem} %{context}"
+msgstr ""
+
+#: ../lib/pdk/validate/yaml/syntax.rb:109
+msgid "Unsupported class: %{message}"
+msgstr ""

metadata

@@ -1,29 +1,35 @@
 --- !ruby/object:Gem::Specification
 name: pdk-akerl
 version: !ruby/object:Gem::Version
-  version: 1.8.0.1
+  version: 1.9.1.1
 platform: ruby
 authors:
 - Puppet, Inc.
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2019-01-04 00:00:00.000000000 Z
+date: 2019-03-08 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 1.15.0
+    - - "<"
       - !ruby/object:Gem::Version
-        version: 2.0.0
+        version: 3.0.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: 2.0.0
+        version: 1.15.0
+    - - "<"
+      - !ruby/object:Gem::Version
+        version: 3.0.0
 - !ruby/object:Gem::Dependency
   name: cri
   requirement: !ruby/object:Gem::Requirement
@@ -178,6 +184,48 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: 0.2.1
+- !ruby/object:Gem::Dependency
+  name: ffi
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 1.9.0
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 1.9.0
+- !ruby/object:Gem::Dependency
+  name: hitimes
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - '='
+      - !ruby/object:Gem::Version
+        version: 1.3.0
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - '='
+      - !ruby/object:Gem::Version
+        version: 1.3.0
+- !ruby/object:Gem::Dependency
+  name: net-ssh
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 4.2.0
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 4.2.0
 - !ruby/object:Gem::Dependency
   name: deep_merge
   requirement: !ruby/object:Gem::Requirement
@@ -276,6 +324,8 @@ files:
 - lib/pdk/validate/tasks/metadata_lint.rb
 - lib/pdk/validate/tasks/name.rb
 - lib/pdk/validate/tasks_validator.rb
+- lib/pdk/validate/yaml/syntax.rb
+- lib/pdk/validate/yaml_validator.rb
 - lib/pdk/version.rb
 - locales/config.yaml
 - locales/pdk.pot
@@ -297,7 +347,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.0.2
+rubygems_version: 3.0.1
 signing_key: 
 specification_version: 4
 summary: A key part of the Puppet Development Kit, the shortest path to better modules