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

Markdown extension matching matches only exact matches #1382

Merged
merged 2 commits into from Aug 6, 2013

Conversation

Projects
None yet
3 participants
@gabysbrain
Contributor

gabysbrain commented Aug 6, 2013

The current markdown processor is too general I think. Currently if you have, for example, a .md extension added to the markdown_ext list then any extension containing md will also be processed by the markdown converter. I don't think this is expected behavior and it prevents me from using, say, .mmd files to the multi-markdown converter.

This change fixes that.

@parkr

This comment has been minimized.

Member

parkr commented Aug 6, 2013

LGTM. @mattr-?

@parkr

This comment has been minimized.

Member

parkr commented Aug 6, 2013

As a note, we'll want to add something to jekyll doctor for this extname.

parkr added a commit that referenced this pull request Aug 6, 2013

Merge pull request #1382 from torsneyt/fix_md_ext
Markdown extension matching matches only exact matches

@parkr parkr merged commit ae218dd into jekyll:master Aug 6, 2013

1 check passed

default The Travis CI build passed
Details

parkr added a commit that referenced this pull request Aug 6, 2013

parkr added a commit that referenced this pull request Aug 28, 2013

@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.