Skip to content

Commit 8fe9dc4

Browse files
committed
rename logger
1 parent bf7c3db commit 8fe9dc4

1 file changed

Lines changed: 13 additions & 5 deletions

File tree

src/somesy/core/writer.py

Lines changed: 13 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@
66

77
from somesy.core.models import Person, ProjectMetadata
88

9-
log = logging.getLogger("somesy")
9+
logger = logging.getLogger("somesy")
1010

1111

1212
class ProjectMetadataWriter(ABC):
@@ -49,7 +49,6 @@ def __init__(
4949
else:
5050
raise FileNotFoundError(f"The file {self.path} does not exist.")
5151

52-
@abstractmethod
5352
def _init_new_file(self) -> None:
5453
"""Create an new suitable target file.
5554
@@ -164,17 +163,17 @@ def _merge_person_metadata(
164163
old_fmt = self._from_person(person)
165164
new_fmt = self._from_person(modified_people[i])
166165
if old_fmt != new_fmt:
167-
log.debug(f"Updating person\n{old_fmt}\nto\n{new_fmt}")
166+
logger.debug(f"Updating person\n{old_fmt}\nto\n{new_fmt}")
168167

169168
if not person_existed:
170169
new_people.append(person_meta)
171170

172171
# show added and removed people in debug log
173172
removed_people = [old[i] for i in range(len(old)) if not still_exists[i]]
174173
for person in removed_people:
175-
log.debug(f"Removing person\n{self._from_person(person)}")
174+
logger.debug(f"Removing person\n{self._from_person(person)}")
176175
for person in new_people:
177-
log.debug(f"Adding person\n{self._from_person(person)}")
176+
logger.debug(f"Adding person\n{self._from_person(person)}")
178177

179178
# return updated list of (still existing) people,
180179
# and all new people coming after them.
@@ -184,6 +183,15 @@ def _merge_person_metadata(
184183
return existing_modified + new_people
185184

186185
def _sync_person_list(self, old: List[Any], new: List[Person]) -> List[Any]:
186+
"""Sync a list of persons with new metadata.
187+
188+
Args:
189+
old (List[Any]): list of persons in format-specific representation
190+
new (List[Person]): list of persons in somesy representation
191+
192+
Returns:
193+
List[Any]: updated list of persons in format-specific representation
194+
"""
187195
old_people: List[Person] = self._parse_people(old)
188196
return self._merge_person_metadata(old_people, new)
189197

0 commit comments

Comments
 (0)