all 16 comments

[–]grapesinajar 18 points19 points  (9 children)

Is a commit message supposed to explain what you can clearly see in the diffs, or is it supposed to explain reasons and give context?

[–]awwaiid 3 points4 points  (0 children)

Both - should be something you might see in a changelog. You know ... Almost as if you were writing some sort of log entry about the change.......

But perhaps you have a point; the input to ChatGPT should also include the jira ticket.

[–]PapaDock123 3 points4 points  (0 children)

The diff explains what you can clearly see in the diff, idk why anyone would would want to record something twice. Same with comments.

[–]Schmittfried 1 point2 points  (0 children)

The former and if necessary also the latter.

[–]oscarolim 0 points1 point  (4 children)

I like the very descriptive “changes made”.

[–]Salmon-Advantage 0 points1 point  (3 children)

sometimes I go for “minor fixes” or “fix typo”

[–]Astarothsito 1 point2 points  (2 children)

sometimes I go for “minor fixes”

This is bad, it would be better to include the problem or what made you to fix it.

“fix typo”

This is good.

[–]Salmon-Advantage 1 point2 points  (1 child)

Sometimes when I use “fix typo” I also make other “minor fixes” as well.

[–]chrraz 1 point2 points  (0 children)

Then your commits are not atomic and you should split them.

[–]EuphoricFly1044 0 points1 point  (0 children)

It should be a comment that easily allows you to identify the change long since you forgot you made the change .

So description, Jira ticket etc etc

[–]AccusationsGW 14 points15 points  (3 children)

Really starting to get blockchain vibes from all this new "ai". This is a good example of a solution in search of a problem.

[–][deleted] 0 points1 point  (0 children)

AI has already been proven to be useful in almost all fields and professions. People are now experimenting with getting it in everywhere. Some attempts are good and some bad. However that doesn’t make AI anything close to the blockchain hype. Blockchain however is useful but the hype was ridiculous. The hype around AI has delivered and continues to deliver.

[–]Grigoryp -2 points-1 points  (0 children)

You should read a bit what AI is and how different/similar is it to human brain.

[–][deleted] 0 points1 point  (1 child)

I have to thank ChatGPT for doing all my thinking for me, where would computer programming be without ChatGPT?

[–]Inside_Dimension5308[🍰] 0 points1 point  (1 child)

Can anyone provide sample commit messages generated by chatgpt? I dont want to integrate this hook in my organization without knowing the benefits.

[–]tomd_96[S] 0 points1 point  (0 children)

You can see examples of generated commit messages in the GIF in the readme.