Skip to content

Dev#66

Merged
dcloud347 merged 15 commits intomainfrom
dev
May 22, 2025
Merged

Dev#66
dcloud347 merged 15 commits intomainfrom
dev

Conversation

@dcloud347
Copy link
Contributor

@dcloud347 dcloud347 commented May 13, 2025

This pull request introduces several improvements to the Prometheus project, focusing on enhancing documentation, centralizing configuration management, and improving code readability. The most significant changes include a complete overhaul of the README.md file, migration from settings.toml to .env for environment configuration, and the refactoring of various services to improve maintainability.

Documentation Updates:

  • Revamped README.md with sections for features, integrations, setup, database configuration, development, and license. Added detailed instructions for Neo4j and PostgreSQL setup, along with .env configuration examples. [1] [2] [3]

Configuration Centralization:

  • Replaced settings.toml with .env for configuration. Updated example.env with variables for logging, Neo4j, LLM models, and other settings. Removed example_settings.toml. [1] [2] [3]
  • Updated docker-compose.yml and docker-compose.win_mac.yml to use .env variables for all environment settings, improving flexibility and security. [1] [2]

Code Refactoring:

  • Improved readability by reformatting function calls and aligning arguments across multiple service classes, such as LLMService, RepositoryService, and ServiceCoordinator. [1] [2] [3] [4] [5]
  • Fixed a typo in initialize_services where resposistory_service was renamed to repository_service. [1] [2]

Logging Enhancements:

  • Updated logging initialization in prometheus/app/main.py to improve clarity and ensure proper configuration of log levels. [1] [2]

These changes collectively enhance the usability, maintainability, and clarity of the Prometheus project.

@dcloud347 dcloud347 requested a review from SophieHYe May 13, 2025 21:19
@dcloud347 dcloud347 merged commit ee32f01 into main May 22, 2025
1 of 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.

Docker does not read the model and API configurations from settings.toml

1 participant