search

go-gitea / gitea

Git with a cup of tea! Painless self-hosted all-in-one software development service, including Git hosting, code review, team collaboration, package registry and CI/CD
https://gitea.com
MIT License
44.71k stars 5.46k forks source link

Cleanup the output from the gitea internal hook(s). #23364

Open rappazzo opened 1 year ago

rappazzo commented 1 year ago

Feature Description

Problem statement

When code is pushed to a repository, there is output sent back from the server: 

remote: .. Processing 2 references
remote: Processed 2 references in total

This seems like it is more debugging output than helpful information. It would make sense to see that output if the push operation streamed the response as it was processing it, but that is not in the git protocol (AFAIK) so it is printed all at once. Additionally, git already provides more detailed information about what was processed at the same time these messages are presented. For example: 

   c0145ff94906..5c9e4aa4447d  local-refA -> remote-refA
 * [new reference]             local-refB -> remote-refB

Proposal

Remove the redundant information about how many references were processed.

Screenshots

No response

yardenshoham commented 1 year ago

Perhaps we should change the default of VERBOSE_PUSH to false