dmitri.shuralyov.com/service/change/gerritapi

Include CL description comment.

Make it more consistent across the services.
dmitshur committed 6 years ago commit e0b1851bb90c56a9e6ab2adcdd90f5143212de3f
Showing partial commit. Full Commit
Collapse all
gerritapi/gerritapi.go
@@ -254,19 +254,17 @@ func (s service) ListTimeline(ctx context.Context, _ string, id uint64, opt *cha
	comments, _, err := s.cl.Changes.ListChangeComments(fmt.Sprint(id))
	if err != nil {
		return nil, err
	}
	var timeline []interface{}
	{
		timeline = append(timeline, change.Comment{
			ID:        "0",
			User:      s.gerritUser(chg.Owner),
			CreatedAt: time.Time(chg.Created),
			Body:      "", // THINK: Include commit message or no?
			Editable:  false,
		})
	}
	timeline = append(timeline, change.Comment{ // CL description.
		ID:        "0",
		User:      s.gerritUser(chg.Owner),
		CreatedAt: time.Time(chg.Created),
		Body:      "", // THINK: Include commit message or no?
		Editable:  false,
	})
	for idx, message := range chg.Messages {
		if strings.HasPrefix(message.Tag, "autogenerated:") {
			switch message.Tag[len("autogenerated:"):] {
			case "gerrit:merged":
				timeline = append(timeline, change.TimelineItem{