Merge pull request #1284 from mojombo/include-exclude-deprecation-simplification

Include exclude deprecation simplification
This commit is contained in:
Parker Moore 2013-07-08 10:49:09 -07:00
commit c4fd5ddc8b
1 changed files with 8 additions and 15 deletions

View File

@ -185,22 +185,15 @@ module Jekyll
config.delete('server_port') config.delete('server_port')
end end
if config.has_key?('exclude') && config['exclude'].is_a?(String) %w[include exclude].each do |option|
Jekyll.logger.warn "Deprecation:", "The 'exclude' configuration option" + if config.fetch(option, []).is_a?(String)
" must now be specified as an array, but you specified" + Jekyll.logger.warn "Deprecation:", "The '#{option}' configuration option" +
" a string. For now, we've treated the string you provided" + " must now be specified as an array, but you specified" +
" as a list of comma-separated values." " a string. For now, we've treated the string you provided" +
config['exclude'] = csv_to_array(config['exclude']) " as a list of comma-separated values."
config[option] = csv_to_array(config[option])
end
end end
if config.has_key?('include') && config['include'].is_a?(String)
Jekyll.logger.warn "Deprecation:", "The 'include' configuration option" +
" must now be specified as an array, but you specified" +
" a string. For now, we've treated the string you provided" +
" as a list of comma-separated values."
config['include'] = csv_to_array(config['include'])
end
config config
end end