-
-
Notifications
You must be signed in to change notification settings - Fork 74
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
Docs cleanup #142
Docs cleanup #142
Conversation
- No _static path exists - White space has meaning: add line break between `code-block` and code - `yml` is not a valid lexer; `yaml` is valid
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.
LGTM
docs/conf.py
Outdated
version = '1.2.0' | ||
# The full version, including alpha/beta/rc tags. | ||
release = '1.2.0' | ||
with open('../version.txt', 'r') as reader: |
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.
You may want to make that path relative to the conf.py
file using os.path.relpath and file.
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.
Do you mean this?
with open(os.path.relpath('../version.txt'), 'r') as reader:
I don't know why using os.path.relpath
would be better, but I would appreciate it if you can enlighten me.
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.
More like with open(os.path.join(os.path.dirname(__file__), os.path.pardir, "version.txt"))
as this will work correctly in all platforms irrespective of which current directory the program is invoked from.
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.
Done in 83758d3.
# Conflicts: # HISTORY.txt
This PR contains trivial updates after I ran
make linkcheck
andmake html
. There are no substantial content revisions.