Renderer#output_ext: honor folders when looking for ext #4401

Merged
merged 1 commit into from Jan 25, 2016

Conversation

Projects
None yet
2 participants
@parkr
Member

parkr commented Jan 25, 2016

Previously, even if the document permalink was a folder, it would look for
an extension on that. For example, if I have:

permalink: "/new-version-jekyll-v3.0.0/"

the output_ext would be ".0". Now, the output_ext honors the trailing
slash and will report based on the converters instead.

Fixes #4397.

Renderer#output_ext: honor folders when looking for ext
Previously, even if the document permalink was a folder, it would look for
an extension on that.  For example, if I have:

    permalink: "/new-version-jekyll-v3.0.0/"

the output_ext would be ".0".  Now, the output_ext honors the trailing
slash and will report based on the converters instead.

@parkr parkr added the fix label Jan 25, 2016

@parkr parkr added this to the 3.1.1 milestone Jan 25, 2016

@parkr

This comment has been minimized.

Show comment
Hide comment
@parkr

parkr Jan 25, 2016

Member

@jekyllbot: merge +dev

Member

parkr commented Jan 25, 2016

@jekyllbot: merge +dev

jekyllbot added a commit that referenced this pull request Jan 25, 2016

@jekyllbot jekyllbot merged commit a4ba824 into master Jan 25, 2016

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details

@jekyllbot jekyllbot deleted the fix-documents-with-permalink-with-slash branch Jan 25, 2016

jekyllbot added a commit that referenced this pull request Jan 25, 2016

@jekyll jekyll locked and limited conversation to collaborators Feb 27, 2017

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.