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

Protect links, surrounding them with angle brackets #50

Merged
merged 3 commits into from
Dec 23, 2014

Conversation

JocelynDelalande
Copy link
Contributor

to avoid breaking them with line-wrap (inline and reference).

Particularly useful with text mail clients. Complies with markdown syntax, tested to work in markdown.js and pandoc backends.

… them with

line-wrap (inline and reference).
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.07%) when pulling 22b2927 on oasiswork:jd-protect-links into 100ae75 on Alir3z4:master.

dest="protect_links",
action="store_true",
default=config.PROTECT_LINKS,
help="protect links from line breaks surrounding them with angle brackets")
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

pep8 please ;)

@Alir3z4
Copy link
Owner

Alir3z4 commented Dec 22, 2014

@JocelynDelalande, @stefanor can we consider this pull-request as a fix for #38 as well ?

@stefanor
Copy link
Contributor

It's more of a possible work around than a fix. There are many uses for this besides e-mail. And my text-mode mail client doesn't treat <> wrapped URLs specially...

@Alir3z4
Copy link
Owner

Alir3z4 commented Dec 23, 2014

@stefanor Thanks for the comment.

@JocelynDelalande could you please solve the merge conflicts? I'll merge and release by end of the day.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.07%) when pulling b7d3e5b on oasiswork:jd-protect-links into b4d4191 on Alir3z4:master.

Alir3z4 added a commit that referenced this pull request Dec 23, 2014
Protect links, surrounding them with angle brackets 
Thanks @JocelynDelalande
@Alir3z4 Alir3z4 merged commit 6a188c7 into Alir3z4:master Dec 23, 2014
@n11c n11c deleted the jd-protect-links branch September 13, 2016 09:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants