8000 Feat/commit message by jsburckhardt · Pull Request #6 · jsburckhardt/gic · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Feat/commit message #6

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 6 commits into from
Sep 2, 2024
Merged

Feat/commit message #6

merged 6 commits into from
Sep 2, 2024

Conversation

jsburckhardt
Copy link
Owner

This pull request includes several updates to enhance the functionality and configuration of the project. The key changes involve improvements to the dev container configuration, updates to the commit message generation logic, and dependency updates.

Dev Container Configuration:

  • Updated the name field and added GPU support in .devcontainer/devcontainer.json.
  • Added configuration examples for using Ollama and Azure AD in the README.md.

Commit Message Generation:

  • Refactored the Execute function in cmd/root.go to separate command execution logic.
  • Added a new method Commit to handle commit operations in internal/git/git.go.
  • Introduced support for Ollama in internal/llm/llm.go.

Dependency Updates:

  • Replaced github.com/jsburckhardt/gic with github.com/ollama/ollama in go.mod.
  • Updated the version of golang.org/x/exp in go.mod.

Configuration Changes:

  • Renamed the Commit field to ShouldCommit in internal/config/config.go.

These changes collectively improve the project's configuration flexibility, enhance the commit message generation process, and update dependencies to newer versions.

Add a new `Commit` function that commits staged changes with a generated message. The function will either execute the commit or print the suggested message based on the commit flag.

fix(makefile): remove unnecessary flag from gofmt command
… and Ollama

Enhance the README with detailed setup instructions for AzureAD and running Ollama locally in a devcontainer.
Included cautions about model selection and links to further details on expected outputs.
- Renamed project from "Getting Started" to "gic".
- Added GPU support with `--gpus all` in run arguments.
- Minor cleanup in the features section and comments.
Added functionality to generate commit messages using the Ollama API within the LLM package. This includes a new method `GenerateCommitMessageOllama` that constructs a request to the Ollama API, processes the response, and returns the generated commit message based on the provided diff input.

Additionally, replaced a deprecated dependency (`github.com/jsburckhardt/gic`) with `github.com/ollama/ollama` in the module files.

BREAKING CHANGE: The connection type must now include support for "ollama".
Adjust the `fmt` target in the Makefile to format Go files in the current directory, improving the ease of running formatting commands.
Refactor the Execute function to improve readability and maintainability.
Updated the configuration field name from `Commit` to `ShouldCommit` for clarity.
Simplified the commit logic to consistently use the configuration struct.

BREAKING CHANGE: The `Commit` parameter in the `git.Commit` function has been changed to accept the entire `config.Config` struct instead.
@jsburckhardt jsburckhardt merged commit e1bdbed into main Sep 2, 2024
3 checks passed
@jsburckhardt jsburckhardt deleted the feat/commit-message branch September 2, 2024 12:09
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
0