feat: runs ruff format and check on the template level and adds a few small logic tweaks

Author: 56kyle

.ruff.toml

@@ -28,7 +28,7 @@ exclude = [
     "node_modules",
     "site-packages",
     "venv",
-    "{{cookiecutter.project_name}}",
+    "[{][{]cookiecutter.project_name[}][}]",
 ]
 
 # Same as Black.

docs/conf.py

@@ -4,6 +4,7 @@
 
 from datetime import date
 
+
 # --- Project information -----------------------------------------------------
 
 # General info about the TEMPLATE documentation itself
@@ -59,8 +60,19 @@
 
 # MyST-Parser settings (allows using Markdown + Sphinx features)
 myst_enable_extensions = [
-    "amsmath", "colon_fence", "deflist", "dollarmath", "html_admonition", "html_image", "replacements",
-    "smartquotes", "strikethrough", "substitution", "tasklist", "attrs_inline", "attrs_block",
+    "amsmath",
+    "colon_fence",
+    "deflist",
+    "dollarmath",
+    "html_admonition",
+    "html_image",
+    "replacements",
+    "smartquotes",
+    "strikethrough",
+    "substitution",
+    "tasklist",
+    "attrs_inline",
+    "attrs_block",
 ]
 
 # Intersphinx mapping: Link to documentation of standard libraries or tools mentioned
@@ -85,7 +97,6 @@
     "sidebar_hide_name": True,  # Hide project name next to logo if logo contains it
     # "light_logo": "logo-light.png",
     # "dark_logo": "logo-dark.png",
-
     # Footer icons
     "footer_icons": [
         {

noxfile.py

@@ -1,7 +1,8 @@
 """Noxfile for the cookiecutter-robust-python template."""
-from pathlib import Path
+
 import shutil
 import tempfile
+from pathlib import Path
 
 import nox
 import platformdirs
@@ -41,11 +42,7 @@
     *("--demo-name", DEFAULT_DEMO_NAME),
 )
 
-TEMPLATE_PYTHON_LOCATIONS: tuple[Path, ...] = (
-    Path("noxfile.py"),
-    Path("scripts"),
-    Path("hooks")
-)
+TEMPLATE_PYTHON_LOCATIONS: tuple[Path, ...] = (Path("noxfile.py"), Path("scripts"), Path("hooks"))
 
 TEMPLATE_CONFIG_AND_DOCS: tuple[Path, ...] = (
     Path("pyproject.toml"),
@@ -59,7 +56,7 @@
     Path("LICENSE"),
     Path("CODE_OF_CONDUCT.md"),
     Path("CHANGELOG.md"),
-    Path("docs/")
+    Path("docs/"),
 )
 
 
@@ -84,6 +81,7 @@ def sync_uv_with_demo(session: Session) -> None:
         external=True,
     )
 
+
 @nox.session(name="uv-in-demo", python=DEFAULT_TEMPLATE_PYTHON_VERSION)
 def uv_in_demo(session: Session) -> None:
     session.install("cookiecutter", "platformdirs", "loguru", "typer")
@@ -136,12 +134,11 @@ def lint(session: Session):
     session.log("Installing linting dependencies for the template source...")
     session.install("-e", ".", "--group", "dev", "--group", "lint")
 
-    locations: list[str] = [str(loc) for loc in TEMPLATE_PYTHON_LOCATIONS]
     session.log(f"Running Ruff formatter check on template files with py{session.python}.")
-    session.run("ruff", "format", *locations, "--check")
+    session.run("ruff", "format")
 
     session.log(f"Running Ruff check on template files with py{session.python}.")
-    session.run("ruff", "check", *locations, "--verbose")
+    session.run("ruff", "check", "--verbose", "--fix")
 
 
 @nox.session(python=DEFAULT_TEMPLATE_PYTHON_VERSION)
@@ -188,11 +185,10 @@ def test(session: Session) -> None:
     # Need to find cookiecutter executable - it's in the template dev env installed by uv sync.
     cookiecutter_command: list[str] = ["uv", "run", "cookiecutter", "--no-input", "--output-dir", str(temp_dir), "."]
 
-
     session.run(*cookiecutter_command, external=True)
 
     # Navigate into the generated project directory
-    generated_project_dir = temp_dir / "test_project" # Use the slug defined in --extra-context
+    generated_project_dir = temp_dir / "test_project"  # Use the slug defined in --extra-context
     if not generated_project_dir.exists():
         session.error(f"Generated project directory not found: {generated_project_dir}")
 
@@ -236,12 +232,11 @@ def release_template(session: Session):
     cz_bump_args = ["uvx", "cz", "bump", "--changelog"]
 
     if increment:
-         cz_bump_args.append(f"--increment={increment}")
+        cz_bump_args.append(f"--increment={increment}")
 
     session.log("Running cz bump with args: %s", cz_bump_args)
     # success_codes=[0, 1] -> Allows code 1 which means 'nothing to bump' if no conventional commits since last release
     session.run(*cz_bump_args, success_codes=[0, 1], external=True)
 
     session.log("Template version bumped and tag created locally via Commitizen/uvx.")
     session.log("IMPORTANT: Push commits and tags to remote (`git push --follow-tags`) to trigger CD for the TEMPLATE.")
-

scripts/generate-demo-project.py

@@ -7,16 +7,12 @@
 from typing import Annotated
 
 import typer
-
 from cookiecutter.main import cookiecutter
 from typer.models import OptionInfo
 
+
 FolderOption: partial[OptionInfo] = partial(
-    typer.Option,
-    dir_okay=True,
-    file_okay=False,
-    resolve_path=True,
-    path_type=Path
+    typer.Option, dir_okay=True, file_okay=False, resolve_path=True, path_type=Path
 )
 
 
