Add alternative .md file extension to detection

[dlo]

No description provided.

[lucapette]

This is not enough because the subtype will be md that Vim will not be able to recognize. I tried to make a pull-request but I wasn't sure @tpope would like it because there were some ugly duplication in my code.

Actually, I'd like to know @tpope opinion on that. Is there a good way to have that? It could be nice. Should we extract subtype recognition in a function?

[dlo]

Bump?

[tpope]

Oh, um, @lucapette is right, isn't he? I haven't actually tried.

I'd also be interested to see where you found this .md-as-liquid in the wild.

[dlo]

Maybe it's just something I made up? Not sure, but I've been using .md as the markdown extension since forever. Didn't think that it would need to change as a result of the subtype (md-as-liquid) as opposed to just markdown.

Jekyll--the software I'm using--parses 4 extensions for markdown: .markdown, .mkd, .mkdn, and .md.

https://github.com/mojombo/jekyll/blob/master/lib/jekyll.rb#L70

[tpope]

You didn't invent that extension. GitHub did. Along with the other three you cite.

The problem is that liquid.vim uses the extension to determine a secondary syntax highlighting to activate. If you rename one of those .md files to .markdown, you should see a pretty obvious difference.

Here's one way to fix it:

autocmd BufNewFile,BufRead *.md
      \ if getline(1) == '---' |
      \   let b:liquid_subtype = 'markdown' |
      \   set ft=liquid |
      \ endif

I guess if you were to verify that works and wrap it up into a commit, I'd accept it. But I wish people would just use .markdown. .md already means "modula2" in Vim, and that ain't going away.

[jumoel]

I added a pull request for this in #5. It works for *.md files.

[tpope]

Resolved.