Skip to content

Commit

Permalink
Change cancel_schedule return noop behavior to align returning None e…
Browse files Browse the repository at this point in the history
…xplicitly (lint fix)
  • Loading branch information
KieranBrantnerMagee committed Nov 13, 2020
1 parent 91cd7ee commit 91954ec
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -299,7 +299,7 @@ def cancel_scheduled_messages(self, sequence_numbers, **kwargs):
else:
numbers = [types.AMQPLong(s) for s in sequence_numbers]
if len(numbers) == 0:
return # no-op on empty list.
return None # no-op on empty list.
request_body = {MGMT_REQUEST_SEQUENCE_NUMBERS: types.AMQPArray(numbers)}
return self._mgmt_request_response_with_retry(
REQUEST_RESPONSE_CANCEL_SCHEDULED_MESSAGE_OPERATION,
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -243,7 +243,7 @@ async def cancel_scheduled_messages(self, sequence_numbers: Union[int, List[int]
else:
numbers = [types.AMQPLong(s) for s in sequence_numbers]
if len(numbers) == 0:
return # no-op on empty list.
return None # no-op on empty list.
request_body = {MGMT_REQUEST_SEQUENCE_NUMBERS: types.AMQPArray(numbers)}
return await self._mgmt_request_response_with_retry(
REQUEST_RESPONSE_CANCEL_SCHEDULED_MESSAGE_OPERATION,
Expand Down

0 comments on commit 91954ec

Please sign in to comment.