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
Changed readme links to relative #551
Changed readme links to relative #551
Conversation
I just realised that this means links on GitHub might not work quite correctly. I'll leave the PR open for review for a time being though. |
Yeah, I'd like the GitHub page to keep working. If there's a way to make them both work, I'm all for it. Please file an issue? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sadly this will make the GitHub front page not work.
# Conflicts: # README.rst # docs/conf.py
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
How about a preprocessor step in conf.py (see new commits) ?
got into a bit of a fight with GitHub workflow. sorry, not something I could have fixed locally. turns out that 1) when creating files with python they need to be opened |
While looking at issue #541, I've noticed that locally built documentation has hyperlinks to the online one. While this works when documentation is online, it's not ideal for offline build. Moreover, when browsing legacy documentation online, the links take one to the
stable
documentation and, arguably, they shouldn't. This is a minor issue, so I thought I'll just submit a PR and see what happens.Sphinx's linkcheck is struggling a bit with the relative links (probably because of the order in which the pages are generated. see this issue), so I've selectivelly silenced them.
I've also taken the liberty to make the .rst table a bit less wide.