Skip to content

Document log message formatting improvement as backlog issue#6

Merged
IanDTM merged 2 commits into
ruff-testfrom
copilot/sub-pr-4-again
Dec 31, 2025
Merged

Document log message formatting improvement as backlog issue#6
IanDTM merged 2 commits into
ruff-testfrom
copilot/sub-pr-4-again

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Dec 31, 2025

Converted a code review comment about string concatenation formatting into a backlog issue rather than addressing it immediately in the current PR.

Changes

  • Created .github/BACKLOG_ISSUE_LOG_MESSAGE_FORMATTING.md documenting the formatting improvement for whack-a-mole/backend/app.py:518-519

Context

The current code uses continuation with leading space:

print(
    f"[Thread-{thread_name}] Whack success!"
    f" Hole {hole}, Time: {result['reaction_time']:.3f}s"
)

Suggested improvement for clarity (trailing space instead):

print(
    f"[Thread-{thread_name}] Whack success! "
    f"Hole {hole}, Time: {result['reaction_time']:.3f}s"
)

Both produce identical output; this is purely a readability concern suitable for future cleanup.


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Co-authored-by: IanDTM <8480521+IanDTM@users.noreply.github.com>
Copilot AI changed the title [WIP] Update to address feedback on Ruff test Document log message formatting improvement as backlog issue Dec 31, 2025
Copilot AI requested a review from IanDTM December 31, 2025 18:46
@IanDTM IanDTM marked this pull request as ready for review December 31, 2025 18:56
@IanDTM IanDTM merged commit 6d99bcb into ruff-test Dec 31, 2025
@IanDTM IanDTM deleted the copilot/sub-pr-4-again branch December 31, 2025 18:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants