Skip to content

Commit 44513c6

Browse files
committed
update comment about forwarded apierrors
1 parent 21fd270 commit 44513c6

File tree

1 file changed

+8
-0
lines changed

1 file changed

+8
-0
lines changed

planet/clients/subscriptions.py

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -281,6 +281,8 @@ async def suspend_subscription(self,
281281
resp = await self._session.request(method='POST',
282282
url=url,
283283
json=json_body)
284+
# Forward APIError. We don't strictly need this clause, but it
285+
# makes our intent clear.
284286
except APIError:
285287
raise
286288
except ClientError: # pragma: no cover
@@ -305,6 +307,8 @@ async def reactivate_subscription(self, subscription_id: str) -> None:
305307

306308
try:
307309
_ = await self._session.request(method='POST', url=url)
310+
# Forward APIError. We don't strictly need this clause, but it
311+
# makes our intent clear.
308312
except APIError:
309313
raise
310314
except ClientError: # pragma: no cover
@@ -359,6 +363,8 @@ async def bulk_suspend_subscriptions(
359363
url=url,
360364
json=json_body,
361365
params=params)
366+
# Forward APIError. We don't strictly need this clause, but it
367+
# makes our intent clear.
362368
except APIError:
363369
raise
364370
except ClientError: # pragma: no cover
@@ -408,6 +414,8 @@ async def bulk_reactivate_subscriptions(
408414
url=url,
409415
json=json_body,
410416
params=params)
417+
# Forward APIError. We don't strictly need this clause, but it
418+
# makes our intent clear.
411419
except APIError:
412420
raise
413421
except ClientError: # pragma: no cover

0 commit comments

Comments
 (0)