]> source.dussan.org Git - gitea.git/commit
Set correct PR status on 3way on conflict checking (#19457) (#19458)
authorGusted <williamzijl7@hotmail.com>
Fri, 22 Apr 2022 01:11:42 +0000 (01:11 +0000)
committerGitHub <noreply@github.com>
Fri, 22 Apr 2022 01:11:42 +0000 (09:11 +0800)
commit09adc26eb63fed671ce45499365139fac092c224
tree21bfd30f4007c1718e65fc150c0071cf81d456ab
parent297346a7628c726a02fa14cbf1ba3edec5122ecd
Set correct PR status on 3way on conflict checking (#19457) (#19458)

- Backport #19457
  - When 3-way merge is enabled for conflict checking, it has a new interesting behavior that it doesn't return any error when it found a conflict, so we change the condition to not check for the error, but instead check if conflictedfiles is populated, this fixes a issue whereby PR status wasn't correctly on conflicted PR's.
  - Refactor the mergeable property(which was incorrectly set and lead me this bug) to be more maintainable.
  - Add a dedicated test for conflicting checking, so it should prevent future issues with this.
  - Ref: Fix the latest error for https://gitea.com/gitea/go-sdk/pulls/579

Co-authored-by: zeripath <art27@cantab.net>
integrations/pull_merge_test.go
models/pull.go
modules/convert/pull.go
services/pull/patch.go