feat: 56 show live changes at commit #56

pkg/cli/actions/commit.go

@@ -100,6 +100,7 @@ func addToStage(raw []string) {
 
 func processFiles(raw []string, unstaged []string, list []string) (newList []string, changesMsg string, defaults []string) {
 	if len(raw) == 0 {
+		loading.Stop()
 		println("No changes to commit ✅")
 		os.Exit(0)
 		return

pkg/git/Commit.go

@@ -14,19 +14,18 @@ import (
 )
 
 func _commit(msg string, body string, ctype string, scope string, ticket string) string {
+	println("\nCommiting files\n")
 	cmd := git_controller.Commit(msg, body, ctype, ticket, scope)
-	out, outErr, err := shell.Out(cmd)
+	out, _, err := shell.OutLive(cmd)
 
 	if err != nil {
 		loading.Stop()
 
 		if strings.Contains(out, "nothing to commit") {
-			fmt.Println(out)
-			fmt.Println("Aborting commit...")
+			fmt.Println("\nAborting commit...")
 
 			os.Exit(1)
 		} else {
-			fmt.Println(outErr)
 			log.Fatal("Commit failed")
 		}
 	}
@@ -62,16 +61,14 @@ func Commit(msg string, body string, ctype string, scope string) {
 	}
 
 	ticket := _checkTicket(git_controller.GetTicketNumber())
-	out := _commit(msg, body, ctype, scope, ticket)
-
 	loading.Stop()
-	fmt.Println(out)
+	_commit(msg, body, ctype, scope, ticket)
+
 }
 
 func CommitWithoutTicket(msg string, body string, ctype string, scope string) {
 	loading.Start("Commiting files ")
-	out := _commit(msg, body, ctype, scope, "")
-
+
 	loading.Stop()
-	fmt.Println(out)
+	_commit(msg, body, ctype, scope, "")
 }

pkg/git/Publish.go

@@ -1,26 +1,20 @@
 package git
 
 import (
-	"fmt"
 	"log"
 
 	git_controller "github.com/Minnek-Digital-Studio/cominnek/controllers/git"
-	"github.com/Minnek-Digital-Studio/cominnek/controllers/loading"
 	"github.com/Minnek-Digital-Studio/cominnek/pkg/shell"
+	"github.com/fatih/color"
 )
 
 func PushPublish() {
-	loading.Start("Pushing to remote ")
+	color.Yellow("\nPushing to remote\n")
 	currentBranch := git_controller.GetCurrentBranch()
 	cmd := git_controller.Publish(currentBranch)
+	_, _, err := shell.OutLive(cmd)
 
-	out, errout, err := shell.Out(cmd)
 	if err != nil {
-		fmt.Println(out)
-		fmt.Println(errout)
-		log.Fatal(errout)
+		log.Fatal("Error pushing to remote")
 	}
-
-	loading.Stop()
-	fmt.Println(out)
 }

pkg/git/Push.go

@@ -1,27 +1,22 @@
 package git
 
 import (
-	"fmt"
 	"log"
 
 	git_controller "github.com/Minnek-Digital-Studio/cominnek/controllers/git"
-	"github.com/Minnek-Digital-Studio/cominnek/controllers/loading"
 	"github.com/Minnek-Digital-Studio/cominnek/pkg/shell"
 )
 
 func _push() {
-	loading.Start("Pushing to remote ")
 	cmd := git_controller.Push()
-	out, errout, err := shell.Out(cmd)
+	_, _, err := shell.OutLive(cmd)
 
 	if err != nil {
-		fmt.Println(out)
-		fmt.Println(errout)
-		log.Fatal(errout)
+		// fmt.Println(out)
+		// fmt.Println(errout)
+		log.Fatal("Error pushing to remote")
 	}
 
-	loading.Stop()
-	fmt.Println(out)
 }
 
 func Push() {

pkg/shell/main.go

@@ -1,14 +1,14 @@
 package shell
 
 import (
-	"bufio"
 	"bytes"
 	"fmt"
+	"io"
 	"log"
+	"os"
 	"os/exec"
 	"runtime"
 
-	"github.com/Minnek-Digital-Studio/cominnek/controllers/loading"
 	"github.com/Minnek-Digital-Studio/cominnek/pkg/events"
 )
 
@@ -35,28 +35,21 @@ func Out(command string) (string, string, error) {
 	return stdout.String(), stderr.String(), err
 }
 
-func OutLive(command string) {
+func OutLive(command string) (string, string, error) {
 	cmd := exec.Command(shellToUse, "-c", command)
 
-	stdout, err := cmd.StdoutPipe()
-	if err != nil {
-		fmt.Println(err)
-	}
+	var stdoutBuf, stderrBuf bytes.Buffer
+	cmd.Stdout = io.MultiWriter(os.Stdout, &stdoutBuf)
+	cmd.Stderr = io.MultiWriter(os.Stderr, &stderrBuf)
+
+	err := cmd.Run()
+	outStr, errStr := stdoutBuf.String(), stderrBuf.String()
 
-	err = cmd.Start()
-	loading.Start("Running command...")
 	if err != nil {
-		fmt.Println(err)
-	}
-
-	scanner := bufio.NewScanner(stdout)
-	loading.Stop()
-	for scanner.Scan() {
-		m := scanner.Text()
-		fmt.Println(m)
+		return outStr, errStr, err
 	}
 
-	cmd.Wait()
+	return outStr, errStr, nil
 }
 
 /*Execute a command and return the output*/