refactor: Make table.New generic

Signed-off-by: Tim Hårek Andreassen <tim@harek.no>

.build.yml

@@ -22,9 +22,7 @@ tasks:
             complete-build
         fi
 
-        version=$(echo "$tag" | tr -d 'v')
-
-        make GOOS="darwin" ARCH="amd64" build-release 
-        make GOOS="dawrin" ARCH="aarch64" build-release 
-        make GOOS="linux" ARCH="amd64" build-release 
-        make GOOS="windows" ARCH="amd64" build-release 
+        make GOOS="darwin" ARCH="amd64" TAG=$tag build-release 
+        make GOOS="darwin" ARCH="aarch64" TAG=$tag build-release 
+        make GOOS="linux" ARCH="amd64" TAG=$tag build-release 
+        make GOOS="windows" ARCH="amd64" TAG=$tag build-release 

Makefile

@@ -8,11 +8,11 @@ endif
 bindir = $(prefix)/bin
 builddir = ./build
 
-VERSION = 0.0.5
+VERSION = 0.0.6
 
 .PHONY: build
 build:
-	go build -o $(builddir)
+	go build -o $(builddir)/
 
 .PHONY: install
 install: build
@@ -30,7 +30,7 @@ build-release: build
 	cp $(builddir)/yr* yr-$(VERSION)/
 
 	tar czf yr-$(VERSION)-$(GOOS)-$(ARCH).tar.gz yr-$(VERSION)
-	hut git artifact upload --rev $(tag) yr-$(VERSION)-$(GOOS)-$(ARCH).tar.gz
+	hut git artifact upload --rev $(TAG) yr-$(VERSION)-$(GOOS)-$(ARCH).tar.gz
 
 .PHONY: test
 test:

cmd/forecast.go

