wip
This commit is contained in:
parent
adffc67ca0
commit
e537e5d00c
@ -104,10 +104,18 @@ func (rs *PRSet) IsConsistent() bool {
|
|||||||
}
|
}
|
||||||
|
|
||||||
func (rs *PRSet) IsReviewed(gitea common.GiteaReviewFetcher) bool {
|
func (rs *PRSet) IsReviewed(gitea common.GiteaReviewFetcher) bool {
|
||||||
common_reviewers := rs.config.Reviewers
|
configReviewers := ParseReviewers(rs.config.Reviewers)
|
||||||
|
|
||||||
is_reviewed := false
|
is_reviewed := false
|
||||||
for _, pr := range rs.prs {
|
for _, pr := range rs.prs {
|
||||||
r, err := FetchGiteaReviews(gitea, common_reviewers, pr.pr.Base.Repo.Owner.UserName, pr.pr.Base.Repo.Name, pr.pr.Index)
|
var reviewers []string
|
||||||
|
if rs.IsPrjGitPR(pr.pr) {
|
||||||
|
reviewers = configReviewers.Prj
|
||||||
|
} else {
|
||||||
|
reviewers = configReviewers.Pkg
|
||||||
|
}
|
||||||
|
|
||||||
|
r, err := FetchGiteaReviews(gitea, reviewers, pr.pr.Base.Repo.Owner.UserName, pr.pr.Base.Repo.Name, pr.pr.Index)
|
||||||
if err != nil {
|
if err != nil {
|
||||||
return false
|
return false
|
||||||
}
|
}
|
||||||
|
@ -16,7 +16,7 @@ import (
|
|||||||
|
|
||||||
func TestPR(t *testing.T) {
|
func TestPR(t *testing.T) {
|
||||||
baseConfig := common.AutogitConfig{
|
baseConfig := common.AutogitConfig{
|
||||||
Reviewers: []string{"super1", "super2"},
|
Reviewers: []string{"+super1", "*super2", "m1", "-m2"},
|
||||||
Branch: "branch",
|
Branch: "branch",
|
||||||
Organization: "foo",
|
Organization: "foo",
|
||||||
GitProjectName: "barPrj",
|
GitProjectName: "barPrj",
|
||||||
@ -104,6 +104,7 @@ func TestPR(t *testing.T) {
|
|||||||
reviews: []*models.PullReview{
|
reviews: []*models.PullReview{
|
||||||
{Body: "LGTM", User: &models.User{UserName: "super1"}, State: common.ReviewStateApproved},
|
{Body: "LGTM", User: &models.User{UserName: "super1"}, State: common.ReviewStateApproved},
|
||||||
{Body: "LGTM", User: &models.User{UserName: "super2"}, State: common.ReviewStateApproved},
|
{Body: "LGTM", User: &models.User{UserName: "super2"}, State: common.ReviewStateApproved},
|
||||||
|
{Body: "LGTM", User: &models.User{UserName: common.Bot_BuildReview}, State: common.ReviewStateApproved},
|
||||||
},
|
},
|
||||||
},
|
},
|
||||||
},
|
},
|
||||||
|
Loading…
x
Reference in New Issue
Block a user