@@ -47,7 +43,7 @@ def _remove_any_existing_demo(parent_path: Path) -> None:
 def main(
     repo_folder: Annotated[Path, FolderOption("--repo-folder", "-r")],
     demos_cache_folder: Annotated[Path, FolderOption("--demos-cache-folder", "-c")],
-    demo_name: Annotated[str, typer.Option("--demo-name", "-d")]
+    demo_name: Annotated[str, typer.Option("--demo-name", "-d")],
 ) -> None:
     """Updates the poetry.lock file."""
     try:

scripts/prepare-github-release.py

@@ -146,7 +146,6 @@ def main(
     description from the draft release notes. The branch contains a single
     commit which updates the version number in the documentation.
     """
-
     if tag is None:
         today = datetime.date.today()
         tag = f"{today:%Y.%-m.%-d}"

scripts/sync-uv-with-demo.py

@@ -7,10 +7,9 @@
 from typing import Annotated
 
 import typer
-
+from loguru import logger
 from typer.models import OptionInfo
 
-from loguru import logger
 
 FolderOption: partial[OptionInfo] = partial(
     typer.Option, dir_okay=True, file_okay=False, resolve_path=True, path_type=Path
@@ -22,9 +21,7 @@ def sync_uv_with_demo(template_folder: Path, demos_cache_folder: Path, demo_name
     demo_uv_lock_path: Path = _find_uv_lock_path(demo_root)
     output_uv_lock_path: Path = _find_uv_lock_path(template_folder)
 
-    _copy_uv_lock_from_demo(
-        demo_uv_lock_path=demo_uv_lock_path, output_uv_lock_path=output_uv_lock_path
-    )
+    _copy_uv_lock_from_demo(demo_uv_lock_path=demo_uv_lock_path, output_uv_lock_path=output_uv_lock_path)
     logger.info(f"Copied demo from {demo_uv_lock_path=} to {output_uv_lock_path=}.")
 
 
@@ -49,13 +46,11 @@ def _find_uv_lock_path(search_root: Path) -> Path:
 def main(
     template_folder: Annotated[Path, FolderOption("--template-folder", "-t", exists=True)],
     demos_cache_folder: Annotated[Path, FolderOption("--demos-cache-folder", "-c", exists=True)],
-    demo_name: Annotated[str, typer.Option("--demo-name", "-d")]
+    demo_name: Annotated[str, typer.Option("--demo-name", "-d")],
 ) -> None:
     """Updates the uv.lock file."""
     try:
-        sync_uv_with_demo(
-            template_folder=template_folder, demos_cache_folder=demos_cache_folder, demo_name=demo_name
-        )
+        sync_uv_with_demo(template_folder=template_folder, demos_cache_folder=demos_cache_folder, demo_name=demo_name)
     except Exception as error:
         typer.secho(f"error: {error}", fg="red")
         sys.exit(1)

tests/conftest.py

@@ -2,7 +2,6 @@
 
 import os
 import subprocess
-
 from pathlib import Path
 from typing import Generator
 
@@ -25,10 +24,7 @@ def robust_python_demo_path(tmp_path_factory: TempPathFactory) -> Path:
         no_input=True,
         overwrite_if_exists=True,
         output_dir=demos_path,
-        extra_context={
-            "project_name": "robust-python-demo",
-            "add_rust_extension": False
-        }
+        extra_context={"project_name": "robust-python-demo", "add_rust_extension": False},
     )
     path: Path = demos_path / "robust-python-demo"
     subprocess.run(["uv", "lock"], cwd=path)
@@ -44,10 +40,7 @@ def robust_maturin_demo_path(tmp_path_factory: TempPathFactory) -> Path:
         no_input=True,
         overwrite_if_exists=True,
         output_dir=demos_path,
-        extra_context={
-            "project_name": "robust-maturin-demo",
-            "add_rust_extension": True
-        }
+        extra_context={"project_name": "robust-maturin-demo", "add_rust_extension": True},
     )
     path: Path = demos_path / "robust-maturin-demo"
     subprocess.run(["uv", "sync"], cwd=path)

tests/constants.py

@@ -1,8 +1,10 @@
 """Module containing constants used throughout all tests."""
+
 import json
 from pathlib import Path
 from typing import Any
 
+
 REPO_FOLDER: Path = Path(__file__).parent.parent
 COOKIECUTTER_FOLDER: Path = REPO_FOLDER / "{{cookiecutter.project_name}}"
 HOOKS_FOLDER: Path = REPO_FOLDER / "hooks"
@@ -36,12 +38,7 @@
     "tox",
     *CHECK_NOX_SESSIONS,
     *FULL_CHECK_NOX_SESSIONS,
-    "coverage"
+    "coverage",
 ]
 
-RUST_NOX_SESSIONS: list[str] = [
-    "format-rust",
-    "lint-rust",
-    "tests-rust",
-    "publish-rust"
-]
+RUST_NOX_SESSIONS: list[str] = ["format-rust", "lint-rust", "tests-rust", "publish-rust"]

tests/integration_tests/__init__.py

@@ -1,2 +1 @@
 """Integration tests for cookiecutter-robust-python."""
-

tests/integration_tests/test_robust_python_demo.py

@@ -1,4 +1,5 @@
 """Tests project generation and template functionality using a Python build backend."""
+
 import subprocess
 from pathlib import Path
 
@@ -24,6 +25,3 @@ def test_demo_project_noxfile(robust_python_demo_path: Path, session: str) -> No
     print(result.stdout)
     print(result.stderr)
     result.check_returncode()
-
-
-