Clean up error handling.

bar/run.go

@@ -30,24 +30,7 @@ func handleCommands() {
 func Run(cmd *cobra.Command, args []string) {
 	viper.BindPFlags(cmd.Flags())
 	cfg, err := config.ReadConfig(viper.GetString("config"))
-	if err != nil {
-		// TODO Make a better default
-		cfg, err = config.ReadConfigStr(`---
-refresh: 5
-modules:
-  - module: text
-    name: post
-    params:
-      text: "post"
-      color: "#11ff11"
-    on-click: xdg-open https://mastodon.ie/
-  - module: date
-    params:
-      format: 06-05-04 15:02
-    on-click: xdg-open https://calendar.google.com/
-`)
 	cobra.CheckErr(err)
-	}
 	if cfg.Refresh < 1 {
 		cfg.Refresh = 5
 	}

modules/modules.go

 package modules
 
 import (
+	"fmt"
+
 	"gopkg.in/yaml.v3"
 )
 
@@ -41,7 +43,7 @@ func (m *Module) UnmarshalYAML(node *yaml.Node) error {
 	case "text":
 		m.Params = NewText(m)
 	default:
-		panic("module unknown")
+		return fmt.Errorf("module '%s' is unknown", params.Module)
 	}
 	return params.Params.Decode(m.Params)
 }