Remove or implement empty if len(resp) > 0: pass blocks in complete_game #38

Closed
opened 2026-02-20 06:52:37 +00:00 by cal · 2 comments
Owner

command_logic/logic_gameplay.py:4299-4309. After posting plays/decisions to API, stubs with pass suggest missing validation or follow-up logic.

`command_logic/logic_gameplay.py:4299-4309`. After posting plays/decisions to API, stubs with `pass` suggest missing validation or follow-up logic.
cal added the
tech-debt
label 2026-02-20 06:52:37 +00:00
cal added the
ai-working
label 2026-03-06 05:31:06 +00:00
cal removed the
ai-working
label 2026-03-06 05:34:58 +00:00
Author
Owner

Fixed in PR #71: #71

Removed both if len(resp) > 0: pass stubs and the now-unused resp = variable assignments. The db_post() calls now run without capturing the return value since it was never used.

Fixed in PR #71: https://git.manticorum.com/cal/paper-dynasty-discord/pulls/71 Removed both `if len(resp) > 0: pass` stubs and the now-unused `resp =` variable assignments. The `db_post()` calls now run without capturing the return value since it was never used.
cal added the
ai-pr-opened
label 2026-03-06 05:35:04 +00:00
Author
Owner

Addressed in PR #104. Removed two resp = await db_post(...) / if len(resp) > 0: pass patterns in command_logic/logic_gameplay.pyresp was captured but never used, and the if-pass block was a no-op.

Addressed in PR #104. Removed two `resp = await db_post(...) / if len(resp) > 0: pass` patterns in `command_logic/logic_gameplay.py` — `resp` was captured but never used, and the if-pass block was a no-op.
Sign in to join this conversation.
No Milestone
No project
No Assignees
1 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: cal/paper-dynasty-discord#38
No description provided.