You cannot select more than 25 topics Topics must start with a letter or number, can include dashes ('-') and can be up to 35 characters long.
zeripath 7a428fae4b
Ensure that all unmerged files are merged when conflict checking (#20528)
There is a subtle bug in the code relating to collating the results of
`git ls-files -u -z` in `unmergedFiles()`. The code here makes the
mistake of assuming that every unmerged file will always have a stage 1
conflict, and this results in conflicts that occur in stage 3 only being
dropped.

This PR simply adjusts this code to ensure that any empty unmergedFile
will always be passed down the channel.

The PR also adds a lot of Trace commands to attempt to help find future
bugs in this code.

Fix #19527

Signed-off-by: Andrew Thornton <art27@cantab.net>
2 years ago
..
check.go Fix checks in PR for empty commits #19603 (#20290) 2 years ago
check_test.go Move issues related files into models/issues (#19931) 2 years ago
commit_status.go Move issues related files into models/issues (#19931) 2 years ago
edits.go Move issues related files into models/issues (#19931) 2 years ago
lfs.go Move issues related files into models/issues (#19931) 2 years ago
main_test.go Use a struct as test options (#19393) 2 years ago
merge.go Add more linters to improve code readability (#19989) 2 years ago
patch.go Ensure that all unmerged files are merged when conflict checking (#20528) 2 years ago
patch_unmerged.go Ensure that all unmerged files are merged when conflict checking (#20528) 2 years ago
pull.go Move issues related files into models/issues (#19931) 2 years ago
pull_test.go Move issues related files into models/issues (#19931) 2 years ago
review.go Dismiss prior pull reviews if done via web in review dismiss (#20197) 2 years ago
temp_repo.go Move issues related files into models/issues (#19931) 2 years ago
update.go Move issues related files into models/issues (#19931) 2 years ago