Skip to content

Commit 122a44c

Browse files
committed
refactor: improve type hints and formatting in keyring provider
1 parent 57e73af commit 122a44c

File tree

1 file changed

+7
-6
lines changed

1 file changed

+7
-6
lines changed

pydantic_settings/sources/providers/keyring.py

Lines changed: 7 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,5 @@
1-
from typing import TYPE_CHECKING, Mapping
1+
from collections.abc import Mapping
2+
from typing import TYPE_CHECKING
23

34
from pydantic_settings import BaseSettings, EnvSettingsSource
45

@@ -7,7 +8,7 @@
78
else:
89
keyring = None
910

10-
def import_keyring():
11+
def import_keyring() -> None:
1112
global keyring
1213
if keyring is not None:
1314
return
@@ -16,7 +17,7 @@ def import_keyring():
1617

1718
return
1819
except ImportError as e:
19-
raise ImportError("Keyring is not installed, run `pip install keyring`") from e
20+
raise ImportError('Keyring is not installed, run `pip install keyring`') from e
2021

2122

2223
class KeyringConfigSettingsSource(EnvSettingsSource):
@@ -40,7 +41,7 @@ def _load_env_vars(self) -> Mapping[str, str | None]:
4041
prefix = self.env_prefix
4142
if not self.case_sensitive:
4243
prefix = self.env_prefix.lower()
43-
env_vars = {}
44+
env_vars: dict[str, str | None] = {}
4445
for field in self.settings_cls.model_fields.keys():
4546
if not self.case_sensitive:
4647
field = field.lower()
@@ -54,9 +55,9 @@ def _load_env_vars(self) -> Mapping[str, str | None]:
5455
return env_vars
5556

5657
def __repr__(self) -> str:
57-
return f"{self.__class__.__name__}(keyring_service={self.keyring_service})"
58+
return f'{self.__class__.__name__}(keyring_service={self.keyring_service})'
5859

5960

6061
__all__ = [
61-
"KeyringConfigSettingsSource",
62+
'KeyringConfigSettingsSource',
6263
]

0 commit comments

Comments
 (0)