Question about github markdown filter (low priority!) about html-pipeline HOT 5 CLOSED

p.s. my gem is quite rough-and-ready... but I did write a failing spec for the quoting issue... patrickdavey/vimwiki_markdown@3299639

from html-pipeline.

@patrickdavey hmmm, I don't know off the top of my head, but what is the output if you ran it directly through the github-markdown gem? I noticed your diff also bumped the gem to a specific version of github-markdown; Does the problem only appear for specific versions?

from html-pipeline.

Hi @jch , I wrote a quick spec file and pushed it up to a repo - spec itself is here:
https://github.com/patrickdavey/githubmarkuptest/blob/master/basic_spec.rb

But to summarise, The only way I could find which correctly gave me a blockquote without a <br> in the middle was:

GitHub::Markup.render('README.markdown', markdown_string)

So.. I guess the pipeline using the HTML::Pipeline::MarkdownFilter is using GitHub::Markdown ultimately which is why I'm seeing the difference.

I guess this means I'd need to use Github::Markup to generate the html from the markdown first, and then pass that into the HTML::Pipeline, which is what they say they're doing over here: https://github.com/github/markup in the README.

Assuming that's correct, please just close this issue. I guess the other thing would be to create a HTML::Pipeline::GithubFlavouredMarkupFilter or similar? If that sounds like something which is useful let me know (and I'll try to make time!)

Thanks for the help, your gem is invaluable! 👍

Best,
Patrick

from html-pipeline.

Yip, changing the call to

    github_markup = GitHub::Markup.render('README.markdown', markdown_body)
    pipeline = HTML::Pipeline.new [
      HTML::Pipeline::SyntaxHighlightFilter
    ]   
    result = pipeline.call(github_markup)
    result[:output].to_s

works fine. Assuming that's the way I should always have been using it (quite probably I misunderstood something) please just close this issue.

from html-pipeline.

@patrickdavey nice detective work!

I guess the other thing would be to create a HTML::Pipeline::GithubFlavouredMarkupFilter or similar?

If you're looking for GitHub flavored markdown, you can pass :gfm => true as an option to the filter:

filter = HTML::Pipeline::MarkdownFilter.new("Hi **world**!", :gfm => true)
filter.call

Closing now because you've fixed your issue, but feel free to comment further.

from html-pipeline.