From 09a17298abf2f6e132aec3ebb110df1d8110748a Mon Sep 17 00:00:00 2001 From: Mike Linksvayer Date: Wed, 3 Feb 2016 09:24:05 -0800 Subject: [PATCH] Fix #341 tests that broke post Jekyll 3 upgrade - explicitly include jekyll-coffeescript needed per http://jekyllrb.com/docs/upgrading/2-to-3/ - option to html-proofer to ignore script embeds; uncertain why warnings appeared post upgrade as it doesn't seem that either jekyll-seo-tag or html-proofer got new versions in https://github.com/github/choosealicense.com/pull/339/files but suppressed now in any case. --- Rakefile | 1 + _config.yml | 1 + 2 files changed, 2 insertions(+) diff --git a/Rakefile b/Rakefile index f0f88ec..d6a2080 100644 --- a/Rakefile +++ b/Rakefile @@ -11,6 +11,7 @@ task :test do sh "bundle exec jekyll build --trace" Rake::Task["spec"].invoke HTML::Proofer.new("./_site", :check_html => true, + :validation => { :ignore_script_embeds => true }, :href_swap => { %r{http://choosealicense.com} => "" }).run end diff --git a/_config.yml b/_config.yml index e96f3da..a6d14c1 100644 --- a/_config.yml +++ b/_config.yml @@ -41,6 +41,7 @@ gems: - jekyll-sitemap - jekyll-redirect-from - jekyll-seo-tag + - jekyll-coffeescript sass: sass_dir: _sass