Skip to content

Conversation

Copy link

Copilot AI commented Nov 14, 2025

Addressing feedback on logging levels in main.py

  • Understand the issue: Default verbosity level (verbose=0) sets logger to "ERROR", hiding important INFO messages
  • Change the default logging level to INFO by adjusting the logger_level array order
  • Verify the fix works correctly
  • Test and validate the changes
  • Run code review and security checks

💡 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.

Copilot AI self-assigned this Nov 14, 2025
Copilot AI mentioned this pull request Nov 14, 2025
@christopolise christopolise marked this pull request as ready for review November 14, 2025 20:42
Changed the logger_level array to start with INFO instead of ERROR,
making INFO the default logging level when no verbose flags are used.
This ensures users see important operational messages (including the
ASCII art banner) without needing to pass -vvv.

The new verbosity levels are:
- Default (no flags): INFO
- -v: DEBUG
- -vv: TRACE

ERROR and WARNING messages will still show at all levels since they
are higher priority than INFO in loguru's logging hierarchy.

Co-authored-by: philipbl <1527647+philipbl@users.noreply.github.com>
@philipbl philipbl closed this Nov 14, 2025
Copilot AI requested a review from philipbl November 14, 2025 20:49
@philipbl philipbl deleted the copilot/sub-pr-89 branch November 14, 2025 21:04
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