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

Update Sphinx theme #975

Merged
merged 3 commits into from
Nov 9, 2023

Conversation

arnaucasau
Copy link
Contributor

Summary

Updates to new Ecosystem theme.

For more information: Qiskit/qiskit_sphinx_theme#232

@zlatko-minev
Copy link
Collaborator

This compiles ok on your end?

@arnaucasau
Copy link
Contributor Author

arnaucasau commented Oct 20, 2023

Similar to #976. It has warning when running tox -e docs. The problem with this PR alone is the major increasing at the running time building the documentation. It takes locally 2 hours and a half. With the changes on #976 we can speed up the process to approximately 16 minutes.

@zlatko-minev
Copy link
Collaborator

That should be merged now

@arnaucasau arnaucasau marked this pull request as ready for review November 9, 2023 21:08
@arnaucasau
Copy link
Contributor Author

arnaucasau commented Nov 9, 2023

Great! Thank you @zlatko-minev!! Without that PR the new theme spends 3+ hours building the documentation, but now it's roughly 20 minutes 👍

@zlatko-minev zlatko-minev merged commit 0cf6a1e into qiskit-community:main Nov 9, 2023
8 checks passed
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