+4 -2

appview/db/pipeline.go

reviewed

··· 168 168 169 169 // this is a mega query, but the most useful one: 170 170 // get N pipelines, for each one get the latest status of its N workflows 171 171 - func GetPipelineStatuses(e Execer, filters ...filter) ([]models.Pipeline, error) { 171 171 + func GetPipelineStatuses(e Execer, limit int, filters ...filter) ([]models.Pipeline, error) { 172 172 var conditions []string 173 173 var args []any 174 174 for _, filter := range filters { ··· 205 205 join 206 206 triggers t ON p.trigger_id = t.id 207 207 %s 208 208 - `, whereClause) 208 208 + order by p.created desc 209 209 + limit %d 210 210 + `, whereClause, limit) 209 211 210 212 rows, err := e.Query(query, args...) 211 213 if err != nil {

+3

appview/pipelines/pipelines.go

reviewed

··· 82 82 83 83 ps, err := db.GetPipelineStatuses( 84 84 p.db, 85 85 + 30, 85 86 db.FilterEq("repo_owner", repoInfo.OwnerDid), 86 87 db.FilterEq("repo_name", repoInfo.Name), 87 88 db.FilterEq("knot", repoInfo.Knot), ··· 125 124 126 125 ps, err := db.GetPipelineStatuses( 127 126 p.db, 127 127 + 1, 128 128 db.FilterEq("repo_owner", repoInfo.OwnerDid), 129 129 db.FilterEq("repo_name", repoInfo.Name), 130 130 db.FilterEq("knot", repoInfo.Knot), ··· 195 193 196 194 ps, err := db.GetPipelineStatuses( 197 195 p.db, 196 196 + 1, 198 197 db.FilterEq("repo_owner", repoInfo.OwnerDid), 199 198 db.FilterEq("repo_name", repoInfo.Name), 200 199 db.FilterEq("knot", repoInfo.Knot),

+2

appview/pulls/pulls.go

reviewed

··· 178 178 179 179 ps, err := db.GetPipelineStatuses( 180 180 s.db, 181 181 + len(shas), 181 182 db.FilterEq("repo_owner", repoInfo.OwnerDid), 182 183 db.FilterEq("repo_name", repoInfo.Name), 183 184 db.FilterEq("knot", repoInfo.Knot), ··· 649 648 repoInfo := f.RepoInfo(user) 650 649 ps, err := db.GetPipelineStatuses( 651 650 s.db, 651 651 + len(shas), 652 652 db.FilterEq("repo_owner", repoInfo.OwnerDid), 653 653 db.FilterEq("repo_name", repoInfo.Name), 654 654 db.FilterEq("knot", repoInfo.Knot),

+1 -14

appview/repo/repo_util.go

reviewed

··· 1 1 package repo 2 2 3 3 import ( 4 4 - "crypto/rand" 5 5 - "math/big" 6 4 "slices" 7 5 "sort" 8 6 "strings" ··· 88 90 return 89 91 } 90 92 91 91 - func randomString(n int) string { 92 92 - const letters = "abcdefghijklmnopqrstuvwxyzABCDEFGHIJKLMNOPQRSTUVWXYZ" 93 93 - result := make([]byte, n) 94 94 - 95 95 - for i := 0; i < n; i++ { 96 96 - n, _ := rand.Int(rand.Reader, big.NewInt(int64(len(letters)))) 97 97 - result[i] = letters[n.Int64()] 98 98 - } 99 99 - 100 100 - return string(result) 101 101 - } 102 102 - 103 93 // grab pipelines from DB and munge that into a hashmap with commit sha as key 104 94 // 105 95 // golang is so blessed that it requires 35 lines of imperative code for this ··· 104 118 105 119 ps, err := db.GetPipelineStatuses( 106 120 d, 121 121 + len(shas), 107 122 db.FilterEq("repo_owner", repoInfo.OwnerDid), 108 123 db.FilterEq("repo_name", repoInfo.Name), 109 124 db.FilterEq("knot", repoInfo.Knot),