search

OpenTrading / OTMql4Zmq

Open Trading Metatrader 4 ZeroMQ Bridge
MIT License
99 stars 46 forks source link

creole markup rendering on github is broken, leaving 404 urls #8

Open MetaMemoryT opened 9 years ago

MetaMemoryT commented 9 years ago

Some links like the one in Readme.creole:

Metatrader using OTMql4Py (https://github.com/OpenTrading/OTMql4Py).

Leave the closing ) as part of the url. Then the url

https://github.com/OpenTrading/OTMql4Py)

Is 404,

Could we switch to an alternate creole url syntax or leave a space after url end to mitigate the broken url issue?

femtotrader commented 9 years ago

Maybe you might write an issue here https://github.com/github/markup/issues

OpenTrading commented 9 years ago

Github's creole is really broken: the rendering of creole in the wiki is different from the rendering of the top-level README.creole. I have some work arounds for the README that I've done another pass on, so it should be better now, but it really needs fixing at their end.

Thanks femtotrader for the link - I was looking for the repository for the creole rendering so I could take a look at the code. I'll look again for it or if you find it please add it here. The problem is, I can't see why the rendering is different (better) in the wiki than the top-level!

OpenTrading commented 9 years ago

A trailing space after the link works around it - they all should be OK now. The page won't change much now.

The README.creole file is a copy of FrontPage.creole in the wiki, with a sed script to try to work around the brokeness. In the wiki, things work OK, even though their implementation is lame.

Thank you for your help catching these details as it makes the project look bad with broken URLs.

OpenTrading commented 9 years ago

I think these are all fixed, but I'll leave the issue open to remind me to track down a simple case to submit a bug report to github.