Skip to content
This repository was archived by the owner on Nov 19, 2024. It is now read-only.

Commit ba1d384

Browse files
authored
cleanup legacy client (#99)
1 parent ee05ccb commit ba1d384

File tree

1 file changed

+12
-10
lines changed

1 file changed

+12
-10
lines changed

cads_api_client/legacy_api_client.py

Lines changed: 12 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -51,15 +51,6 @@ def __exit__(
5151
self.logger.removeHandler(handler)
5252

5353

54-
def _deprecated_warning(**kwargs: Any) -> None:
55-
if kwargs := {k: v for k, v in kwargs.items() if v is not None}:
56-
warnings.warn(
57-
f"The following parameters are deprecated: {kwargs}."
58-
" Set them to None to silence this warning.",
59-
UserWarning,
60-
)
61-
62-
6354
class LegacyApiClient(cdsapi.api.Client): # type: ignore[misc]
6455
def __init__(
6556
self,
@@ -83,7 +74,9 @@ def __init__(
8374
forget: None = None,
8475
session: requests.Session | None = None,
8576
) -> None:
86-
_deprecated_warning(full_stack=full_stack, metadata=metadata, forget=forget)
77+
self.issue_deprecated_kwargs_warning(
78+
full_stack=full_stack, metadata=metadata, forget=forget
79+
)
8780

8881
self.url, self.key, verify = cdsapi.api.get_url_key_verify(url, key, verify)
8982
self.verify = bool(verify)
@@ -130,6 +123,15 @@ def __init__(
130123
},
131124
)
132125

126+
@classmethod
127+
def issue_deprecated_kwargs_warning(self, **kwargs: Any) -> None:
128+
if kwargs := {k: v for k, v in kwargs.items() if v is not None}:
129+
warnings.warn(
130+
f"The following parameters are deprecated: {kwargs}."
131+
" Set them to None to silence this warning.",
132+
UserWarning,
133+
)
134+
133135
@classmethod
134136
def raise_not_implemented_error(self) -> None:
135137
raise NotImplementedError(

0 commit comments

Comments
 (0)