-
Notifications
You must be signed in to change notification settings - Fork 1
Fix Sphinx theme loading by removing invalid import #22
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
Conversation
The issue was that conf.py was trying to import sphinx_rtd_theme as a module from the local directory, which doesn't work with modern Sphinx versions. Changes: - Removed: sys.path.insert for sphinx_rtd_theme directory - Removed: import sphinx_rtd_theme (invalid with local theme) - Added: html_theme_path = ["."] to point to local theme directory Now Sphinx will find the local sphinx_rtd_theme/ directory without the problematic import statement that was causing 'no setup() function' error. Error was: 'extension sphinx_rtd_theme has no setup() function'
Test Results (py3.11 on ubuntu-24.04)10 tests 9 ✅ 14s ⏱️ Results for commit d265b6f. |
Test Results (py3.9 on ubuntu-24.04)10 tests 9 ✅ 12s ⏱️ Results for commit d265b6f. |
Test Results (py3.10 on ubuntu-22.04)10 tests 9 ✅ 13s ⏱️ Results for commit d265b6f. |
Test Results (py3.11 on ubuntu-22.04)10 tests 9 ✅ 14s ⏱️ Results for commit d265b6f. |
Test Results (py3.10 on ubuntu-24.04)10 tests 9 ✅ 14s ⏱️ Results for commit d265b6f. |
Test Results (py3.12 on ubuntu-24.04)10 tests 9 ✅ 15s ⏱️ Results for commit d265b6f. |
Test Results (py3.13 on ubuntu-24.04)10 tests 9 ✅ 15s ⏱️ Results for commit d265b6f. |
Test Results (py3.13 on ubuntu-22.04)10 tests 9 ✅ 16s ⏱️ Results for commit d265b6f. |
Test Results (py3.12 on ubuntu-22.04)10 tests 9 ✅ 16s ⏱️ Results for commit d265b6f. |
Test Results (py3.9 on ubuntu-22.04)10 tests 9 ✅ 13s ⏱️ Results for commit d265b6f. |
Test Results (py3.13 on macos-26)10 tests 9 ✅ 16s ⏱️ Results for commit d265b6f. |
Test Results (py3.14 on ubuntu-24.04)10 tests 9 ✅ 18s ⏱️ Results for commit d265b6f. |
Test Results (py3.14 on ubuntu-22.04)10 tests 9 ✅ 19s ⏱️ Results for commit d265b6f. |
Test Results (py3.12 on macos-26)10 tests 9 ✅ 18s ⏱️ Results for commit d265b6f. |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #22 +/- ##
=======================================
Coverage 30.59% 30.59%
=======================================
Files 4 4
Lines 268 268
=======================================
Hits 82 82
Misses 186 186
Flags with carried forward coverage won't be shown. Click here to find out more. 🚀 New features to boost your workflow:
|
Test Results (py3.9 on macos-26)10 tests 9 ✅ 19s ⏱️ Results for commit d265b6f. |
Test Results (py3.10 on macos-26)10 tests 9 ✅ 20s ⏱️ Results for commit d265b6f. |
Test Results (py3.12 on macos-15-intel)10 tests 9 ✅ 46s ⏱️ Results for commit d265b6f. |
Test Results (py3.10 on macos-15-intel)10 tests 9 ✅ 36s ⏱️ Results for commit d265b6f. |
Test Results (py3.11 on macos-15-intel)10 tests 9 ✅ 37s ⏱️ Results for commit d265b6f. |
Test Results (py3.13 on macos-15-intel)10 tests 9 ✅ 33s ⏱️ Results for commit d265b6f. |
Test Results (py3.14 on macos-15-intel)10 tests 9 ✅ 32s ⏱️ Results for commit d265b6f. |
Test Results (py3.9 on macos-15-intel)10 tests 9 ✅ 37s ⏱️ Results for commit d265b6f. |
Test Results (py3.14 on macos-26)10 tests 9 ✅ 18s ⏱️ Results for commit d265b6f. |
Test Results (py3.11 on macos-26)10 tests 9 ✅ 17s ⏱️ Results for commit d265b6f. |
The issue was that conf.py was trying to import sphinx_rtd_theme as a module from the local directory, which doesn't work with modern Sphinx versions.
Changes:
Now Sphinx will find the local sphinx_rtd_theme/ directory without the problematic import statement that was causing 'no setup() function' error.
Error was: 'extension sphinx_rtd_theme has no setup() function'