@@ -21,7 +21,7 @@ var (
 
 func init() {
 	rootCmd.AddCommand(forecastCmd)
-	rootCmd.Flags().IntVarP(&interval, "interval", "i", 0, "Number of hours to output")
+	forecastCmd.Flags().IntVarP(&interval, "interval", "i", 0, "Number of hours to output")
 }
 
 func forecast(cmd *cobra.Command, args []string) {
@@ -39,12 +39,13 @@ func forecast(cmd *cobra.Command, args []string) {
 		outputJson(f.Forecast)
 	}
 
-	t := table.New()
+	t := table.New("time", "temp.", "rain", "wind")
 
 	for _, item := range f.Forecast {
 		if !isUTC {
 			item.Time = item.Time.Local()
 		}
+
 		t.Row(
 			item.Time.Format("Mon, 2 Jan 15:04"),
 			fmt.Sprintf("%.1f °C", item.Temperature),

cmd/internal/ui/table/table.go

@@ -7,8 +7,7 @@ import (
 	"github.com/charmbracelet/lipgloss/table"
 )
 
-// TODO: Make more generic
-func New() *table.Table {
+func New(headers ...string) *table.Table {
 	re := lipgloss.NewRenderer(os.Stdout)
 
 	const (
@@ -16,23 +15,41 @@ func New() *table.Table {
 		lightGray = lipgloss.Color("#dedede")
 	)
 	var (
-		HeaderStyle  = re.NewStyle().Foreground(white).Bold(true).Align(lipgloss.Center)
-		CellStyle    = re.NewStyle().Padding(0, 2)
+		HeaderStyle = re.
+				NewStyle().
+				Foreground(white).
+				Bold(true).
+				Align(lipgloss.Center)
+
+		CellStyle = re.NewStyle().
+				Padding(0, 2).
+				AlignHorizontal(lipgloss.Right)
+
 		EvenRowStyle = CellStyle.Foreground(lightGray)
 	)
 
 	t := table.New().
 		Border(lipgloss.RoundedBorder()).
 		StyleFunc(func(row, col int) lipgloss.Style {
-			switch {
-			case row == 0:
+			if row == 0 {
 				return HeaderStyle
-			case row%2 == 0:
-				return EvenRowStyle
-			default:
-				return CellStyle
 			}
+
+			var style lipgloss.Style
+			switch {
+			case row%2 == 0:
+				style = EvenRowStyle
+			default:
+				style = CellStyle
+			}
+
+			// right align numeric values for better readability
+			if col > 0 && row > 1 {
+				style.AlignHorizontal(lipgloss.Right)
+			}
+
+			return style
 		}).
-		Headers("time", "temp.", "rain", "wind")
+		Headers(headers...)
 	return t
 }

cmd/root.go

@@ -9,7 +9,7 @@ import (
 var rootCmd = &cobra.Command{
 	Use:     "yr",
 	Short:   "yr is CLI tool for weather forecasts",
-	Version: "v0.0.5",
+	Version: "v0.0.6",
 }
 
 func Execute() {

cmd/today.go

@@ -30,7 +30,7 @@ func today(cmd *cobra.Command, args []string) {
 		outputJson(f.Forecast)
 	}
 
-	t := table.New()
+	t := table.New("time", "temp.", "rain", "wind")
 
 	today := time.Now()
 	for _, item := range f.Forecast {

cmd/tomorrow.go

@@ -36,7 +36,7 @@ func tomorrow(cmd *cobra.Command, args []string) {
 		os.Exit(0)
 	}
 
-	t := table.New()
+	t := table.New("time", "temp.", "rain", "wind")
 
 	tomorrows := []yr.Forecast{}
 

pkg/cache/cache.go

@@ -23,22 +23,32 @@ func New[T any](path string) *Cache[T] {
 	return &Cache[T]{path}
 }
 
+var (
+	ErrorUnableToMarshalJSON        = errors.New("unable to marshal as JSON")
+	ErrorUnableToEnsureCacheDir     = errors.New("unable to ensure cache-dir is present")
+	ErrorUnableToWriteCache         = errors.New("unable to write data to cache file")
+	ErrorUnableToOpenCacheFile      = errors.New("unable to open cache")
+	ErrorUnableToReadCacheFile      = errors.New("unable to read cache")
+	ErrorUnableToUnmarshalCacheFile = errors.New("unable to unmarshal JSON from cache-file")
+	ErrorUnableToReadCachePath      = errors.New("unable to read from cache path")
+)
+
 // Add data as JSON-file into cache-dir, returns nil if success
 func (c *Cache[T]) Add(name string, data any) error {
 	j, err := json.MarshalIndent(data, "", " ")
 	if err != nil {
-		return fmt.Errorf("Failed to convert data as JSON.")
+		return ErrorUnableToMarshalJSON
 	}
 	err = os.MkdirAll(c.Path, 0646)
 	if err != nil {
-		return fmt.Errorf("Failed to ensure cache-dir is present.")
+		return ErrorUnableToEnsureCacheDir
 	}
 
 	filePath := filepath.Join(c.Path, filename(name))
 	d := []byte(j)
 	err = os.WriteFile(filePath, d, 0646)
 	if err != nil {
-		return fmt.Errorf("Failed write data to cache-file.")
+		return ErrorUnableToWriteCache
 	}
 
 	return nil
@@ -53,19 +63,19 @@ func (c *Cache[T]) Get(name string) (*T, error) {
 
 	rawFile, err := os.Open(filePath)
 	if err != nil {
-		return nil, fmt.Errorf("Failed to read cache-file.")
+		return nil, ErrorUnableToOpenCacheFile
 	}
 
 	var result T
 
 	bytes, err := io.ReadAll(rawFile)
 	if err != nil {
-		return nil, fmt.Errorf("Failed to read data from cache-file.")
+		return nil, ErrorUnableToReadCacheFile
 	}
 
 	err = json.Unmarshal(bytes, &result)
 	if err != nil {
-		return nil, fmt.Errorf("Failed to parse JSON from cache-file.")
+		return nil, ErrorUnableToUnmarshalCacheFile
 	}
 
 	return &result, nil
@@ -80,7 +90,7 @@ func (c *Cache[T]) List() ([]Saved[T], error) {
 	var result []Saved[T]
 	filenames, err := list(c.Path)
 	if err != nil {
-		return nil, fmt.Errorf("Unable to read from cache path")
+		return nil, ErrorUnableToReadCachePath
 	}
 
 	if len(filenames) == 0 {
@@ -102,7 +112,7 @@ func (c *Cache[T]) List() ([]Saved[T], error) {
 func list(path string) ([]string, error) {
 	dir, err := os.ReadDir(path)
 	if err != nil {
-		return nil, fmt.Errorf("Unable to read from path")
+		return nil, ErrorUnableToReadCachePath
 	}
 	var filenames []string
 
@@ -145,5 +155,5 @@ func IsExpired(expire time.Time, now *time.Time) bool {
 		now = &tmp
 	}
 
-	return now.Unix() >= expire.Unix()
+	return now.After(expire)
 }

pkg/cache/cache_test.go

@@ -55,7 +55,7 @@ func TestIsExpired(t *testing.T) {
 	assert.NoError(err)
 
 	result = IsExpired(expirationTime, &now)
-	assert.True(result)
+	assert.False(result)
 
 	now, err = time.Parse(layout, "2024-10-04 22:23:00")
 	assert.NoError(err)

pkg/met/met.go

@@ -3,6 +3,7 @@ package met
 
 import (
 	"encoding/json"
+	"errors"
 	"fmt"
 	"io"
 	"net/http"
@@ -13,10 +14,18 @@ type Met struct {
 	siteName string
 }
 
+var (
+	ErrorMissingSiteName           = errors.New("siteName must be defined")
+	ErrorMissingHeaderExpires      = errors.New("missing header 'Expires'")
+	ErrorMissingHeaderLastModified = errors.New("missing header 'Last-Modified'")
+	ErrorParseExpires              = errors.New("unable to parse 'Expires'")
+	ErrorParseLastModified         = errors.New("unable to parse 'Last-Modified'")
+)
+
 // Returns valid Met client or error if empty siteName is provided
 func New(siteName string) (*Met, error) {
 	if siteName == "" {
-		return nil, fmt.Errorf("`siteName` must be defined.")
+		return nil, ErrorMissingSiteName
 	}
 
 	return &Met{
@@ -60,19 +69,19 @@ func (m *Met) Forecast(lat, lon float64, alt *int) (*LocationForecastResult, err
 
 	expiresRaw, ok := resp.Header["Expires"]
 	if !ok {
-		return nil, fmt.Errorf("Unable to get 'Expires' header")
+		return nil, ErrorMissingHeaderExpires
 	}
 	expires, err := time.Parse(time.RFC1123, expiresRaw[0])
 	if err != nil {
-		return nil, fmt.Errorf("Unable to parse `expiresRaw`: %w", err)
+		return nil, errors.Join(ErrorParseExpires, err)
 	}
 	lastModifiedRaw, ok := resp.Header["Last-Modified"]
 	if !ok {
-		return nil, fmt.Errorf("Unable to get 'Last-Modified' header")
+		return nil, ErrorMissingHeaderLastModified
 	}
 	lastModified, err := time.Parse(time.RFC1123, lastModifiedRaw[0])
 	if err != nil {
-		return nil, fmt.Errorf("Unable to parse `Last-Modified`: %w", err)
+		return nil, errors.Join(ErrorParseLastModified, err)
 	}
 
 	forecast := LocationForecast{}

pkg/nominatim/nominatim.go

@@ -2,6 +2,7 @@ package nominatim
 
 import (
 	"encoding/json"
+	"errors"
 	"fmt"
 	"io"
 	"net/http"
@@ -14,9 +15,13 @@ type Nominatim struct {
 	siteName string
 }
 
+var (
+	ErrorMissingSiteName = errors.New("siteName must be defined")
+)
+
 func New(siteName string) (*Nominatim, error) {
 	if siteName == "" {
-		return nil, fmt.Errorf("`siteName` must be defined.")
+		return nil, ErrorMissingSiteName
 	}
 
 	return &Nominatim{
@@ -94,7 +99,7 @@ func (n *Nominatim) Reverse(lat, lon float64) (*ReverseResult, error) {
 	}
 
 	if allFieldsNil(result) {
-		return nil, fmt.Errorf("No result")
+		return nil, fmt.Errorf("no result")
 	}
 
 	return &result, nil

yr/yr.go

@@ -165,7 +165,7 @@ func (c *Client) ForecastCoords(coords *nominatim.Coordinates, location *string)
 	}
 
 	if location == nil {
-		return nil, fmt.Errorf("Location name is not set")
+		return nil, fmt.Errorf("location name is not set")
 	}
 
 	ts := f.Properties.Timeseries