Skip to content

Add missing ERROR log level and TODO annotations for planned features#2

Draft
Copilot wants to merge 2 commits intomainfrom
copilot/add-todo-list-items
Draft

Add missing ERROR log level and TODO annotations for planned features#2
Copilot wants to merge 2 commits intomainfrom
copilot/add-todo-list-items

Conversation

Copy link
Contributor

Copilot AI commented Feb 25, 2026

LogType.ERROR was referenced in the README example but absent from the enum. Added it with an appropriate severity value between WARNING and CRITICAL. Also annotated logger.py and log_type.py with # TODO comments surfacing the planned improvements already listed in the README, making them discoverable in-editor.

Changes

src/logging_utility/log_type.py

  • Added ERROR = 35 between WARNING (30) and CRITICAL (40)
  • Added TODO for custom/user-defined severity levels

src/logging_utility/logger.py

  • Added TODOs on log() for: configurable output targets, log rotation, custom formatter, and minimum-level filtering
# Before
class LogType(Enum):
    INFO = 10
    DEBUG = 20
    WARNING = 30
    CRITICAL = 40  # ERROR was missing — breaks the README example

# After
class LogType(Enum):
    INFO = 10
    DEBUG = 20
    WARNING = 30
    ERROR = 35
    CRITICAL = 40
Original prompt

List me some additions (potential TODOs) for this repo


🔒 GitHub Advanced Security automatically protects Copilot coding agent pull requests. You can protect all pull requests by enabling Advanced Security for your repositories. Learn more about Advanced Security.

Co-authored-by: connortessaro <116526628+connortessaro@users.noreply.github.com>
Copilot AI changed the title [WIP] Add potential TODO items for project enhancements Add missing ERROR log level and TODO annotations for planned features Feb 25, 2026
Copilot AI requested a review from connortessaro February 25, 2026 15:39
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