Skip to content

Commit bfa01e6

Browse files
caksoylarjoelspadin
authored andcommitted
fix: Resolve user home path at the time of setting
1 parent 1dc0273 commit bfa01e6

1 file changed

Lines changed: 6 additions & 1 deletion

File tree

zmk/config.py

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -24,6 +24,9 @@ class Settings(StrEnum):
2424
CORE_EXPLORER = "core.explorer" # Directory editor tool
2525

2626

27+
_PATH_SETTINGS = (Settings.USER_HOME,)
28+
29+
2730
class Config:
2831
"""Wrapper around ConfigParser to store CLI configuration"""
2932

@@ -58,6 +61,8 @@ def getboolean(self, name: str, **kwargs) -> bool:
5861
def set(self, name: str, value: str) -> None:
5962
"""Set a setting"""
6063
section, option = self._split_option(name)
64+
if name in _PATH_SETTINGS:
65+
value = str(Path(value).resolve())
6166
self._parser.set(section, option, value)
6267

6368
def remove(self, name: str) -> None:
@@ -95,7 +100,7 @@ def home_path(self) -> Path | None:
95100

96101
@home_path.setter
97102
def home_path(self, value: Path) -> None:
98-
self.set(Settings.USER_HOME, str(value.resolve()))
103+
self.set(Settings.USER_HOME, str(value))
99104

100105
def get_repo(self) -> Repo:
101106
"""

0 commit comments

Comments
 (0)