Skip to content

Commit c5ee995

Browse files
chore(internal): bump ruff version (#203)
1 parent 5088404 commit c5ee995

13 files changed

+66
-122
lines changed

pyproject.toml

+7-5
Original file line numberDiff line numberDiff line change
@@ -77,8 +77,8 @@ format = { chain = [
7777
"check:ruff",
7878
"typecheck",
7979
]}
80-
"check:ruff" = "ruff ."
81-
"fix:ruff" = "ruff --fix ."
80+
"check:ruff" = "ruff check ."
81+
"fix:ruff" = "ruff check --fix ."
8282

8383
typecheck = { chain = [
8484
"typecheck:pyright",
@@ -162,6 +162,11 @@ reportPrivateUsage = false
162162
line-length = 120
163163
output-format = "grouped"
164164
target-version = "py37"
165+
166+
[tool.ruff.format]
167+
docstring-code-format = true
168+
169+
[tool.ruff.lint]
165170
select = [
166171
# isort
167172
"I",
@@ -192,9 +197,6 @@ unfixable = [
192197
]
193198
ignore-init-module-imports = true
194199

195-
[tool.ruff.format]
196-
docstring-code-format = true
197-
198200
[tool.ruff.lint.flake8-tidy-imports.banned-api]
199201
"functools.lru_cache".msg = "This function does not retain type information for the wrapped function's arguments; The `lru_cache` function from `_utils` should be used instead"
200202

requirements-dev.lock

+1-1
Original file line numberDiff line numberDiff line change
@@ -80,7 +80,7 @@ pytz==2023.3.post1
8080
# via dirty-equals
8181
respx==0.20.2
8282
rich==13.7.1
83-
ruff==0.1.9
83+
ruff==0.5.6
8484
setuptools==68.2.2
8585
# via nodeenv
8686
six==1.16.0

src/prompt_foundry_python_sdk/_base_client.py

+21-42
Original file line numberDiff line numberDiff line change
@@ -124,16 +124,14 @@ def __init__(
124124
self,
125125
*,
126126
url: URL,
127-
) -> None:
128-
...
127+
) -> None: ...
129128

130129
@overload
131130
def __init__(
132131
self,
133132
*,
134133
params: Query,
135-
) -> None:
136-
...
134+
) -> None: ...
137135

138136
def __init__(
139137
self,
@@ -166,8 +164,7 @@ def has_next_page(self) -> bool:
166164
return False
167165
return self.next_page_info() is not None
168166

169-
def next_page_info(self) -> Optional[PageInfo]:
170-
...
167+
def next_page_info(self) -> Optional[PageInfo]: ...
171168

172169
def _get_page_items(self) -> Iterable[_T]: # type: ignore[empty-body]
173170
...
@@ -903,8 +900,7 @@ def request(
903900
*,
904901
stream: Literal[True],
905902
stream_cls: Type[_StreamT],
906-
) -> _StreamT:
907-
...
903+
) -> _StreamT: ...
908904

909905
@overload
910906
def request(
@@ -914,8 +910,7 @@ def request(
914910
remaining_retries: Optional[int] = None,
915911
*,
916912
stream: Literal[False] = False,
917-
) -> ResponseT:
918-
...
913+
) -> ResponseT: ...
919914

920915
@overload
921916
def request(
@@ -926,8 +921,7 @@ def request(
926921
*,
927922
stream: bool = False,
928923
stream_cls: Type[_StreamT] | None = None,
929-
) -> ResponseT | _StreamT:
930-
...
924+
) -> ResponseT | _StreamT: ...
931925

932926
def request(
933927
self,
@@ -1156,8 +1150,7 @@ def get(
11561150
cast_to: Type[ResponseT],
11571151
options: RequestOptions = {},
11581152
stream: Literal[False] = False,
1159-
) -> ResponseT:
1160-
...
1153+
) -> ResponseT: ...
11611154

11621155
@overload
11631156
def get(
@@ -1168,8 +1161,7 @@ def get(
11681161
options: RequestOptions = {},
11691162
stream: Literal[True],
11701163
stream_cls: type[_StreamT],
1171-
) -> _StreamT:
1172-
...
1164+
) -> _StreamT: ...
11731165

11741166
@overload
11751167
def get(
@@ -1180,8 +1172,7 @@ def get(
11801172
options: RequestOptions = {},
11811173
stream: bool,
11821174
stream_cls: type[_StreamT] | None = None,
1183-
) -> ResponseT | _StreamT:
1184-
...
1175+
) -> ResponseT | _StreamT: ...
11851176

11861177
def get(
11871178
self,
@@ -1207,8 +1198,7 @@ def post(
12071198
options: RequestOptions = {},
12081199
files: RequestFiles | None = None,
12091200
stream: Literal[False] = False,
1210-
) -> ResponseT:
1211-
...
1201+
) -> ResponseT: ...
12121202

12131203
@overload
12141204
def post(
@@ -1221,8 +1211,7 @@ def post(
12211211
files: RequestFiles | None = None,
12221212
stream: Literal[True],
12231213
stream_cls: type[_StreamT],
1224-
) -> _StreamT:
1225-
...
1214+
) -> _StreamT: ...
12261215

12271216
@overload
12281217
def post(
@@ -1235,8 +1224,7 @@ def post(
12351224
files: RequestFiles | None = None,
12361225
stream: bool,
12371226
stream_cls: type[_StreamT] | None = None,
1238-
) -> ResponseT | _StreamT:
1239-
...
1227+
) -> ResponseT | _StreamT: ...
12401228

12411229
def post(
12421230
self,
@@ -1469,8 +1457,7 @@ async def request(
14691457
*,
14701458
stream: Literal[False] = False,
14711459
remaining_retries: Optional[int] = None,
1472-
) -> ResponseT:
1473-
...
1460+
) -> ResponseT: ...
14741461

14751462
@overload
14761463
async def request(
@@ -1481,8 +1468,7 @@ async def request(
14811468
stream: Literal[True],
14821469
stream_cls: type[_AsyncStreamT],
14831470
remaining_retries: Optional[int] = None,
1484-
) -> _AsyncStreamT:
1485-
...
1471+
) -> _AsyncStreamT: ...
14861472

14871473
@overload
14881474
async def request(
@@ -1493,8 +1479,7 @@ async def request(
14931479
stream: bool,
14941480
stream_cls: type[_AsyncStreamT] | None = None,
14951481
remaining_retries: Optional[int] = None,
1496-
) -> ResponseT | _AsyncStreamT:
1497-
...
1482+
) -> ResponseT | _AsyncStreamT: ...
14981483

14991484
async def request(
15001485
self,
@@ -1709,8 +1694,7 @@ async def get(
17091694
cast_to: Type[ResponseT],
17101695
options: RequestOptions = {},
17111696
stream: Literal[False] = False,
1712-
) -> ResponseT:
1713-
...
1697+
) -> ResponseT: ...
17141698

17151699
@overload
17161700
async def get(
@@ -1721,8 +1705,7 @@ async def get(
17211705
options: RequestOptions = {},
17221706
stream: Literal[True],
17231707
stream_cls: type[_AsyncStreamT],
1724-
) -> _AsyncStreamT:
1725-
...
1708+
) -> _AsyncStreamT: ...
17261709

17271710
@overload
17281711
async def get(
@@ -1733,8 +1716,7 @@ async def get(
17331716
options: RequestOptions = {},
17341717
stream: bool,
17351718
stream_cls: type[_AsyncStreamT] | None = None,
1736-
) -> ResponseT | _AsyncStreamT:
1737-
...
1719+
) -> ResponseT | _AsyncStreamT: ...
17381720

17391721
async def get(
17401722
self,
@@ -1758,8 +1740,7 @@ async def post(
17581740
files: RequestFiles | None = None,
17591741
options: RequestOptions = {},
17601742
stream: Literal[False] = False,
1761-
) -> ResponseT:
1762-
...
1743+
) -> ResponseT: ...
17631744

17641745
@overload
17651746
async def post(
@@ -1772,8 +1753,7 @@ async def post(
17721753
options: RequestOptions = {},
17731754
stream: Literal[True],
17741755
stream_cls: type[_AsyncStreamT],
1775-
) -> _AsyncStreamT:
1776-
...
1756+
) -> _AsyncStreamT: ...
17771757

17781758
@overload
17791759
async def post(
@@ -1786,8 +1766,7 @@ async def post(
17861766
options: RequestOptions = {},
17871767
stream: bool,
17881768
stream_cls: type[_AsyncStreamT] | None = None,
1789-
) -> ResponseT | _AsyncStreamT:
1790-
...
1769+
) -> ResponseT | _AsyncStreamT: ...
17911770

17921771
async def post(
17931772
self,

src/prompt_foundry_python_sdk/_compat.py

+8-16
Original file line numberDiff line numberDiff line change
@@ -159,22 +159,19 @@ def model_parse(model: type[_ModelT], data: Any) -> _ModelT:
159159
# generic models
160160
if TYPE_CHECKING:
161161

162-
class GenericModel(pydantic.BaseModel):
163-
...
162+
class GenericModel(pydantic.BaseModel): ...
164163

165164
else:
166165
if PYDANTIC_V2:
167166
# there no longer needs to be a distinction in v2 but
168167
# we still have to create our own subclass to avoid
169168
# inconsistent MRO ordering errors
170-
class GenericModel(pydantic.BaseModel):
171-
...
169+
class GenericModel(pydantic.BaseModel): ...
172170

173171
else:
174172
import pydantic.generics
175173

176-
class GenericModel(pydantic.generics.GenericModel, pydantic.BaseModel):
177-
...
174+
class GenericModel(pydantic.generics.GenericModel, pydantic.BaseModel): ...
178175

179176

180177
# cached properties
@@ -193,26 +190,21 @@ class typed_cached_property(Generic[_T]):
193190
func: Callable[[Any], _T]
194191
attrname: str | None
195192

196-
def __init__(self, func: Callable[[Any], _T]) -> None:
197-
...
193+
def __init__(self, func: Callable[[Any], _T]) -> None: ...
198194

199195
@overload
200-
def __get__(self, instance: None, owner: type[Any] | None = None) -> Self:
201-
...
196+
def __get__(self, instance: None, owner: type[Any] | None = None) -> Self: ...
202197

203198
@overload
204-
def __get__(self, instance: object, owner: type[Any] | None = None) -> _T:
205-
...
199+
def __get__(self, instance: object, owner: type[Any] | None = None) -> _T: ...
206200

207201
def __get__(self, instance: object, owner: type[Any] | None = None) -> _T | Self:
208202
raise NotImplementedError()
209203

210-
def __set_name__(self, owner: type[Any], name: str) -> None:
211-
...
204+
def __set_name__(self, owner: type[Any], name: str) -> None: ...
212205

213206
# __set__ is not defined at runtime, but @cached_property is designed to be settable
214-
def __set__(self, instance: object, value: _T) -> None:
215-
...
207+
def __set__(self, instance: object, value: _T) -> None: ...
216208
else:
217209
try:
218210
from functools import cached_property as cached_property

src/prompt_foundry_python_sdk/_files.py

+4-8
Original file line numberDiff line numberDiff line change
@@ -39,13 +39,11 @@ def assert_is_file_content(obj: object, *, key: str | None = None) -> None:
3939

4040

4141
@overload
42-
def to_httpx_files(files: None) -> None:
43-
...
42+
def to_httpx_files(files: None) -> None: ...
4443

4544

4645
@overload
47-
def to_httpx_files(files: RequestFiles) -> HttpxRequestFiles:
48-
...
46+
def to_httpx_files(files: RequestFiles) -> HttpxRequestFiles: ...
4947

5048

5149
def to_httpx_files(files: RequestFiles | None) -> HttpxRequestFiles | None:
@@ -83,13 +81,11 @@ def _read_file_content(file: FileContent) -> HttpxFileContent:
8381

8482

8583
@overload
86-
async def async_to_httpx_files(files: None) -> None:
87-
...
84+
async def async_to_httpx_files(files: None) -> None: ...
8885

8986

9087
@overload
91-
async def async_to_httpx_files(files: RequestFiles) -> HttpxRequestFiles:
92-
...
88+
async def async_to_httpx_files(files: RequestFiles) -> HttpxRequestFiles: ...
9389

9490

9591
async def async_to_httpx_files(files: RequestFiles | None) -> HttpxRequestFiles | None:

src/prompt_foundry_python_sdk/_response.py

+4-8
Original file line numberDiff line numberDiff line change
@@ -262,12 +262,10 @@ def _parse(self, *, to: type[_T] | None = None) -> R | _T:
262262

263263
class APIResponse(BaseAPIResponse[R]):
264264
@overload
265-
def parse(self, *, to: type[_T]) -> _T:
266-
...
265+
def parse(self, *, to: type[_T]) -> _T: ...
267266

268267
@overload
269-
def parse(self) -> R:
270-
...
268+
def parse(self) -> R: ...
271269

272270
def parse(self, *, to: type[_T] | None = None) -> R | _T:
273271
"""Returns the rich python representation of this response's data.
@@ -366,12 +364,10 @@ def iter_lines(self) -> Iterator[str]:
366364

367365
class AsyncAPIResponse(BaseAPIResponse[R]):
368366
@overload
369-
async def parse(self, *, to: type[_T]) -> _T:
370-
...
367+
async def parse(self, *, to: type[_T]) -> _T: ...
371368

372369
@overload
373-
async def parse(self) -> R:
374-
...
370+
async def parse(self) -> R: ...
375371

376372
async def parse(self, *, to: type[_T] | None = None) -> R | _T:
377373
"""Returns the rich python representation of this response's data.

src/prompt_foundry_python_sdk/_types.py

+3-6
Original file line numberDiff line numberDiff line change
@@ -111,8 +111,7 @@ class NotGiven:
111111
For example:
112112
113113
```py
114-
def get(timeout: Union[int, NotGiven, None] = NotGiven()) -> Response:
115-
...
114+
def get(timeout: Union[int, NotGiven, None] = NotGiven()) -> Response: ...
116115
117116
118117
get(timeout=1) # 1s timeout
@@ -162,16 +161,14 @@ def build(
162161
*,
163162
response: Response,
164163
data: object,
165-
) -> _T:
166-
...
164+
) -> _T: ...
167165

168166

169167
Headers = Mapping[str, Union[str, Omit]]
170168

171169

172170
class HeadersLikeProtocol(Protocol):
173-
def get(self, __key: str) -> str | None:
174-
...
171+
def get(self, __key: str) -> str | None: ...
175172

176173

177174
HeadersLike = Union[Headers, HeadersLikeProtocol]

0 commit comments

Comments
 (0)