gitea

Git with a cup of tea, painless self-hosted git service Mirror for internal git.with.parts use https://git.with.parts

History

zeripath 198342efe4 Add setting to disable the git apply step in test patch (#22130 ) (#22170 ) Backport #22130 For a long time Gitea has tested PR patches using a git apply --check method, and in fact prior to the introduction of a read-tree assisted three-way merge in #18004, this was the only way of checking patches. Since #18004, the git apply --check method has been a fallback method, only used when the read-tree three-way merge method has detected a conflict. The read-tree assisted three-way merge method is much faster and less resource intensive method of detecting conflicts. #18004 kept the git apply method around because it was thought possible that this fallback might be able to rectify conflicts that the read-tree three-way merge detected. I am not certain if this could ever be the case. Given the uncertainty here and the now relative stability of the read-tree method - this PR makes using this fallback optional but enables it by default. A `log.Critical` has been added which will alert if the `git apply --check` method was successful at checking a PR that `read-tree` failed on. The hope is that none of these log.Critical messages will be found and there will be no significant difference in conflict detection. Thus we will be able to remove the git apply fallback in future, and/or improve the read-tree three-way merge method to catch any conflicts that git apply method might have been able to fix. An additional benefit for anyone who disables the check method is that patch checking should be significantly less resource intensive and much quicker. (See https://github.com/go-gitea/gitea/issues/22083\#issuecomment-1347961737) Ref #22083 Signed-off-by: Andrew Thornton <art27@cantab.net> <!-- Please check the following: 1. Make sure you are targeting the `main` branch, pull requests on release branches are only allowed for bug fixes. 2. Read contributing guidelines: https://github.com/go-gitea/gitea/blob/main/CONTRIBUTING.md 3. Describe what your pull request does and which issue you're targeting (if any) --> Signed-off-by: Andrew Thornton <art27@cantab.net> Co-authored-by: KN4CK3R <admin@oldschoolhack.me>		2 years ago
..
agit	Replace all instances of fmt.Errorf(%v) with fmt.Errorf(%w) (#21551 )	2 years ago
asymkey	…
attachment	Replace all instances of fmt.Errorf(%v) with fmt.Errorf(%w) (#21551 )	2 years ago
auth	Revert unrelated changes for SMTP auth (#21767 ) (#21768 )	2 years ago
automerge	…
comments	…
context	…
cron	Refactor git command arguments and make all arguments to be safe to be used (#21535 )	2 years ago
externalaccount	…
forms	Revert unrelated changes for SMTP auth (#21767 ) (#21768 )	2 years ago
gitdiff	Refactor git command arguments and make all arguments to be safe to be used (#21535 )	2 years ago
issue	Fix issue/PR numbers (#22037 ) (#22044 )	2 years ago
lfs	Prevent Authorization header for presigned LFS urls (#21531 )	2 years ago
mailer	Correct the fallbacks for mailer configuration (#21945 ) (#21953 )	2 years ago
markup	Link mentioned user in markdown only if they are visible to viewer (#21554 )	2 years ago
migrations	Replace all instances of fmt.Errorf(%v) with fmt.Errorf(%w) (#21551 )	2 years ago
mirror	Refactor git command arguments and make all arguments to be safe to be used (#21535 )	2 years ago
org	Replace all instances of fmt.Errorf(%v) with fmt.Errorf(%w) (#21551 )	2 years ago
packages	…
pull	Add setting to disable the git apply step in test patch (#22130 ) (#22170 )	2 years ago
release	Revert "Do not send notifications for draft releases (#21451 )" (#21594 )	2 years ago
repository	Do not list active repositories as unadopted (#22034 ) (#22166 )	2 years ago
task	Replace all instances of fmt.Errorf(%v) with fmt.Errorf(%w) (#21551 )	2 years ago
user	Replace all instances of fmt.Errorf(%v) with fmt.Errorf(%w) (#21551 )	2 years ago
webhook	Replace all instances of fmt.Errorf(%v) with fmt.Errorf(%w) (#21551 )	2 years ago
wiki	Replace all instances of fmt.Errorf(%v) with fmt.Errorf(%w) (#21551 )	2 years ago