update

NorthRealm · NorthRealm · commit 0506162f17c3 · 2025-08-09T13:11:43.000+08:00
diff --git a/services/mailer/mail_workflow_run.go b/services/mailer/mail_workflow_run.go
@@ -149,7 +149,7 @@ func MailActionsTrigger(ctx context.Context, sender *user_model.User, repo *repo
 	if setting.MailService == nil {
 		return
 	}
-	if run.Status.IsSkipped() {
+	if !run.Status.IsDone() || run.Status.IsSkipped() {
 		return
 	}
 
diff --git a/services/mailer/notify.go b/services/mailer/notify.go
@@ -208,8 +208,5 @@ func (m *mailNotifier) RepoPendingTransfer(ctx context.Context, doer, newOwner *
 }
 
 func (m *mailNotifier) WorkflowRunStatusUpdate(ctx context.Context, repo *repo_model.Repository, sender *user_model.User, run *actions_model.ActionRun) {
-	if !run.Status.IsDone() {
-		return
-	}
 	MailActionsTrigger(ctx, sender, repo, run)
 }

Original file line number	Diff line number	Diff line change
`@@ -149,7 +149,7 @@ func MailActionsTrigger(ctx context.Context, sender user_model.User, repo repo`
`149`	`149`	`if setting.MailService == nil {`
`150`	`150`	`return`
`151`	`151`	`}`
`152`		`- if run.Status.IsSkipped() {`
	`152`	`+ if !run.Status.IsDone() \|\| run.Status.IsSkipped() {`
`153`	`153`	`return`
`154`	`154`	`}`
`155`	`155`
Original file line number	Diff line number	Diff line change
`@@ -208,8 +208,5 @@ func (m mailNotifier) RepoPendingTransfer(ctx context.Context, doer, newOwner `
`208`	`208`	`}`
`209`	`209`
`210`	`210`	`func (m mailNotifier) WorkflowRunStatusUpdate(ctx context.Context, repo repo_model.Repository, sender user_model.User, run actions_model.ActionRun) {`
`211`		`- if !run.Status.IsDone() {`
`212`		`- return`
`213`		`- }`
`214`	`211`	`MailActionsTrigger(ctx, sender, repo, run)`
`215`	`212`	`}`