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.

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

Custom commit messages: sentence casing and bz keywords #38

Open
eclipseo opened this issue Aug 15, 2023 · 9 comments
Open

Custom commit messages: sentence casing and bz keywords #38

eclipseo opened this issue Aug 15, 2023 · 9 comments

Comments

@eclipseo
Copy link

Hi,

I'm not really keen on the current commit messages, either for import or updates. They are not capitalised and they don't include the correct format for bug numbers (Close: rhbz#XXXXXX, Closes: rhbz#XXXXXX, Fix: rhbz#XXXXXX, Fixes: rhbz#XXXXXX).
How can I customize them?

@juhp
Copy link
Owner

juhp commented Aug 19, 2023

Hi, thanks for reporting this feedback

Do you have any concrete examples?
I take it you are not using fbrnch commit?
On the plus side fbrnch will add mentioned bugs to Bodhi when pushing a manual update (ie not for rawhide/branched)

Let me think a little on this, certainly I agree the Closes/Fixes keywards are important I guess.
Capitalizing is probably more readable too, I suppose.

@eclipseo
Copy link
Author

eclipseo commented Aug 19, 2023

No I was importing a new package.

fbrnch import -F golang-github-hashicorp-hc-install

And I got that commit message:

https://src.fedoraproject.org/rpms/golang-github-hashicorp-hc-install/c/8136ca061e5ff24737f2fdb40a52085da888172f?branch=rawhide

import #2224864

What I usually do is :

git commit -m "Initial import" -m "Close: rhbz#2225225"

That format will automatically close the bug for updates too. So I wish to be able to customize them.

I haven't tested fbrnch update yet but from the code it seemed similar.

@juhp
Copy link
Owner

juhp commented Oct 2, 2023

Are these bz keywords documented somewhere?
Or where are they implemented? (I tried grepping bodhi but nothing)

@juhp
Copy link
Owner

juhp commented Oct 17, 2023

I would like to understand how those keywords work in Fedora infra before proceeding

@juhp
Copy link
Owner

juhp commented Oct 18, 2023

@juhp
Copy link
Owner

juhp commented Oct 18, 2023

Let me start addressing this with a small tweak then, which should help automatic updates for %autochangelog.

For non-%autochangelog this would require an appropriate %changelog message of course

juhp added a commit that referenced this issue Oct 19, 2023
@juhp
Copy link
Owner

juhp commented Oct 19, 2023

I released 1.3.3 with this tweak
so autoupdates should now close Review Request bugs at least when using autochangelog.

@juhp
Copy link
Owner

juhp commented Aug 19, 2024

@eclipseo did that help at all?

I suppose I can capitalize the commit message too - I personally tend not to do that, but I get it is more readable

@juhp juhp changed the title Custom commit messages Custom commit messages: sentence casing and bz keywords Aug 19, 2024
@juhp
Copy link
Owner

juhp commented Aug 19, 2024

In 1.5.1 I endeavor to sentence-case commit/changelog messages generated by fbrnch.

I am a little wary of forcing it onto user-provided messages.

juhp added a commit that referenced this issue Dec 13, 2024
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

No branches or pull requests

2 participants