Skip to content
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

Add jekyll-commonmark as an available converter in core #6547

Closed
2 tasks done
pathawks opened this issue Nov 12, 2017 · 15 comments
Closed
2 tasks done

Add jekyll-commonmark as an available converter in core #6547

pathawks opened this issue Nov 12, 2017 · 15 comments

Comments

@pathawks
Copy link
Member

Should we add jekyll-commonmark to core?

We could start by adding a stub MarkDown converter to lib/jekyll/converters/markdown/ like we have for rdiscount and redcarpet?

/cc: @jekyll/core @jekyll/plugin-core

@ashmaroli
Copy link
Member

Should we add jekyll-commonmark to core?

Maybe start testing on Windows before that..?

@pathawks
Copy link
Member Author

@ashmaroli Could you set that up for jekyll/jekyll-commonmark? That would be awesome 😍:tada:

@ashmaroli
Copy link
Member

Could you set that up for jekyll/jekyll-commonmark? That would be awesome

Sure.. TODO for tomorrow 😃

@pathawks
Copy link
Member Author

@ashmaroli
Copy link
Member

ashmaroli commented Nov 13, 2017

Required:

  • Plugin Test-suite
  • Compatibility with Linux, OSX, Windows

@pathawks
Copy link
Member Author

Windows Build status

@ashmaroli
Copy link
Member

CI is only running RuboCop actually.. there are no RSpec files to be run..

@pathawks
Copy link
Member Author

Oh! 😅

@jekyllbot jekyllbot added the stale Nobody stepped up to work on this issue. label Jan 21, 2018
@jekyllbot

This comment has been minimized.

@pathawks
Copy link
Member Author

@jekyllbot jekyllbot removed the stale Nobody stepped up to work on this issue. label Jan 21, 2018
@parkr
Copy link
Member

parkr commented Jan 21, 2018

@pathawks Excellent! Want to submit a PR to make this first-class in Jekyll? E.g. markdown: cmark or whatever it’s now called would activate this plugin.

One thing to consider as well is a markdown: GFM which adds some github extensions to this plugin as settings to make it as close to the markdown experience on GitHub as possible. That can come later though.

@DirtyF
Copy link
Member

DirtyF commented Jan 21, 2018

@pathawks
Copy link
Member Author

@DirtyF Yes, I have been monitoring that as well.

If Github is doing this on their own, maybe we should just use their implementation instead?

@jekyllbot jekyllbot added the stale Nobody stepped up to work on this issue. label Mar 21, 2018
@Crunch09 Crunch09 added pinned and removed stale Nobody stepped up to work on this issue. labels Mar 21, 2018
@jekyll jekyll deleted a comment from jekyllbot Mar 22, 2018
@ashmaroli
Copy link
Member

@pathawks What's the status on this..?

@parkr
Copy link
Member

parkr commented Feb 12, 2019

I'd really love to merge the jekyll-commonmark-ghpages code upstream and just use jekyll-commonmark!

@jekyll jekyll locked and limited conversation to collaborators Aug 20, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

6 participants