Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Logging #19

Open
Sfayson1 opened this issue Nov 19, 2024 · 0 comments · May be fixed by #22
Open

Logging #19

Sfayson1 opened this issue Nov 19, 2024 · 0 comments · May be fixed by #22
Assignees

Comments

@Sfayson1
Copy link

Sfayson1 commented Nov 19, 2024

Centralize and organize the logging configuration into a dedicated utility to improve maintainability, reuse, and separation of concerns.

Current State:

  • Configurations include:
    • File and console logging.
    • Log formatting with timestamp, log level, and message.

Required Changes:

  1. Create a new directory structure
  2. Extract existing logging setup:
  • Move the current logging.basicConfig() setup from app.py into logging_util.py.
  • Organize logging into reusable functions
  1. Update app.py

Technical Notes
Ensure the logger works across all modules, not just app.py.
Validate that both console and file logging function correctly.
Maintain compatibility with existing log messages.

Acceptance Criteria
All logging configuration moved to utils/logging_util.py.
Log formatting consistent across the application.
app.py imports and uses setup_logging.
No loss of existing logging functionality.

@Sfayson1 Sfayson1 moved this from Todo to In Progress in VetsAI Nov 21, 2024
@Sfayson1 Sfayson1 linked a pull request Nov 21, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: In Progress
Development

Successfully merging a pull request may close this issue.

2 participants