chore(dev): move dev-dependencies inside dependency-groups (#542)

* chore(dev): move dev-dependencies inside dependency-groups

* fix(ci): ci was not broken

Author: jeertmans

.github/workflows/tests.yml

@@ -96,7 +96,7 @@ jobs:
       uses: ssciwr/setup-mesa-dist-win@v2
 
     - name: Run pytest
-      run: uv run --python ${{ matrix.pyversion }} --frozen --extra tests pytest
+      run: uv run --python ${{ matrix.pyversion }} --frozen --group tests --no-dev pytest
 
     - name: Upload to codecov.io
       uses: codecov/codecov-action@v5

.readthedocs.yaml

@@ -6,13 +6,13 @@ build:
   apt_packages:
   - libpango1.0-dev
   - ffmpeg
+  jobs:
+    post_create_environment:
+    - asdf plugin add uv
+    - asdf install uv latest
+    - asdf global uv latest
+    - UV_PROJECT_ENVIRONMENT=$READTHEDOCS_VIRTUALENV_PATH uv sync --group docs --no-dev --no-cache
 sphinx:
   builder: html
   configuration: docs/source/conf.py
   fail_on_warning: true
-python:
-  install:
-  - method: pip
-    path: .
-    extra_requirements:
-    - docs

CHANGELOG.md

@@ -10,6 +10,15 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 (unreleased)=
 ## [Unreleased](https://github.com/jeertmans/manim-slides/compare/v5.5.1...HEAD)
 
+(unreleased-chore)=
+### Chore
+
+- Moved `docs` and `tests` extras, as well as `dev-dependencies`,
+  inside groups in `dependency-groups`. This could break existing code
+  when using one of those extras, but as they were not part of the public API,
+  we do not consider this to be a **breaking change**.
+  [#542](https://github.com/jeertmans/manim-slides/pull/542)
+
 (v5.5.1)=
 ## [v5.5.1](https://github.com/jeertmans/manim-slides/compare/v5.5.0...v5.5.1)
 

docs/source/contributing/workflow.md

@@ -32,7 +32,7 @@ and development dependencies. If not already, please install this tool.
 With uv, installation becomes straightforward:
 
 ```bash
-uv sync --all-extras
+uv sync
 ```
 
 :::{note}

pyproject.toml

@@ -2,6 +2,39 @@
 build-backend = "hatchling.build"
 requires = ["hatchling", "hatch-fancy-pypi-readme"]
 
+[dependency-groups]
+dev = [
+  {include-group = "docs"},
+  {include-group = "tests"},
+  "bump-my-version>=0.20.3",
+  "pre-commit>=3.5.0",
+]
+docs = [
+  "manim-slides[magic,manim,pyqt6,sphinx-directive]",
+  "furo>=2023.5.20",
+  "ipykernel>=6.25.1",
+  "myst-parser>=2.0.0",
+  "nbsphinx>=0.9.2",
+  "pandoc>=2.3",
+  "pygments<2.19",  # See: https://github.com/ManimCommunity/manim/issues/4104
+  "sphinx>=7.0.1",
+  "sphinxcontrib-programoutput>=0.18",
+  "sphinx-design>=0.6.1",
+  "sphinx-click>=4.4.0",
+  "sphinx-copybutton>=0.5.1",
+  "sphinxext-opengraph>=0.7.5",
+]
+tests = [
+  "importlib-metadata>=8.6.1;python_version<'3.10'",
+  "manim-slides[full,manimgl,pyqt6,pyside6,sphinx-directive]",
+  "pytest>=7.4.0",
+  "pytest-cov>=4.1.0",
+  "pytest-env>=0.8.2",
+  "pytest-missing-modules>=0.1.0",
+  "pytest-qt>=4.2.0",
+  "setuptools>=73.0.1",
+]
+
 [project]
 authors = [{name = "Jérome Eertmans", email = "[email protected]"}]
 classifiers = [
@@ -42,21 +75,6 @@ name = "manim-slides"
 requires-python = ">=3.9"
 
 [project.optional-dependencies]
-docs = [
-  "manim-slides[magic,manim,pyqt6,sphinx-directive]",
-  "furo>=2023.5.20",
-  "ipykernel>=6.25.1",
-  "myst-parser>=2.0.0",
-  "nbsphinx>=0.9.2",
-  "pandoc>=2.3",
-  "pygments<2.19",  # See: https://github.com/ManimCommunity/manim/issues/4104
-  "sphinx>=7.0.1",
-  "sphinxcontrib-programoutput>=0.18",
-  "sphinx-design>=0.6.1",
-  "sphinx-click>=4.4.0",
-  "sphinx-copybutton>=0.5.1",
-  "sphinxext-opengraph>=0.7.5",
-]
 full = [
   "manim-slides[magic,manim,sphinx-directive]",
 ]
@@ -68,15 +86,6 @@ pyqt6-full = ["manim-slides[full,pyqt6]"]
 pyside6 = ["pyside6>=6.6.1,!=6.8.1.1"]
 pyside6-full = ["manim-slides[full,pyside6]"]
 sphinx-directive = ["docutils>=0.20.1", "manim-slides[manim]"]
-tests = [
-  "importlib-metadata>=8.6.1;python_version<'3.10'",
-  "manim-slides[full,manimgl,pyqt6,pyside6,sphinx-directive]",
-  "pytest>=7.4.0",
-  "pytest-cov>=4.1.0",
-  "pytest-env>=0.8.2",
-  "pytest-missing-modules>=0.1.0",
-  "pytest-qt>=4.2.0",
-]
 
 [project.scripts]
 manim-slides = "manim_slides.__main__:cli"
@@ -225,10 +234,3 @@ isort = {known-first-party = ["manim_slides", "tests"]}
 [tool.ruff.lint.per-file-ignores]
 "docs/source/reference/magic_example.ipynb" = ["F403", "F405"]
 "tests/test_slide.py" = ["N801"]
-
-[tool.uv]
-dev-dependencies = [
-  "bump-my-version>=0.20.3",
-  "pre-commit>=3.5.0",
-  "setuptools>=73.0.1",
-]