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

Allow parentheses inside of links #222

Merged
merged 4 commits into from
Dec 6, 2015
Merged

Allow parentheses inside of links #222

merged 4 commits into from
Dec 6, 2015

Conversation

icco
Copy link
Contributor

@icco icco commented Nov 29, 2015

Fixes #116. Most code originally by @miekg.

@icco
Copy link
Contributor Author

icco commented Dec 3, 2015

Thoughts @rtfb?

@rtfb
Copy link
Collaborator

rtfb commented Dec 3, 2015

Ah, sorry I haven't replied. I've taken a look and it looks good on the surface, but I wanted to give it a closer look within context. I'll try to do that tomorrow.

@icco
Copy link
Contributor Author

icco commented Dec 5, 2015

Fixed @rtfb, thanks for the comments!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Link parsing doesn't correctly match URLs when they include brackets
2 participants