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

Open
opened 2025-11-02 09:06:30 -06:00 by GiteaMirror · 1 comment
Owner

Originally created by @rappazzo on GitHub (Mar 7, 2023).

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

Originally created by @rappazzo on GitHub (Mar 7, 2023). ### 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_
GiteaMirror added the type/enhancementtype/proposal labels 2025-11-02 09:06:30 -06:00
Author
Owner

@yardenshoham commented on GitHub (Apr 7, 2023):

Perhaps we should change the default of VERBOSE_PUSH to false

@yardenshoham commented on GitHub (Apr 7, 2023): Perhaps we should change the default of `VERBOSE_PUSH` to `false`
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/gitea#10398