From a30c82252948a82e137d5a14d0acff5f15786a10 Mon Sep 17 00:00:00 2001 From: viljarb Date: Wed, 11 Mar 2026 18:17:52 +0100 Subject: [PATCH] fixing logic a bit --- cmd/pm/tasks/priorityManagement.go | 17 ++++++++++++++--- 1 file changed, 14 insertions(+), 3 deletions(-) diff --git a/cmd/pm/tasks/priorityManagement.go b/cmd/pm/tasks/priorityManagement.go index 3c9ebdc..662792e 100644 --- a/cmd/pm/tasks/priorityManagement.go +++ b/cmd/pm/tasks/priorityManagement.go @@ -124,17 +124,23 @@ func (t *PriorityManagementTask) Validate(ctx context.Context) ValidationFeedbac return expect.ValidationFeedback } + taskpart1 := 0 + taskpart2 := 0 taskIssue := t.setupIssue if taskIssue.Assignee != "Me" { expect.Assert(taskIssue.Assignee == "Me", fmt.Sprintf("'%s' not assigned to you, but to '%s'", taskIssue.Title, taskIssue.Assignee)) - return expect.ValidationFeedback + taskpart1++ } if taskIssue.Status == models.StatusOpen { expect.Assert(taskIssue.Status == models.StatusInProgress, fmt.Sprintf("'%s' status should be 'In Progress', but was '%s'", taskIssue.Title, taskIssue.Status)) + taskpart1++ + } + + if taskpart1>0 { return expect.ValidationFeedback } @@ -144,16 +150,21 @@ func (t *PriorityManagementTask) Validate(ctx context.Context) ValidationFeedbac if issues[ii].Priority != 4 { expect.Assert(issues[ii].Priority == 4, fmt.Sprintf("'%s' priority should be 4 (critical), but was '%d'", issues[ii].Title, issues[ii].Priority)) - return expect.ValidationFeedback + taskpart2++ } } else { if issues[ii].Priority != 1 { expect.Assert(issues[ii].Priority == 1, fmt.Sprintf("'%s' priority should be 1 (low), but was '%d'", issues[ii].Title, issues[ii].Priority)) - return expect.ValidationFeedback + taskpart2++ } } } + + + if taskpart2>0 { + return expect.ValidationFeedback + } expect.Assert(taskIssue.Status == models.StatusClosed, fmt.Sprintf("'%s' is not set to closed", taskIssue.Title))