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

Remove ifconfig Sphinx extension from docs #3338

Merged
merged 2 commits into from
Jun 18, 2024
Merged

Conversation

justinmayer
Copy link
Member

@justinmayer justinmayer commented Jun 18, 2024

Originally added in #815, something about this configuration seems to be causing an obscure ReadTheDocs build error. Plus, it seems ReadTheDocs has since added this version-warning functionality as a beta feature. After merging this PR, if the latter built-in beta feature proves to deliver similar functionality, we could consider removing the last_stable variable and any other related changes that are no longer relevant.

This PR also adds matplotlib to our documentation package dependencies. Without it installed, Sphinx output says that social cards cannot be generated.

Without `matplotlib` installed, Sphinx output says that social cards
cannot be generated.
Originally added in #815, something about this seems to be causing an
obscure ReadTheDocs build error.
@justinmayer justinmayer merged commit 8d63d1c into master Jun 18, 2024
29 of 30 checks passed
@justinmayer justinmayer deleted the remove-sphinx-ifconfig branch June 18, 2024 07:27
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