cmd/list: Crude tablewriter error handling

Signed-off-by: Julien Champoux <[email protected]>

Author: Linkster78

internal/cmd/table.go

@@ -50,31 +50,55 @@ func RenderTable(w io.Writer, format string, header []string, data [][]string, r
 
 	switch format {
 	case TableFormatTable:
-		table := getBaseTable(w, header, data)
+		table, err := getBaseTable(w, header, data)
+		if err != nil {
+			return err
+		}
+
 		table.Options(tablewriter.WithRendition(tw.Rendition{
 			Settings: tw.Settings{
 				Separators: tw.Separators{
 					BetweenRows: tw.On,
 				},
 			},
 		}))
-		table.Render()
+
+		err = table.Render()
+		if err != nil {
+			return err
+		}
 
 	case TableFormatCompact:
-		table := getBaseTable(w, header, data)
+		table, err := getBaseTable(w, header, data)
+		if err != nil {
+			return err
+		}
+
 		table.Options(tablewriter.WithRendition(tw.Rendition{
 			Borders: tw.BorderNone,
 			Settings: tw.Settings{
 				Lines:      tw.LinesNone,
 				Separators: tw.SeparatorsNone,
 			},
 		}))
-		table.Render()
+
+		err = table.Render()
+		if err != nil {
+			return err
+		}
 
 	case TableFormatMarkdown:
-		table := getBaseTable(w, header, data)
+		table, err := getBaseTable(w, header, data)
+		if err != nil {
+			return err
+		}
+
 		table.Options(tablewriter.WithRenderer(renderer.NewMarkdown()))
-		table.Render()
+
+		err = table.Render()
+		if err != nil {
+			return err
+		}
 
 	case TableFormatCSV:
 		w := csv.NewWriter(w)
@@ -118,7 +142,7 @@ func RenderTable(w io.Writer, format string, header []string, data [][]string, r
 	return nil
 }
 
-func getBaseTable(w io.Writer, header []string, data [][]string) *tablewriter.Table {
+func getBaseTable(w io.Writer, header []string, data [][]string) (*tablewriter.Table, error) {
 	table := tablewriter.NewTable(
 		w,
 		tablewriter.WithRowAlignment(tw.AlignLeft),
@@ -129,8 +153,8 @@ func getBaseTable(w io.Writer, header []string, data [][]string) *tablewriter.Ta
 		}),
 	)
 	table.Header(header)
-	table.Bulk(data)
-	return table
+	err := table.Append(data)
+	return table, err
 }
 
 // Column represents a single column in a table.