-
Notifications
You must be signed in to change notification settings - Fork 113
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
Change Markdown rendering to GitHub-flavored renderer #1321
Comments
Code syntax highlighting will have to come from Coderay, Rouge, or some other additional text processor to markup the text passed to our custom renderer. Displaying HTML comments in the markdown appears to be the outcome of escaping the HTML in the custom Renderer. |
We will not be getting to this in the next 6 months, but am open to revisiting in the future. |
Adding options—that I'm not sure how I missed earlier—to this rollup in case it gets reopened in the future:
|
Currently tables that use html table syntax don't render. With #1622 this is fixed so tables do render properly. |
We've just noticed another subtle issue with the renderer.
|
I can picture how a relative URL like that which works in the context of GitHub won't work on Supermarket because the cookbook's How much work do we want to put into Supermarket duplicating the GitHub experience and content? |
I think it would be nice for it to render the markdown to not look broken, even if the link is borked |
Aaaah. I see now. Gotcha. 🤔 |
Relative links got nerfed when we switched to the "safe" version of the markdown renderer. Relative links don't start with a string on the library's safe allow list. I think while Supermarket is using Redcarpet for markdown rendering, it won't be supporting rendering relative links as links. There's an attempt at switching over to GitHub's own markdown rendering pipeline, but the test failures there need addressing. |
I think we have a good work around then, we just won't make them relative and always point at master/blob |
I think we've solved this one enough at this point that we can potentially close it out. If we run into additional rendering issues we should raise them 1 at a time to track/fix |
A couple of issues have come in that boil down to "Supermarket's Markdown renderer is not Github-flavored." Given the prevalence of community cookbooks whose READMEs are forged in the fires of GitHub, it is reasonable to expect that the README will render similarly on GitHub and on Supermarket.
Opening this issue as a roll-up to investigate configuring the existing renderer or changing it out for something that acts like GitHub's.
Some things to check:
<!-- -->
HTML commentsrenders tablesWe do this already.The text was updated successfully, but these errors were encountered: