-
Notifications
You must be signed in to change notification settings - Fork 18
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
How does this differ from jekyll-commonmark-ghpages? #39
Comments
It is my understanding that the commonmarker Gem that we require is merely an FFI wrapper around libcmark. If that is no longer the case, we can certainly update this project or at least update the Readme. To answer your broader question: ifI remember correctly, we both started at around the same time. My intent when starting this project was (primarily) to utilize the fastest Markdown processor available in Ruby and (secondarily) conform to the then emerging CommonMark spec. I imagine the GitHub Pages team is more focused on building a processor that behaves similar to the GitHub Flavored Markdown processor used on the site. The fact that we are both using the same tech under the hood is kind of an implementation detail. 🤷🏼♂️ I hope this answers your question. 👍🏼 |
Thanks for the quick reply @pathawks. From the
Quickly skimming through the commit history, it looks like it may have originally used |
I've opened #38 with a few README tweaks to make the current state of things a little clearer. |
(similar question posted at github/jekyll-commonmark-ghpages#17)
I notice that while the readme for this project talks about using
libcmark
, it actually appears to achieve this via thecommonmarker
gem.Looking at
commonmarker
, it talks about being a ruby wrapper forlibcmark-gfm
, which apparently includes all of the GFM extensions already:This would make me think that
jekyll-commonmark
by itself already supports GFM.--
Adding
jekyll-commonmark
to my Gemfile resulted in the following in myGemfile.lock
:Resetting, then adding
jekyll-commonmark-ghpages
to myGemfile
resulted in the following in myGemfile.lock
:By the looks of this, by using
jekyll-commonmark
instead ofjekyll-commonmark-ghpages
I would actually be getting a more up to date version of commonmarker (and thus the underlying c implementation), that supports GFM. Which to me, at face value, is definitely not what I would have expected.The text was updated successfully, but these errors were encountered: