fix: rename ambiguous variable l to ln (ruff E741)

Author: aclark4life

src/dbx_python_cli/commands/project.py

@@ -608,7 +608,7 @@ def _fix_broken_editable_installs(
         url = data.get("url", "")
         if not url.startswith("file://"):
             continue
-        source_path = Path(url[len("file://"):])
+        source_path = Path(url[len("file://") :])
         if source_path.exists() and (
             (source_path / "pyproject.toml").exists()
             or (source_path / "setup.py").exists()
@@ -664,13 +664,28 @@ def _ensure_package_installed(
     repo_name = import_name.replace("_", "-")
 
     if base_dir is not None:
-        for clone_path in [Path(base_dir) / repo_name, Path(base_dir) / "django" / repo_name]:
+        for clone_path in [
+            Path(base_dir) / repo_name,
+            Path(base_dir) / "django" / repo_name,
+        ]:
             if clone_path.exists() and (
-                (clone_path / "pyproject.toml").exists() or (clone_path / "setup.py").exists()
+                (clone_path / "pyproject.toml").exists()
+                or (clone_path / "setup.py").exists()
             ):
-                typer.echo(f"📦 Installing {repo_name} from local clone at {clone_path}...")
+                typer.echo(
+                    f"📦 Installing {repo_name} from local clone at {clone_path}..."
+                )
                 subprocess.run(
-                    ["uv", "pip", "install", "--reinstall", "--python", python_path, "-e", str(clone_path)],
+                    [
+                        "uv",
+                        "pip",
+                        "install",
+                        "--reinstall",
+                        "--python",
+                        python_path,
+                        "-e",
+                        str(clone_path),
+                    ],
                     capture_output=not verbose,
                     check=False,
                 )
@@ -985,7 +1000,9 @@ def run_project(
     _fix_broken_editable_installs(python_path, proj.project_path, verbose)
 
     # Ensure django_mongodb_extensions is available: prefer a local clone over PyPI.
-    _ensure_package_installed("django_mongodb_extensions", python_path, proj.base_dir, verbose)
+    _ensure_package_installed(
+        "django_mongodb_extensions", python_path, proj.base_dir, verbose
+    )
 
     # Check if frontend exists
     frontend_path = proj.project_path / "frontend"

src/dbx_python_cli/commands/spec.py

@@ -30,6 +30,7 @@
 # Helpers
 # ---------------------------------------------------------------------------
 
+
 def _find_specs_dir(config, base_dir) -> Path | None:
     """Find the specifications repo directory via the repo config."""
     repo = find_repo_by_name("specifications", base_dir, config)
@@ -100,13 +101,19 @@ def _apply_patches(driver_repo, verbose: bool = False) -> bool:
         return True
 
     cmd = [
-        "git", "apply", "-R", "--allow-empty", "--whitespace=fix",
+        "git",
+        "apply",
+        "-R",
+        "--allow-empty",
+        "--whitespace=fix",
         *[str(p) for p in patches],
     ]
     if verbose:
         typer.echo(f"[verbose] Running: {' '.join(cmd)}")
 
-    result = subprocess.run(cmd, cwd=str(driver_repo["path"]), check=False, capture_output=True, text=True)
+    result = subprocess.run(
+        cmd, cwd=str(driver_repo["path"]), check=False, capture_output=True, text=True
+    )
     if result.returncode != 0:
         typer.echo(f"❌ Failed to apply patches: {result.stderr.strip()}", err=True)
         return False
@@ -117,6 +124,7 @@ def _apply_patches(driver_repo, verbose: bool = False) -> bool:
 # dbx spec sync
 # ---------------------------------------------------------------------------
 
+
 @app.command("sync")
 def spec_sync(
     ctx: typer.Context,
@@ -186,13 +194,17 @@ def spec_sync(
     else:
         mdb_specs = _find_specs_dir(config, base_dir)
         if not mdb_specs:
-            typer.echo("❌ Error: Could not find the 'specifications' repository", err=True)
+            typer.echo(
+                "❌ Error: Could not find the 'specifications' repository", err=True
+            )
             typer.echo("\nClone it with: dbx clone specifications")
             typer.echo("Or specify the path with: --specs-dir <path>")
             raise typer.Exit(1)
 
     if not mdb_specs.exists():
-        typer.echo(f"❌ Error: Specifications directory not found: {mdb_specs}", err=True)
+        typer.echo(
+            f"❌ Error: Specifications directory not found: {mdb_specs}", err=True
+        )
         raise typer.Exit(1)
 
     script = driver_repo["path"] / ".evergreen" / "resync-specs.sh"
@@ -223,7 +235,9 @@ def spec_sync(
         typer.echo(f"   Working directory: {cwd}")
         patch_count = len(_list_patches(_get_patch_dir(driver_repo)))
         if patch_count:
-            typer.echo(f"\n📋 {patch_count} patch(es) would be applied with --apply-patches")
+            typer.echo(
+                f"\n📋 {patch_count} patch(es) would be applied with --apply-patches"
+            )
         return
 
     spec_label = ", ".join(specs) if specs else "all"
@@ -248,6 +262,7 @@ def spec_sync(
 # dbx spec list
 # ---------------------------------------------------------------------------
 
+
 @app.command("list")
 def spec_list(
     ctx: typer.Context,
@@ -273,13 +288,17 @@ def spec_list(
     else:
         mdb_specs = _find_specs_dir(config, base_dir)
         if not mdb_specs:
-            typer.echo("❌ Error: Could not find the 'specifications' repository", err=True)
+            typer.echo(
+                "❌ Error: Could not find the 'specifications' repository", err=True
+            )
             typer.echo("\nClone it with: dbx clone specifications")
             typer.echo("Or specify the path with: --specs-dir <path>")
             raise typer.Exit(1)
 
     if not mdb_specs.exists():
-        typer.echo(f"❌ Error: Specifications directory not found: {mdb_specs}", err=True)
+        typer.echo(
+            f"❌ Error: Specifications directory not found: {mdb_specs}", err=True
+        )
         raise typer.Exit(1)
 
     source_dir = mdb_specs / "source"
@@ -289,7 +308,9 @@ def spec_list(
         typer.echo(f"[verbose] Listing specs in: {search_dir}\n")
 
     spec_dirs = sorted(
-        d.name for d in search_dir.iterdir() if d.is_dir() and not d.name.startswith(".")
+        d.name
+        for d in search_dir.iterdir()
+        if d.is_dir() and not d.name.startswith(".")
     )
 
     if not spec_dirs:
@@ -307,6 +328,7 @@ def spec_list(
 # dbx spec patch list
 # ---------------------------------------------------------------------------
 
+
 @patch_app.command("list")
 def patch_list(
     ctx: typer.Context,
@@ -352,6 +374,7 @@ def patch_list(
 # dbx spec patch create
 # ---------------------------------------------------------------------------
 
+
 @patch_app.command("create")
 def patch_create(
     ctx: typer.Context,
@@ -425,7 +448,9 @@ def patch_create(
     diff = result.stdout
     if not diff.strip():
         typer.echo("❌ No changes to capture (git diff is empty)", err=True)
-        typer.echo("\nMake sure you have unstaged changes in the driver repo after syncing.")
+        typer.echo(
+            "\nMake sure you have unstaged changes in the driver repo after syncing."
+        )
         raise typer.Exit(1)
 
     if dry_run:
@@ -447,6 +472,7 @@ def patch_create(
 # dbx spec patch remove
 # ---------------------------------------------------------------------------
 
+
 @patch_app.command("remove")
 def patch_remove(
     ctx: typer.Context,
@@ -488,6 +514,7 @@ def patch_remove(
 # dbx spec patch apply
 # ---------------------------------------------------------------------------
 
+
 @patch_app.command("apply")
 def patch_apply(
     ctx: typer.Context,
@@ -527,7 +554,9 @@ def patch_apply(
         typer.echo(f"No patch files found in {patch_dir}")
         return
 
-    typer.echo(f"🩹 {'Would apply' if dry_run else 'Applying'} {len(patches)} patch(es) to {repo_name}:")
+    typer.echo(
+        f"🩹 {'Would apply' if dry_run else 'Applying'} {len(patches)} patch(es) to {repo_name}:"
+    )
     for p in patches:
         files = _parse_patch_files(p)
         typer.echo(f"  • {p.stem} ({len(files)} file(s))")

src/dbx_python_cli/templates/project_template/project_name/views.py

@@ -8,6 +8,7 @@ def get_context_data(self, **kwargs):
         context = super().get_context_data(**kwargs)
         try:
             from wagtail.models import Site
+
             site = Site.find_for_request(self.request)
             if site:
                 context["wagtail_site"] = site

tests/test_project_command.py

@@ -259,7 +259,9 @@ def test_enable_wagtail_activates_settings(tmp_path):
     assert "from .wagtail import *  # noqa" in content
     assert "INSTALLED_APPS += WAGTAIL_INSTALLED_APPS  # noqa: F405" in content
     assert "MIDDLEWARE += WAGTAIL_MIDDLEWARE  # noqa: F405" in content
-    assert "MIGRATION_MODULES.update(WAGTAIL_MIGRATION_MODULES)  # noqa: F405" in content
+    assert (
+        "MIGRATION_MODULES.update(WAGTAIL_MIGRATION_MODULES)  # noqa: F405" in content
+    )
 
 
 def test_enable_wagtail_adds_url_patterns(tmp_path):
@@ -290,7 +292,9 @@ def test_create_pyproject_toml_includes_wagtail_dep(tmp_path):
     _create_pyproject_toml(tmp_path, "myproject", wagtail=True)
 
     content = (tmp_path / "pyproject.toml").read_text()
-    deps_section = content[content.index("dependencies"):content.index("[project.optional-dependencies]")]
+    deps_section = content[
+        content.index("dependencies") : content.index("[project.optional-dependencies]")
+    ]
     assert '"wagtail"' in deps_section
 
 
@@ -301,7 +305,9 @@ def test_create_pyproject_toml_excludes_wagtail_dep_by_default(tmp_path):
     _create_pyproject_toml(tmp_path, "myproject")
 
     content = (tmp_path / "pyproject.toml").read_text()
-    deps_section = content[content.index("dependencies"):content.index("[project.optional-dependencies]")]
+    deps_section = content[
+        content.index("dependencies") : content.index("[project.optional-dependencies]")
+    ]
     assert '"wagtail"' not in deps_section
 
 
@@ -323,7 +329,9 @@ def test_fix_broken_editable_installs_reinstalls_missing_source(tmp_path):
     dist_info = site_packages / "wagtail-7.4a0.dist-info"
     dist_info.mkdir()
     (dist_info / "direct_url.json").write_text(
-        json.dumps({"dir_info": {"editable": True}, "url": "file:///nonexistent/wagtail"})
+        json.dumps(
+            {"dir_info": {"editable": True}, "url": "file:///nonexistent/wagtail"}
+        )
     )
     (dist_info / "top_level.txt").write_text("wagtail\n")
 
@@ -362,14 +370,18 @@ def test_fix_broken_editable_installs_skips_undeclared_packages(tmp_path):
     dist_info = site_packages / "old_project-1.0.dist-info"
     dist_info.mkdir()
     (dist_info / "direct_url.json").write_text(
-        json.dumps({"dir_info": {"editable": True}, "url": "file:///nonexistent/old-project"})
+        json.dumps(
+            {"dir_info": {"editable": True}, "url": "file:///nonexistent/old-project"}
+        )
     )
     (dist_info / "top_level.txt").write_text("old_project\n")
 
     python_path = "/fake/python"
 
     with patch("dbx_python_cli.commands.project.subprocess.run") as mock_run:
-        mock_run.return_value = MagicMock(returncode=0, stdout=str(site_packages), stderr="")
+        mock_run.return_value = MagicMock(
+            returncode=0, stdout=str(site_packages), stderr=""
+        )
         _fix_broken_editable_installs(python_path, project_path)
 
     # Only the site.getsitepackages() call — no import check, no reinstall
@@ -404,7 +416,9 @@ def test_fix_broken_editable_installs_skips_valid_source(tmp_path):
     python_path = "/fake/python"
 
     with patch("dbx_python_cli.commands.project.subprocess.run") as mock_run:
-        mock_run.return_value = MagicMock(returncode=0, stdout=str(site_packages), stderr="")
+        mock_run.return_value = MagicMock(
+            returncode=0, stdout=str(site_packages), stderr=""
+        )
         _fix_broken_editable_installs(python_path, project_path)
 
     # Only the site.getsitepackages() call — source is valid, no reinstall