Skip to content
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

Implement a fallback html_theme #916

Merged
merged 3 commits into from
Oct 20, 2024
Merged

Conversation

alcarney
Copy link
Member

@alcarney alcarney commented Oct 20, 2024

image

Building on PRs like #913 and #915, this PR handles the case where the user's conf.py requests a html theme that's not available. In such a situation the sphinx agent will

  • Suppress the associated error
  • Add a config override to select Sphinx's built-in alabaster theme
  • Re-run the Sphinx initialization process.

This allows us to parse the file once in a central location
The sphinx agent can now handle the case where the requested
`html_theme` is not available in the environment by suppressing the
raised error, overriding the value of `html_theme` and attempting to
run `Sphinx.__init__` again.
@alcarney alcarney merged commit 591748e into swyddfa:develop Oct 20, 2024
12 of 13 checks passed
@alcarney alcarney deleted the missing-theme branch October 20, 2024 22:21
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.

1 participant