fix lints

Signed-off-by: Carlos Panato <[email protected]>

Author: cpanato

cmd/ci-reporter/cmd/root.go

@@ -246,8 +246,8 @@ func PrintReporterData(cfg *Config, reports *CIReportDataFields) error {
 		if err != nil {
 			return fmt.Errorf("could not write to output stream: %w", err)
 		}
-		data := [][]string{}
 
+		data := [][]string{}
 		table := tablewriter.NewTable(out,
 			tablewriter.WithConfig(tablewriter.Config{
 				Header: tw.CellConfig{
@@ -292,8 +292,13 @@ func PrintReporterData(cfg *Config, reports *CIReportDataFields) error {
 			}
 		}
 
-		table.Append(data)
-		table.Render()
+		if err := table.Bulk(data); err != nil {
+			return err
+		}
+
+		if err := table.Render(); err != nil {
+			return err
+		}
 
 		// write a summary
 		countCategories := map[string]int{}

cmd/schedule-builder/cmd/markdown.go

@@ -77,14 +77,14 @@ func parsePatchSchedule(patchSchedule PatchSchedule) string {
 				continue
 			}
 
-			table.Append([]string{
+			_ = table.Append([]string{
 				targetDate.Format(refDateMonthly),
 				strings.TrimSpace(upcoming.CherryPickDeadline),
 				strings.TrimSpace(upcoming.TargetDate),
 			})
 		}
 
-		table.Render()
+		_ = table.Render()
 
 		output = append(output, tableString.String())
 	}
@@ -130,14 +130,14 @@ func parsePatchSchedule(patchSchedule PatchSchedule) string {
 
 		// Check if the next patch release is in the Previous Patch list, if yes dont read in the output
 		if !patchReleaseInPreviousList(releaseSchedule.Next.Release, releaseSchedule.PreviousPatches) {
-			table.Append([]string{strings.TrimSpace(releaseSchedule.Next.Release), strings.TrimSpace(releaseSchedule.Next.CherryPickDeadline), strings.TrimSpace(releaseSchedule.Next.TargetDate), ""})
+			_ = table.Append([]string{strings.TrimSpace(releaseSchedule.Next.Release), strings.TrimSpace(releaseSchedule.Next.CherryPickDeadline), strings.TrimSpace(releaseSchedule.Next.TargetDate), ""})
 		}
 
 		for _, previous := range releaseSchedule.PreviousPatches {
-			table.Append([]string{strings.TrimSpace(previous.Release), strings.TrimSpace(previous.CherryPickDeadline), strings.TrimSpace(previous.TargetDate), strings.TrimSpace(previous.Note)})
+			_ = table.Append([]string{strings.TrimSpace(previous.Release), strings.TrimSpace(previous.CherryPickDeadline), strings.TrimSpace(previous.TargetDate), strings.TrimSpace(previous.Note)})
 		}
 
-		table.Render()
+		_ = table.Render()
 
 		output = append(output, tableString.String())
 	}
@@ -204,10 +204,10 @@ func parseReleaseSchedule(releaseSchedule ReleaseSchedule) string {
 		)
 
 		for _, timeline := range releaseSchedule.Timeline {
-			table.Append([]string{strings.TrimSpace(timeline.What), strings.TrimSpace(timeline.Who), strings.TrimSpace(timeline.When), strings.TrimSpace(timeline.Week), strings.TrimSpace(timeline.CISignal), ""})
+			_ = table.Append([]string{strings.TrimSpace(timeline.What), strings.TrimSpace(timeline.Who), strings.TrimSpace(timeline.When), strings.TrimSpace(timeline.Week), strings.TrimSpace(timeline.CISignal), ""})
 		}
 
-		table.Render()
+		_ = table.Render()
 
 		relSched.TimelineOutput = tableString.String()
 	}

pkg/gcp/build/build.go

@@ -419,10 +419,14 @@ func ListJobs(project string, lastJobs int64) error {
 	)
 
 	for _, build := range req.Builds {
-		table.Append([]string{strings.TrimSpace(build.StartTime), strings.TrimSpace(build.FinishTime), strings.TrimSpace(build.Status), strings.TrimSpace(build.LogUrl)})
+		if err := table.Append([]string{strings.TrimSpace(build.StartTime), strings.TrimSpace(build.FinishTime), strings.TrimSpace(build.Status), strings.TrimSpace(build.LogUrl)}); err != nil {
+			return err
+		}
 	}
 
-	table.Render()
+	if err := table.Render(); err != nil {
+		return err
+	}
 
 	return nil
 }

pkg/gcp/gcb/history.go

@@ -205,13 +205,17 @@ func (h *History) Run() error {
 		out := time.Time{}.Add(diff)
 
 		step := fmt.Sprintf("`%s%s`", mock, subcommand)
-		table.Append([]string{
+		if err := table.Append([]string{
 			step, command, logs, start,
 			out.Format("15:04:05"), status[job.Status],
-		})
+		}); err != nil {
+			return fmt.Errorf("append row to table: %w", err)
+		}
 	}
 
-	table.Render()
+	if err := table.Render(); err != nil {
+		return err
+	}
 
 	fmt.Print(tableString.String())