New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
convertible: use Document::YAML_FRONT_MATTER_REGEXP to parse transformable files #4786
Conversation
@@ -41,7 +41,7 @@ def read_yaml(base, name, opts = {}) | |||
begin | |||
self.content = File.read(site.in_source_dir(base, name), | |||
Utils.merged_file_read_opts(site, opts)) | |||
if content =~ /\A(---\s*\n.*?\n?)^((---|\.\.\.)\s*$\n?)/m | |||
if content =~ Document::YAML_FRONT_MATTER_REGEXP |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is only used in Convertible, why is on Document if it's only used here?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Take that back.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You're OK with it then?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I see no problem.
😄 @jekyllbot: merge +bug |
* master: (58 commits) Update history to reflect merge of #4792 [ci skip] Update history to reflect merge of #4793 [ci skip] Update history to reflect merge of #4804 [ci skip] Update history to reflect merge of #4754 [ci skip] Update history to reflect merge of #4813 [ci skip] Added missing single quote on rsync client side command Add v3.0.4 and v3.1.3 to the history. Fixed typo Add jekyll-autoprefixer plugin Explicitly require Filters rather than implicitly. Update history to reflect merge of #4786 [ci skip] Update history to reflect merge of #4789 [ci skip] updates example domain in config template Globalize Jekyll's Filters. Update JRuby to 9.0.5.0; Drop the double digit test. Update Rack-Jekyll Heroku deployment blog post url convertible: use Document::YAML_FRONT_MATTER_REGEXP to parse transformable files Update history to reflect merge of #4734 [ci skip] Update history to reflect merge of #4478 [ci skip] Fix rubocop warning. ...
Replaces #4736, which was my commit in a PR but with some git conflict.
/cc @jekyll/build