Skip to content

Conversation

@grillazz
Copy link
Owner

No description provided.

@grillazz grillazz requested a review from Copilot November 14, 2025 09:31
@grillazz grillazz self-assigned this Nov 14, 2025
Copilot finished reviewing on behalf of grillazz November 14, 2025 09:33
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR updates the rotoger logging library from version 0.1.1 to 0.2.1, adapting to its simplified API that removes the need for class instantiation.

  • Updated dependency version in both pyproject.toml and uv.lock
  • Migrated from Rotoger().get_logger() to direct get_logger() function calls across all files
  • Replaced custom AppLogger with rotoger's get_logger() in scheduler.py

Reviewed Changes

Copilot reviewed 12 out of 13 changed files in this pull request and generated no comments.

Show a summary per file
File Description
pyproject.toml Updated rotoger dependency from 0.1.1 to 0.2.1
uv.lock Updated lock file with new version hashes and metadata
app/services/smtp.py Updated import and logger initialization to use new get_logger() API
app/services/scheduler.py Replaced AppLogger with rotoger's get_logger()
app/services/auth.py Updated import and logger initialization to use new get_logger() API
app/models/base.py Updated import and logger initialization to use new get_logger() API
app/main.py Updated import and logger initialization to use new get_logger() API
app/exception_handlers/base.py Updated import and logger initialization to use new get_logger() API
app/database.py Updated import and logger initialization to use new get_logger() API
app/api/user.py Updated import and logger initialization to use new get_logger() API
app/api/stuff.py Updated import and logger initialization to use new get_logger() API
app/api/ml.py Updated import and logger initialization to use new get_logger() API
app/api/health.py Updated import and logger initialization to use new get_logger() API

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@grillazz grillazz merged commit e8f9a30 into main Nov 14, 2025
2 checks passed
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