Delete comment

Author: NorthRealm

modules/actions/run.go

@@ -13,7 +13,7 @@ import (
 	"code.gitea.io/gitea/modules/storage"
 )
 
-// TODO: When deleting a run, it should at lease delete artifacts, tasks logs, database record.
+// DeleteRun deletes a done workflow run
 func DeleteRun(ctx context.Context, repoID int64, run *actions.ActionRun, jobs []*actions.ActionRunJob) error {
 	tasks := make(actions.TaskList, 0)
 
@@ -79,6 +79,5 @@ func DeleteRun(ctx context.Context, repoID int64, run *actions.ActionRun, jobs [
 		}
 	}
 
-	// TODO: Delete commit status? Looks like it has no direct reference to a run/task/job. Not quite feasible without modifying db models (Dangerous).
 	return nil
 }

templates/repo/actions/runs_list.tmpl

@@ -36,7 +36,6 @@
 					<div class="run-list-meta">{{svg "octicon-calendar" 16}}{{DateUtils.TimeSince .Updated}}</div>
 					<div class="run-list-meta">{{svg "octicon-stopwatch" 16}}{{.Duration}}</div>
 				</div>
-				<!-- TODO: ? -->
 				{{if and (.Link) (or (eq .Status.String "success") (eq .Status.String "skipped") (eq .Status.String "cancelled"))}}
 					<button class="ui btn interact-bg link-action tw-p-2"
 						data-url="{{.Link}}/delete"