Fix missing Sphinx extensions in requirements.txt#450
Fix missing Sphinx extensions in requirements.txt#450mayank2295 wants to merge 2 commits intonumfocus:mainfrom
Conversation
❌ Deploy Preview for stupendous-kringle-a86e81 failed.
|
There was a problem hiding this comment.
Pull request overview
This PR addresses issue #449 by adding missing Sphinx extensions to the project's dependency files. The extensions were already being used in conf.py but were not explicitly listed in the requirements, which could cause build failures in clean environments.
Changes:
- Added six Sphinx extensions to requirements.txt and environment.yml
- Added version constraint (>=1.0.0) to jupyter-book dependency
Reviewed changes
Copilot reviewed 1 out of 2 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| requirements.txt | Added six Sphinx extensions (sphinx-togglebutton, sphinx-copybutton, sphinx-design, sphinx-book-theme, sphinx-jupyterbook-latex, sphinx-multitoc-numbering) and version constraint for jupyter-book |
| environment.yml | Added the same six Sphinx extensions and version constraint for jupyter-book to maintain consistency with requirements.txt |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
Hi! 👋 I would like to work on this issue. I've already analyzed the codebase and identified all the missing dependencies. Here's my plan:
I've already updated Could you please assign this issue to me? This is my first open source contribution and I'm excited to help! |
Description of Changes
This PR addresses #449 by adding all Sphinx extensions used in
conf.py to both requirements.txt and environment.yml
Changes:
Added sphinx-togglebutton
Added sphinx-copybutton
Added sphinx-design
Added sphinx-book-theme
Added sphinx-jupyterbook-latex
Added sphinx-multitoc-numbering
Testing:
Built the book locally with sphinx-build.