Skip to content

Commit

Permalink
Add type hints where missing in IMS client
Browse files Browse the repository at this point in the history
  • Loading branch information
mharding-hpe committed Oct 1, 2024
1 parent 9d74363 commit 3242926
Showing 1 changed file with 3 additions and 2 deletions.
5 changes: 3 additions & 2 deletions src/bos/operators/utils/clients/ims.py
Original file line number Diff line number Diff line change
Expand Up @@ -91,7 +91,7 @@ def get_image(image_id: str, session: RequestsSession|None=None) -> dict:
raise


def patch_image(image_id, data, session=None):
def patch_image(image_id: str, data: dict, session: RequestsSession|None=None) -> None:
if not data:
LOGGER.warning("patch_image called without data; returning without action.")
return
Expand All @@ -111,7 +111,8 @@ def patch_image(image_id, data, session=None):
raise


def tag_image(image_id: str, operation: str, key: str, value: str = None, session=None) -> None:
def tag_image(image_id: str, operation: str, key: str, value: str=None,
session: RequestsSession|None=None) -> None:
if operation not in IMS_TAG_OPERATIONS:
msg = f"{operation} not valid. Expecting one of {IMS_TAG_OPERATIONS}"
LOGGER.error(msg)
Expand Down

0 comments on commit 3242926

Please sign in to comment.