summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorDavid Schmitt <david.schmitt@puppetlabs.com>2015-06-01 12:21:59 +0100
committerDavid Schmitt <david.schmitt@puppetlabs.com>2015-06-01 12:21:59 +0100
commitad7d12e460599995b5b53b9d05881a9a547425d0 (patch)
tree10e560fff01bcf66a47e11eb39cd870cbdc7b691
parent65b56c711dc882d7d7f8e26c4e020a6beb297c75 (diff)
Remove duplicate rake task and enable metadata.json linting
The :validate task is already provided by puppetlabs_spec_helper/rake_tasks and would check files twice. The metadata.json linting is just good form.
-rw-r--r--Gemfile1
-rw-r--r--Rakefile12
2 files changed, 1 insertions, 12 deletions
diff --git a/Gemfile b/Gemfile
index 82a5204..9d38f35 100644
--- a/Gemfile
+++ b/Gemfile
@@ -17,6 +17,7 @@ group :development, :unit_tests do
gem 'mocha', :require => false
gem 'puppetlabs_spec_helper', :require => false
gem 'puppet-lint', :require => false
+ gem 'metadata-json-lint', :require => false
gem 'pry', :require => false
gem 'simplecov', :require => false
end
diff --git a/Rakefile b/Rakefile
index 4ed1327..13ee3d9 100644
--- a/Rakefile
+++ b/Rakefile
@@ -4,15 +4,3 @@ require 'puppet-lint/tasks/puppet-lint'
PuppetLint.configuration.send('disable_80chars')
PuppetLint.configuration.ignore_paths = ["spec/**/*.pp", "pkg/**/*.pp"]
-desc "Validate manifests, templates, and ruby files in lib."
-task :validate do
- Dir['manifests/**/*.pp'].each do |manifest|
- sh "puppet parser validate --noop #{manifest}"
- end
- Dir['lib/**/*.rb'].each do |lib_file|
- sh "ruby -c #{lib_file}"
- end
- Dir['templates/**/*.erb'].each do |template|
- sh "erb -P -x -T '-' #{template} | ruby -c"
- end
-end