Feat/connection request #17
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Refactor the config package to manage connection settings via a new ConnectionConfig struct, loading values from environment variables. Remove cli.go as its functionality has been integrated into the config handling. Add a method to create a sample `.env` file for easier environment setup. Enhance validation for the connection configuration to check for required environment variables based on the selected service provider. BREAKING CHANGE: This refactor changes the structure of the configuration management and requires updating the environment variable setup in the deployment.
…es setup Update the README to include detailed instructions for setting environment variables required for LLM connections. Added clarification on the supported connection types and provided an example `.env` file. Also, modified the sample configuration to ensure it matches the new structure and updates in the code regarding the handling of different connection types and their corresponding configuration values.
…iguration Eliminate tokens related code in config management to streamline configuration handling. The `Tokens` field in `ConnectionConfig` and associated environment variable handling have been removed for simplification. No functional changes were introduced since this refactor serves to enhance code clarity.
…n README - Introduce a new .gic configuration file to enhance the commit message generation process. - Update README with instructions for generating a sample .env file and configuring connections for OpenAI, Azure, and Ollama. - Remove outdated sample.gic.yaml file to streamline documentation.
…prove formatting and linting Add package documentation for the cmd module, improve `makefile` commands to format code with `goimports`, and include a linting fix command that automatically corrects issues. Additionally, update the usage of azopenai's ChatRequestSystemMessageContent for better integration with Azure SDK. Related to changes in dependencies and configurations, various package versions were updated in `go.mod` and `go.sum`.
…sistency Updated struct and function names for uniformity in naming conventions across the configuration management code.
Refactor error handling in the logger to provide clearer distinction between error and warning logs. This includes renaming the logging method comment from "Error" to "Warn" for proper documentation.
…inter and cleanup comments
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This pull request includes significant updates to the configuration and environment variable handling for the project, as well as improvements to the command-line interface and dependencies. The most important changes include adding support for generating a sample
.envfile, updating the README with detailed environment variable instructions, and refactoring the configuration loading process.Configuration and Environment Variables:
.envfile using thegic --create-sample-dotenvcommand. (cmd/root.go,README.md) [1] [2] [3] [4] [5]README.md) [1] [2] [3]Codebase Refactoring:
internal/config/config.go) [1] [2] [3]internal/cli/cli.gofile, consolidating environment variable loading into the configuration package. (internal/cli/cli.go)Dependency Updates:
go.modto their latest versions. (go.mod)