@@ -336,10 +336,16 @@ func (s service) ListTimeline(ctx context.Context, repo string, id uint64, opt * Body: c.Message, }) } } } sort.Slice(cs, func(i, j int) bool { if cs[i].File == cs[j].File { return cs[i].Line < cs[j].Line } return cs[i].File < cs[j].File }) timeline = append(timeline, change.Review{ ID: fmt.Sprint(idx), // TODO: message.ID is not uint64; e.g., "bfba753d015916303152305cee7152ea7a112fe0". User: s.gerritUser(message.Author), CreatedAt: message.Date.Time, Body: body, @@ -414,10 +420,16 @@ func (s service) ListTimeline(ctx context.Context, repo string, id uint64, opt * Body: c.Message, }) } } } sort.Slice(cs, func(i, j int) bool { if cs[i].File == cs[j].File { return cs[i].Line < cs[j].Line } return cs[i].File < cs[j].File }) reviewState := reviewState(labels) if body == "" && len(cs) == 0 && reviewState == state.ReviewNoScore && labels != "" { // Skip an empty comment that, e.g., just sets a Run-TryBot+1 label. continue }