Skip to content

Clean up Sphinx configuration and remove redundant code#9780

Open
dhruvatr wants to merge 1 commit into
borgbackup:masterfrom
dhruvatr:patch-1
Open

Clean up Sphinx configuration and remove redundant code#9780
dhruvatr wants to merge 1 commit into
borgbackup:masterfrom
dhruvatr:patch-1

Conversation

@dhruvatr

Copy link
Copy Markdown

Description

Checklist

  • PR is against master (or maintenance branch if only applicable there)
  • New code has tests and docs where appropriate
  • Tests pass (run tox or the relevant test subset)
  • Commit messages are clean and reference related issues

Summary

This PR performs minor cleanup in the Sphinx configuration file:

  • Removed redundant app.setup_extension("sphinxcontrib.jquery") call.
  • Removed the unused initial extensions = [] assignment.
  • Reformatted html_sidebars for better readability.
  • Reformatted extlinks dictionary for consistency.

Type of Change

  • Documentation
  • Code cleanup
  • No functional changes

Testing

  • No functional changes were made.
  • Configuration remains valid.

Comment thread docs/conf.py


def setup(app):
app.setup_extension("sphinxcontrib.jquery")

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

did you check whether we use jquery somewhere?

@ThomasWaldmann

Copy link
Copy Markdown
Member

@dhruvatr we use a specific black version (see requirements.d/*) to auto-format our stuff.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants