#729 Re expression removed from markdown rendering
Merged by pingou. Opened by cverna.
cverna/pagure pagure_667  into  master

Download 729.patch
no initial comment

Fixes for https://pagure.io/pagure/issue/667

But won't this remove all the links? Shouldn't we instead ensure that it starts with http/ftp(s)?

I have removed the re expression which rendered the www.foo.com and all the foo.com, foo.net and foo.org

The 2 expressions left will render all the http or https links

Could you rebase to the latest master?

Arf sorry, I thought you were offline and merged the PR you reviewed, so it still needs rebasing :(

I am on and offline, not easy with a 3 month old baby :baby_bottle: .

Rebase done :smile:

We'll need to speak about our experiences ;-)

Thanks for your understanding :)

Metadata