Repository navigation
Manage dev and docs dependencies with uv dependency groups - #585
Conversation
Move dev tools and docs requirements from optional dependencies and docs/requirements.txt into [dependency-groups], pin Ruff exactly, build the docs with uv in CI and on Read the Docs, and let Dependabot bump exact pins and upper bounds.
Yes, let's remove! |
tompollard
left a comment
tompollard
left a comment
There was a problem hiding this comment.
Thanks, this looks good! Happy to remove the sphinx-rtd-theme and readthedocs-sphinx-search here or in a separate PR.
|
Done, I've removed them. |
|
Thanks @cbrnr! do you have permissions to merge PRs? If so, I'll leave you to merge when ready. If not, I'll add you as a maintainer so you can do this in future. |
|
I do and I'll merge once all tests ideally come back green (or fail due to PhysioNet access)! |
|
Alright, first job already failed as expected, so I'll just go ahead and merge. I'll submit the next PR soon (conversion to plain pytest). |
It failed the instant I added the 🚀 to your comment, so think it might have been my fault. |
[project.optional-dependencies]to[dependency-groups], so they are no longer published as awfdb[dev]extradocsgroup, removedocs/requirements.txt, and build the docs with uv in CI and on Read the Docs== 0.17.3), because new releases can change formatting and lint results (thanks @tompollard for the suggestion!)uventry to Dependabot (monthly, 7-day cooldown,increase-if-necessary): exact pins and upper bounds get bumped, lower bounds are left alone. Group the GitHub Actions updates into one PR.I have one question:
sphinx-rtd-themeandreadthedocs-sphinx-searchare not used bydocs/conf.py(which uses theclassictheme). Should we remove them?