Merge pull request #2 from kaduartur/feature/spinner-windows

[FEATURE] Show title when running on Windows

Author: kaduartur

cmd/main.go

@@ -1,6 +1,7 @@
 package main
 
 import (
+	"errors"
 	"fmt"
 	"time"
 
@@ -16,10 +17,17 @@ func main() {
 	s.Stop()
 
 	fmt.Println()
-	// Run with another template
+	// Run with another template and success message
 	sp := spinner.New("Loading...")
 	sp.SetTemplate(template.Arrow)
 	sp.Start()
 	time.Sleep(time.Second * 2)
-	sp.Stop()
+	sp.Success("Success")
+
+	fmt.Println()
+	// Run with error message
+	spi := spinner.New("Loading...")
+	spi.Start()
+	time.Sleep(time.Second * 2)
+	spi.Error(errors.New("error to load"))
 }

pkg/spinner/spinner.go

@@ -3,6 +3,7 @@ package spinner
 import (
 	"fmt"
 	"io"
+	"runtime"
 	"sync"
 	"syscall"
 	"time"
@@ -56,7 +57,14 @@ func StartNew(title string) *Spinner {
 
 // Start starts the spinner execution
 func (s *Spinner) Start() *Spinner {
+	goos := runtime.GOOS
+	if goos == "windows" { // Show title without animation on Windows
+		fmt.Print(s.Title)
+		return s
+	}
+
 	go s.writer()
+
 	return s
 }
 
@@ -90,13 +98,13 @@ func (s *Spinner) Stop() {
 // Success gives success feedback for users and stops the spinner execution
 func (s *Spinner) Success(msg string) {
 	s.Stop()
-	fmt.Println(msg)
+	fmt.Printf("\r%s\n", msg)
 }
 
 // Error gives error feedback for users and stops the spinner execution
 func (s *Spinner) Error(err error) {
 	s.Stop()
-	fmt.Println(err)
+	fmt.Printf("\r%s\n", err)
 }
 
 // animate runs the template animation