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

Markdown fails to parse Autolinks #9564

Closed
mpastell opened this issue Jan 2, 2015 · 1 comment · Fixed by #11299
Closed

Markdown fails to parse Autolinks #9564

mpastell opened this issue Jan 2, 2015 · 1 comment · Fixed by #11299
Labels
bug Indicates an unexpected problem or unintended behavior docsystem The documentation building system

Comments

@mpastell
Copy link
Contributor

mpastell commented Jan 2, 2015

It seems that Markdown.parse in 0.4 doesn't parse autolinks (http://spec.commonmark.org/0.14/#autolink).

julia> Markdown.parse("<http://github.com>")
  <http://github.com>

As a related question, should I report Markdown bugs here or to Markdown.jl? This is the third Markdown bug I come across today and reported the two others to Markdown.jl, but they also exist in base.Markdown.

@ihnorton ihnorton added docs This change adds or pertains to documentation bug Indicates an unexpected problem or unintended behavior labels Jan 6, 2015
@hayd
Copy link
Member

hayd commented May 5, 2015

This is the correct place for Markdown bugs.

This is a pretty easy enhancement, we can simply parse this to the same object as if we'd seen [http://github.com](http://github.com). I have a patch to add this and auto emails too.

@quinnj quinnj added docsystem The documentation building system and removed docs This change adds or pertains to documentation labels Jun 28, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Indicates an unexpected problem or unintended behavior docsystem The documentation building system
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants