Skip to content

Commit 32e0520

Browse files
committed
ruff: Fix B005 Using .strip() with multi-character strings is misleading.
Signed-off-by: Anders Kaseorg <[email protected]>
1 parent 347490c commit 32e0520

File tree

1 file changed

+6
-2
lines changed

1 file changed

+6
-2
lines changed

zulip_bots/zulip_bots/game_handler.py

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -693,7 +693,11 @@ def verify_users(self, users: List[str], message: Dict[str, Any] = {}) -> List[s
693693
verified_users = []
694694
failed = False
695695
for u in users:
696-
user = u.strip().lstrip("@**").rstrip("**")
696+
user = u.strip()
697+
if user.startswith("@**"):
698+
user = user[len("@**") :]
699+
if user.endswith("**"):
700+
user = user[: -len("**")]
697701
if (
698702
user == self.get_bot_username() or user == self.email
699703
) and not self.supports_computer:
@@ -1007,7 +1011,7 @@ def end_game(self, winner: str) -> None:
10071011
if winner == "draw":
10081012
self.broadcast("It was a draw!")
10091013
elif winner.startswith("except:"):
1010-
loser = winner.lstrip("except:")
1014+
loser = winner[len("except:") :]
10111015
else:
10121016
winner_name = self.game_adapter.get_username_by_email(winner)
10131017
self.broadcast(f"**{winner_name}** won! :tada:")

0 commit comments

Comments
 (0)