-
Notifications
You must be signed in to change notification settings - Fork 236
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
Switch Markdown rendering from Text::MultiMarkdown to CommonMark #2894
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Generally looks good.
We'll need to wait to merge this until we can change our deployments to include libcmark. It also looks like it needs to get installed in CI.
bfddd9c
to
5a210bb
Compare
The docker container has no |
bbcd61d
to
aed87be
Compare
The content of this looks good. I've marked it as draft so that it doesn't get merged yet, until we can get the front end switched over to our new systems. |
aed87be
to
2d23d04
Compare
2d23d04
to
79767ab
Compare
4c4c19c
to
7b3fac8
Compare
Custom rendering function to add `id` attribute to headings. Fixes metacpan#2312
7b3fac8
to
eae0e33
Compare
Custom rendering function to add
id
attribute to headings.Fixes #2312