feat: sort by commit date (#459)

api/yolopb.proto

@@ -100,6 +100,9 @@ message BuildList {
 
     // filter builds with merge requests
     bool with_no_mergerequest = 14;
+
+    // sort by commit date
+    bool sort_by_commit_date = 15;
   }
   message Response {
     repeated Build builds = 1;

go/pkg/yolostore/store.go

@@ -2,6 +2,7 @@ package yolostore
 
 import (
 	"fmt"
+	"sort"
 	"strings"
 
 	"berty.tech/yolo/v2/go/pkg/yolopb"
@@ -240,6 +241,7 @@ type GetBuildListOpts struct {
 	MergeRequestState    []yolopb.MergeRequest_State
 	Branch               []string
 	Limit                int32
+	SortByCommitDate     bool
 }
 
 //  i.e, has_project=berty/berty -> has_project=https://github.com/berty/berty
@@ -384,6 +386,19 @@ func (s *store) GetBuildList(bl GetBuildListOpts) ([]*yolopb.Build, error) {
 		}
 	}
 
+	// sort by commit date
+	if bl.SortByCommitDate {
+		sort.Slice(builds, func(i, j int) bool {
+			if builds[i].HasCommit.CreatedAt == nil {
+				return false
+			}
+			if builds[j].HasCommit.CreatedAt == nil {
+				return true
+			}
+			return builds[i].HasCommit.CreatedAt.After(*builds[j].HasCommit.CreatedAt)
+		})
+	}
+
 	return builds, nil
 }
 

go/pkg/yolosvc/api_buildlist.go

@@ -32,6 +32,7 @@ func (svc *service) BuildList(ctx context.Context, req *yolopb.BuildList_Request
 		MergeRequestState:    req.MergerequestState,
 		Branch:               req.Branch,
 		Limit:                req.Limit,
+		SortByCommitDate:     req.SortByCommitDate,
 	}
 
 	var err error

go/pkg/yolosvc/driver_github.go

@@ -424,6 +424,8 @@ func (worker *githubWorker) batchFromWorkflowRun(run *github.WorkflowRun, prs []
 	updatedAt := run.GetUpdatedAt().Time
 	commitURL := run.GetHeadRepository().GetHTMLURL() + "/commit/" + run.GetHeadSHA()
 
+	commitCreatedAt := run.GetHeadCommit().GetTimestamp().Time
+
 	newBuild := yolopb.Build{
 		ID:              run.GetHTMLURL(),
 		ShortID:         fmt.Sprintf("%d", run.GetID()),
@@ -440,6 +442,17 @@ func (worker *githubWorker) batchFromWorkflowRun(run *github.WorkflowRun, prs []
 		Message:         run.GetHeadCommit().GetMessage(),
 	}
 
+	newCommit := yolopb.Commit{}
+	if run.GetHeadCommit() != nil {
+		newCommit = yolopb.Commit{
+			ID:        run.GetHeadCommit().GetID(),
+			CreatedAt: &commitCreatedAt,
+			Message:   run.GetHeadCommit().GetMessage(),
+			Driver:    yolopb.Driver_GitHub,
+			Branch:    run.GetHeadBranch(),
+		}
+	}
+
 	if len(prs) > 0 {
 		pr := prs[0] // only take the first one
 		newBuild.HasMergerequestID = pr.GetHTMLURL()
@@ -495,6 +508,7 @@ func (worker *githubWorker) batchFromWorkflowRun(run *github.WorkflowRun, prs []
 
 	guessMissingBuildInfo(&newBuild)
 	batch.Builds = append(batch.Builds, &newBuild)
+	batch.Commits = append(batch.Commits, &newCommit)
 	return batch
 }