Skip to content

Conversation

@nebjak
Copy link
Contributor

@nebjak nebjak commented Sep 13, 2025

No description provided.

@nebjak nebjak requested a review from Copilot September 14, 2025 06:58
Copy link

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 adds basic configuration support via environment variables to TLQ. The main changes replace hardcoded values with configurable options and add comprehensive testing for the configuration system.

  • Adds a configuration module that reads settings from environment variables
  • Replaces hardcoded constants with configurable values (port, message size limit, log level)
  • Updates documentation to explain the new configuration options

Reviewed Changes

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

Show a summary per file
File Description
src/config/mod.rs New configuration module with environment variable parsing and comprehensive tests
src/services/mod.rs Replaces hardcoded MESSAGE_SIZE_LIMIT with configurable max_message_size
src/main.rs Updates server startup to use configurable port and log level
src/lib.rs Adds config module to public API
USAGE.md Documents new configuration options with examples
README.md Adds configuration section with examples
Dockerfile Sets default environment variables and updates healthcheck

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

@nebjak nebjak merged commit a5d736c into main Sep 14, 2025
1 check 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