-
Notifications
You must be signed in to change notification settings - Fork 26
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
Possibly faulty parsing of close references #159
Comments
Ty Lukas But maybe this is a quick fix? I'll keep you posted |
ex_doc (or rather EarmarkParser) does not consider the whitespace between reference links significant so joins multiple references into a single link. Many MD standards seem to work this way, though the majority do not. Anyway we need to fix this so that the links render properly, see RobertDober/earmark_parser#159 for details on when/if this issue is solved.
ex_doc (or rather EarmarkParser) does not consider the whitespace between reference links significant so joins multiple references into a single link. Many MD standards seem to work this way, though the majority do not. Anyway we need to fix this so that the links render properly, see RobertDober/earmark_parser#159 for details on when/if this issue is solved.
I noticed it while reading our release notes where we list which issues and PRs a certain change was introduced by. It was an easy fix on our side and the scripts that generate the information have been updated to always insert a comma, so no rush on our end. |
Good because while GFM compliance is more or less expected it is always risky to make patch releases which change behviour (on extreme edge case, but still).
|
My bad no, I'll keep it open |
The parser seems to think that
[a] [b]
is the same as[a][b]
.The standard is a bit undecided. Gruber Markdown seems to do the same interpretation as earmark_parser, while github (and in extentions commonmark) thinks that it should be treated as two separate references.
I'm opening this issue to just track the discepency and see if you think we should do something about it.
The text was updated successfully, but these errors were encountered: