Skip to content
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.

Sign up for GitHub

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

Jump to bottom

[Feature]: add context for WHY in commit message #398

Open
di-sukharev opened this issue Aug 29, 2024 · 2 comments
Open

[Feature]: add context for WHY in commit message #398

di-sukharev opened this issue Aug 29, 2024 · 2 comments

Comments

@di-sukharev
Copy link
Owner

Description

lets help ai to understand why the changes were done

Suggested Solution

a flag: oco -- why="string"

mb string can be ticket description

Alternatives

No response

Additional Context

No response
@di-sukharev
Copy link
Owner Author

i would also make the WHY generation optional and turned off by default, as it's not 100% accurate and I would be ok personally with just "what has been changed" — it's enough now

@di-sukharev di-sukharev mentioned this issue Sep 1, 2024
Merged
@di-sukharev
Copy link
Owner Author

i added an OCO_WHY boolean config.

but this ticket still requires updating all the few shots we send in different languages to the LLMs, will do next

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@di-sukharev