Move commands out of internal package and fix my bad formatting with …

_example/main.go

@@ -17,7 +17,7 @@ import (
 	"github.com/justanotherorganization/l5424/x5424"
 
 	// Import the commands we want to be registered.
-	_ "github.com/justanotherorganization/graceless/internal/commands/sed"
+	_ "github.com/justanotherorganization/graceless/commands/sed"
 )
 
 var (
@@ -49,7 +49,7 @@ func init() {
 	}
 
 	if slackToken == "" {
-		logger.Log(x5424.Severity, l5424.EmergencyLvl, "Slack token must be set")
+		logger.Log(x5424.Severity, l5424.EmergencyLvl, "Slack token must be set\n")
 		os.Exit(1)
 	}
 
@@ -108,13 +108,12 @@ func main() {
 			case <-ctx.Done():
 				return
 			case <-signals:
-				logger.Log(x5424.Severity, l5424.InfoLvl, "Exiting...")
 				cancel()
 				return
 			case err := <-errCh:
 				if err != nil {
 					// For now treat all errors as non-fatal.
-					logger.Log(x5424.Severity, l5424.ErrorLvl, err)
+					logger.Log(x5424.Severity, l5424.ErrorLvl, err, "\n")
 				}
 			}
 		}

internal/commands/sed/sed.go → commands/sed/sed.go

@@ -7,7 +7,7 @@ import (
 	"os/exec"
 	"strings"
 
-	cmds "github.com/justanotherorganization/graceless/internal/commands"
+	cmds "github.com/justanotherorganization/graceless/commands"
 	"github.com/justanotherorganization/justanotherbotkit/commands"
 	"github.com/justanotherorganization/justanotherbotkit/transport"
 	"github.com/pkg/errors"

graceless.go

@@ -4,9 +4,9 @@ import (
 	"context"
 	"strings"
 
+	cmds "github.com/justanotherorganization/graceless/commands"
 	"github.com/justanotherorganization/graceless/config"
 	"github.com/justanotherorganization/graceless/internal"
-	cmds "github.com/justanotherorganization/graceless/internal/commands"
 	"github.com/justanotherorganization/justanotherbotkit/commands"
 	"github.com/justanotherorganization/justanotherbotkit/proto"
 	"github.com/justanotherorganization/justanotherbotkit/transport"
@@ -41,7 +41,7 @@ func (g *Graceless) Start(ctx context.Context, cancel context.CancelFunc, errCh
 		startMsg += " in safemode"
 	}
 
-	g.config.Log(x5424.Severity, l5424.InfoLvl, startMsg)
+	g.config.Log(x5424.Severity, l5424.InfoLvl, startMsg, "\n")
 
 	// Start building our database (as early as possible).
 	if !g.Safemode() {
@@ -212,8 +212,8 @@ func (g *Graceless) Start(ctx context.Context, cancel context.CancelFunc, errCh
 		}
 	}
 
-	g.config.Log(x5424.Severity, l5424.InfoLvl, "Shutting down...")
-	g.config.Log(x5424.Severity, l5424.InfoLvl, "Goodbye")
+	g.config.Log(x5424.Severity, l5424.InfoLvl, "Shutting down...\n")
+	g.config.Log(x5424.Severity, l5424.InfoLvl, "Goodbye\n")
 }
 
 // Safemode returns whather we are currently running in safemode.

internal/hello.go

@@ -4,8 +4,8 @@ import (
 	"fmt"
 	"strings"
 
+	"github.com/justanotherorganization/graceless/commands"
 	"github.com/justanotherorganization/graceless/config"
-	"github.com/justanotherorganization/graceless/internal/commands"
 	"github.com/justanotherorganization/justanotherbotkit/transport"
 	"github.com/pkg/errors"
 )