-
-
Notifications
You must be signed in to change notification settings - Fork 5.4k
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
Option to disable appending of Co-authored-by/Co-committed-by lines to the commit message body #17194
Comments
Hm, I have to say, I am not a particular fan of this request: First of all, you would need to define where this setting will be customizable: For the whole server, for a specific user/ organization, or even for a specific repo. Depending on your choice, you have more or fewer edge cases and hence more complexity and potential bugs. There are already way too many configuration options, and there will be even more. You should still be able to use Gitea in the end; you should not be stuck configuring it for eternity. Secondly, those lines are appended to the end of the commit message. Lastly, perhaps your opinion on these lines changes over time, and suddenly you do want to find out who co-authored and co-committed these changes without much hassle. My approach on this would be the following: |
You can change the template repo/issue/view_content/pull.tmpl L403. We should probably extract this bit out to a subtemplate then administrators could change it this on a per user basis. |
Thank you for your answer. The main reason for this feature request is because the git logs are in the following form:
As you can see my name appears three times in the log. I even wonder if this is a bug if we print the same user details in the author line as well as in the message trailer:
I could try to implement this by myself so you can give your opinion and eventually merge it if makes sense. |
If the author and the co-author has the same email. It should be a bug. |
I would like to add that these additional lines are currently added directly as a second and third line in the commit message, in contrast to git recommendations/rules that the second line should be empty, separating the short commit message from the long description if there is one. As a result,
I would not mind these additional lines (except that they often do not contain actually useful information) if they at least skipped a line. |
Just our two cents' worth: we use gitea in a small non-profit, with a rather centralised workflow. In our use-case, the forced appending of Co-authored-by/Co-committed-by lines to each merge commit message body is not meaningful. Being able to disable those automatically appended lines would be wonderful. |
This PR adds another option to app.ini make co-committed-by and co-authored-by trailers optional on a per server basis. Fix go-gitea#17194 Signed-off-by: Andrew Thornton <[email protected]>
This PR adds another option to app.ini make co-committed-by and co-authored-by trailers optional on a per server basis. Fix #17194 Signed-off-by: Andrew Thornton <[email protected]>
Thank you for this new global option, it is much appreciated. Heartfelt gratitude to gitea's developers. Our users, all non-technical (small non-profit working in the humanities field) are able to use Git only because of gitea. |
A little curious, if the users are all non-technical, how do you use git, what is your workflow? It will make Gitea developers know more about the real users. 😊 |
@wxiaoguang Thank you for asking. I actually wanted to mention, but felt it was off topic and adding unwanted noise to the ticket. Our non-profit helps with the preservation of folklore songs. All helpers involved are computer literate, but, perhaps erring on the humanities side, are a bit averse to deeper technological aspects. CLI used is not even remotely possible, at least for now. We have a private Gitea instance, only for our projects. Each song is transcribed using a textual form (ABC, Lilypond, MusicXML) and added to a Song repository via Gitea. Interaction happens only via Gitea GUI (no CLI). Over the course of time, somebody may add more details, or fix some incorrect notation, editing and doing a PR. All changes are checked via an internal mailing list. Each change and each approval comes on the project's combined, concerted and aggregate effort. That is why, in our use-case, having "Co-authored-by/Co-committed-by" line is not meaningful to us. |
…7848) This PR adds another option to app.ini make co-committed-by and co-authored-by trailers optional on a per server basis. Fix go-gitea#17194 Signed-off-by: Andrew Thornton <[email protected]>
Feature Description
I think it would be great to have an option to disable appending of these lines. I work in a team that does not like to pollute the commit message body with these lines when we use the squash merge style.
Screenshots
No response
The text was updated successfully, but these errors were encountered: