Skip to content

Refactor/Update Formatters Config #8

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

Merged
merged 10 commits into from
Dec 13, 2024

Conversation

Msameim181
Copy link
Owner

@Msameim181 Msameim181 commented Dec 13, 2024

This pull request introduces several significant changes to the chromatrace project, including updates to the logging configuration, new integrations, and improvements to the documentation. The most important changes are summarized below:

Documentation Updates

  • Updated the README.md to mention support for process IDs and new integrations with Uvicorn, FastAPI, and SocketIO. [1] [2]

Logging Configuration Enhancements

  • Added support for process IDs in the logging format by introducing the show_process_id setting in LoggingSettings. [1] [2]
  • Simplified and consolidated formatter classes into a new BasicFormatter class, which now handles both colored and plain formatting. (F0f2b926L4R4, [1] [2])

Codebase Simplification

  • Refactored the _setup_syslog_handler method to use the new BasicFormatter class and removed redundant formatter classes. [1] [2]
  • Updated import statements and removed unused formatter classes from logging_settings.py. (F0f2b926L4R4, src/chromatrace/logging_settings.pyR2-R16)

Version Update

  • Incremented the package version from 0.2.9 to 0.2.11 to reflect the new features and improvements.

Example Updates

  • Modified example configurations to demonstrate the use of the new show_process_id setting. [1] [2]

@Msameim181 Msameim181 merged commit 911d2e7 into develop Dec 13, 2024
3 checks passed
@Msameim181 Msameim181 deleted the refactor/update_formatters_config branch December 13, 2024 18:14
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