Skip to content
This repository has been archived by the owner on Aug 21, 2024. It is now read-only.

Commit

Permalink
fix: set navigation_with_keys value
Browse files Browse the repository at this point in the history
False was the default value for navigation_with_keys. However, in version 0.14.2 of pydata-sphinx-theme, this default was removed and a warning was added that would be emitted whenever navigation_with_keys was not set. Because of the "SPHINXOPTS = -W" configuration in tox.ini, all warnings are promoted to an error. Therefore, it's necesary to set this value. I have set it to the default value explicitly.

Please see the following GitHub comments for context.
* pydata/pydata-sphinx-theme#1539
* pydata/pydata-sphinx-theme#987 (comment)
  • Loading branch information
MichaelRoytman committed Oct 27, 2023
1 parent 0a5d5f5 commit ef96288
Showing 1 changed file with 7 additions and 0 deletions.
7 changes: 7 additions & 0 deletions docs/en_us/dashboard/source/conf.py
Original file line number Diff line number Diff line change
Expand Up @@ -21,6 +21,13 @@
"use_repository_button": True,
"use_issues_button": True,
"use_edit_page_button": True,
# False was the default value for navigation_with_keys. However, in version 0.14.2 of pydata-sphinx-theme, this default
# was removed and a warning was added that would be emitted whenever navigation_with_keys was not set. Because of the
# "SPHINXOPTS = -W" configuration in tox.ini, all warnings are promoted to an error. Therefore, it's necesary to set
# this value. I have set it to the default value explicitly. Please see the following GitHub comments for context.
# https://github.com/pydata/pydata-sphinx-theme/issues/1539
# https://github.com/pydata/pydata-sphinx-theme/issues/987#issuecomment-1277214209
"navigation_with_keys": False,
# Please don't change unless you know what you're doing.
"extra_footer": """
<a rel="license" href="https://creativecommons.org/licenses/by-sa/4.0/">
Expand Down

0 comments on commit ef96288

Please sign in to comment.