Skip to content

Commit 5535321

Browse files
committed
Fix typing issues
1 parent 81ee308 commit 5535321

File tree

2 files changed

+7
-4
lines changed

2 files changed

+7
-4
lines changed

src/prompt_siren/build_images.py

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,7 @@
2121
import tempfile
2222
from dataclasses import dataclass
2323
from pathlib import Path
24-
from typing import Literal
24+
from typing import Literal, TypeVar
2525

2626
import click
2727
from pydantic import BaseModel
@@ -440,7 +440,10 @@ async def build_all_specs(self, specs: list[ImageBuildSpec]) -> list[BuildError]
440440
return errors
441441

442442

443-
def _maybe_override_cache_dir(config: BaseModel, cache_dir: str | None) -> BaseModel:
443+
T = TypeVar("T", bound=BaseModel)
444+
445+
446+
def _maybe_override_cache_dir(config: T, cache_dir: str | None) -> T:
444447
"""Override cache_dir on configs that define it."""
445448
if cache_dir is None:
446449
return config

tests/test_build_images.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -6,11 +6,11 @@
66
from unittest.mock import AsyncMock
77

88
import pytest
9-
from pydantic import BaseModel
10-
from prompt_siren.build_images import ImageBuilder, _maybe_override_cache_dir
9+
from prompt_siren.build_images import _maybe_override_cache_dir, ImageBuilder
1110
from prompt_siren.datasets.swebench_dataset.config import SwebenchDatasetConfig
1211
from prompt_siren.sandbox_managers.docker.plugins.errors import DockerClientError
1312
from prompt_siren.sandbox_managers.image_spec import BuildImageSpec
13+
from pydantic import BaseModel
1414

1515

1616
class MockDockerClient:

0 commit comments

Comments
 (0)