Skip to content

Conversation

grillazz
Copy link
Owner

No description provided.

@grillazz grillazz requested a review from Copilot October 14, 2025 09:31
Copy link
Contributor

@Copilot 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 codebase to use the new Rotoger class instead of the deprecated AppStructLogger from the rotoger library, along with bumping the minimum version requirement to 0.1.1.

  • Replace all imports of AppStructLogger with Rotoger
  • Update logger instantiation from AppStructLogger().get_logger() to Rotoger().get_logger()
  • Bump rotoger dependency version to >=0.1.1

Reviewed Changes

Copilot reviewed 11 out of 12 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
pyproject.toml Updated rotoger dependency to require version >=0.1.1
app/services/smtp.py Replaced AppStructLogger import and instantiation with Rotoger
app/services/auth.py Replaced AppStructLogger import and instantiation with Rotoger
app/models/base.py Replaced AppStructLogger import and instantiation with Rotoger
app/main.py Replaced AppStructLogger import and instantiation with Rotoger
app/exception_handlers/base.py Replaced AppStructLogger import and instantiation with Rotoger
app/database.py Replaced AppStructLogger import and instantiation with Rotoger
app/api/user.py Replaced AppStructLogger import and instantiation with Rotoger
app/api/stuff.py Replaced AppStructLogger import and instantiation with Rotoger
app/api/ml.py Replaced AppStructLogger import and instantiation with Rotoger
app/api/health.py Replaced AppStructLogger import and instantiation with Rotoger and added blank lines

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Comment on lines 14 to +16



Copy link

Copilot AI Oct 14, 2025

Choose a reason for hiding this comment

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

Unnecessary blank lines added. These extra blank lines should be removed to maintain consistent code formatting.

Suggested change

Copilot uses AI. Check for mistakes.

@grillazz grillazz merged commit a5d63d4 into main Oct 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.

1 participant