-
Notifications
You must be signed in to change notification settings - Fork 35
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
Routingtable as label #19
Conversation
6a472aa
to
a76fced
Compare
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.
Thanks for your contribution! Could you add a changelog and .. versionadded::
directives to docs?
this aims at eliminating a warning generated by sphinx-build on swh-web's index, due to the presence of an explicit link to the http-routingtable.html file (which does not exists at that time, thus generated a warning). This should be killed once sphinx-contrib/httpdomain#19 is accepted and released.
@dahlia yup, but which version are we targeting? 1.7.1 or 1.8? |
@douardda As its target branch is master, the target version should be 1.8.0. |
a76fced
to
bdacb4a
Compare
@dahlia ok I think it should be good now. |
doc/conf.py
Outdated
# The full version, including alpha/beta/rc tags. | ||
release = '1.7.0' | ||
release = '1.9.0' |
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.
It seems typo. It should be 1.8.0.
using the 'routingtable' label, ie. writing :ref:`routingtable` in rst docs.
…label also add a new changelog entry.
bdacb4a
to
141723c
Compare
Indeed, sorry my keyboard tend to double strokes (used ^A in vim). |
Thank you! |
Make the generated routing table "referencable" using the 'routingtable' label.