-
Notifications
You must be signed in to change notification settings - Fork 17
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
Speedup readthedocs build with uv #604
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #604 +/- ##
=======================================
Coverage 96.21% 96.21%
=======================================
Files 11 11
Lines 1188 1188
=======================================
Hits 1143 1143
Misses 45 45
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me, at least it is in sync with readthedocs/readthedocs.org#11322.
@unkcpz do you want to have a look?
Thanks @yakutovicha @unkcpz I've merged this, by you can still review this as part of #607 |
I configured the readthedocs build to use
uv
to install dependencies, according to readthedocs/readthedocs.org#11322This cuts the build time in half (by full minute!) so imo it's worth doing.