Skip to content

Commit

Permalink
Apply suggestions from code review
Browse files Browse the repository at this point in the history
  • Loading branch information
CommanderStorm authored May 14, 2024
1 parent 76fb77a commit 589419a
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions data/processors/export.py
Original file line number Diff line number Diff line change
Expand Up @@ -136,7 +136,7 @@ def _make_sure_is_safe(obj: object):
for name in removed_names:
if name in content and allowed_variation not in content:
raise RuntimeError(
f"{name} was purposely renamed due to NS context. " "Please make sure it is not included"
f"{name} was purposely renamed due to NS context. Please make sure it is not included"
)
elif isinstance(obj, dict):
for key, val in obj.items():
Expand All @@ -150,7 +150,7 @@ def _make_sure_is_safe(obj: object):
elif isinstance(obj, bool) or isinstance(obj, int) or isinstance(obj, float) or obj is None:
pass
else:
raise ValueError(f"unhandled type: {type(obj)!s}")
raise ValueError(f"unhandled type: {type(obj)}")


def export_for_api(data: dict, path: str) -> None:
Expand Down

0 comments on commit 589419a

Please sign in to comment.