Using uv for docs & adding automated docs generating in PR - #1171
Using uv for docs & adding automated docs generating in PR#1171kennethshsu wants to merge 9 commits into
Conversation
Pyright Type CompletenessView the full Project (full
Other symbols referenced but not exported by
Symbols without documentation:
Patch (exported symbols added or changed by this PR): no exported symbol type-completeness changes detected. |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #1171 +/- ##
==========================================
- Coverage 91.27% 90.86% -0.42%
==========================================
Files 91 91
Lines 5422 5331 -91
Branches 691 677 -14
==========================================
- Hits 4949 4844 -105
- Misses 339 347 +8
- Partials 134 140 +6
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit 1cc844d. Configure here.
|
I see that you're getting those weird codecov failures too. See #1172 for an example where the only files I changed were I'm pretty sure those lines that are marked as misses are actually getting indirectly hit and that this is just a weird a pytest/codecov bug. I guess we'll need to add some tests to directly hit the lines, despite the minor redundancy involved. |
|
@kennethshsu we should make development versions on rtd hidden. |
|
Ya I'm working on it, if you look above in the actions there's actually a RTC build. The webhook isn't working 100% yet and I keep running into errors, trying to resolve. |
|
that's what i mean. that rtd build is not hidden and everyone visiting the docsite can see it. |
Ya this again, I kinda want to increase the threshold again... |
This should be fixed now (remember to clear your cache). |
|
@kennethshsu getting this when i try to visit the rtd version for this branch also, i'm not seeing a fix for #1151 based on the diff |
Don't go there, click on the link in the checks instead.
Ya, I put in the summary that there's no diff on here, it's basically just reading the docs and relink the GitHub webhook on RTC, which is just a fancy way of saying clicking a few buttons in the right order. I only put the ticket here so we can properly track and close out the ticket in the release notes. |
|
o wow, so we can actually get rid of experimental? |




Summary of Changes
Related GitHub Issue(s)
Closes #976
Closes #1151
Additional Context for Reviewers
The new doc site built this way doesn't seem to have the double search or pancake icon bugs
uv run pytest) and documentation changes (uv run jb build docs --builder=custom --custom-builder=doctest)Note
Low Risk
Tooling-only changes to docs build and RTC install; no application or library runtime code is modified.
Overview
Docs and CI workflow now use uv instead of pip for installing the project with the
docsextra and running Jupyter Book builds.On Read the Docs,
readthedocs.yamlreplacespip installwithuv syncand thedocsextra. Contributor docs indocs/README.mdshowuv sync --extra docsanduv run --directory docs jb build .from the repo root. The PR template checklist matches that path and adds a “do not edit below” section so the checklist stays consistent.Reviewed by Cursor Bugbot for commit a6fc39e. Bugbot is set up for automated code reviews on this repo. Configure here.