Removing logging

Author: mschenk42

command/record.go

@@ -75,9 +75,6 @@ func (c RecordCmd) Run(args []string) int {
 	}
 
 	if err := event.Record(fileToRecord); err != nil && !(err == project.ErrNotInitialized || err == project.ErrFileNotFound) {
-		if err := project.Log(err); err != nil {
-			c.Ui.Error(err.Error())
-		}
 		return 1
 	} else if err == nil && status {
 		var (

event/manager.go

@@ -5,15 +5,13 @@
 package event
 
 import (
-	"fmt"
 	"io/ioutil"
 	"os"
 	"path/filepath"
 	"strconv"
 	"strings"
 
 	"github.com/git-time-metric/gtm/epoch"
-	"github.com/git-time-metric/gtm/project"
 )
 
 // Record creates an event for a source
@@ -65,10 +63,7 @@ func Process(gtmPath string, interim bool) (map[int64]map[string]int, error) {
 
 		sourcePath, err := readEventFile(eventFilePath)
 		if err != nil {
-			project.Log(fmt.Sprintf("\nRemoving corrupt event file %s, %s\n", eventFilePath, err))
-			if err := os.Remove(eventFilePath); err != nil {
-				project.Log(fmt.Sprintf("\nError removing event file %s, %s\n", eventFilePath, err))
-			}
+			os.Remove(eventFilePath)
 			continue
 		}
 

main.go

@@ -8,7 +8,6 @@ import (
 	"os"
 
 	"github.com/git-time-metric/gtm/command"
-	"github.com/git-time-metric/gtm/project"
 	"github.com/mitchellh/cli"
 )
 
@@ -64,9 +63,7 @@ func main() {
 
 	exitStatus, err := c.Run()
 	if err != nil {
-		if err := project.Log(err); err != nil {
-			ui.Error(err.Error())
-		}
+		ui.Error(err.Error())
 	}
 
 	os.Exit(exitStatus)

metric/metric.go

@@ -16,7 +16,6 @@ import (
 
 	"github.com/git-time-metric/gtm/epoch"
 	"github.com/git-time-metric/gtm/note"
-	"github.com/git-time-metric/gtm/project"
 	"github.com/git-time-metric/gtm/scm"
 	"github.com/git-time-metric/gtm/util"
 )
@@ -187,10 +186,7 @@ func loadMetrics(gtmPath string) (map[string]FileMetric, error) {
 
 		metricFile, err := readMetricFile(metricFilePath)
 		if err != nil {
-			project.Log(fmt.Sprintf("Removing corrupt metric file %s, %s", metricFilePath, err))
-			if err := os.Remove(metricFilePath); err != nil {
-				project.Log(fmt.Sprintf("Unable to delete corrupt metric file %s, %s", metricFilePath, err))
-			}
+			os.Remove(metricFilePath)
 			continue
 		}
 

project/project.go

@@ -9,7 +9,6 @@ import (
 	"errors"
 	"fmt"
 	"io/ioutil"
-	"log"
 	"os"
 	"path/filepath"
 	"runtime"
@@ -312,23 +311,6 @@ func Paths(wd ...string) (string, string, error) {
 	return repoPath, gtmPath, nil
 }
 
-// Log logs to a gtm log in the GTMDir
-func Log(v ...interface{}) error {
-	_, gtmPath, err := Paths()
-	if err != nil {
-		return err
-	}
-	f, err := os.OpenFile(filepath.Join(gtmPath, "gtm.log"), os.O_RDWR|os.O_CREATE|os.O_APPEND, 0666)
-	if err != nil {
-		return fmt.Errorf("error opening log file: %v", err)
-	}
-	defer func() { _ = f.Close() }()
-	log.SetOutput(f)
-
-	log.Println(v)
-	return nil
-}
-
 func removeTags(gtmPath string) error {
 	files, err := ioutil.ReadDir(gtmPath)
 	if err != nil {

report/query.go

@@ -34,7 +34,6 @@ func retrieveNotes(projects []ProjectCommits, terminalOff bool) commitNoteDetail
 			var commitNote note.CommitNote
 			commitNote, err = note.UnMarshal(n.Note)
 			if err != nil {
-				project.Log(fmt.Sprintf("Error unmarshalling note \n\n%s \n\n%s", n.Note, err))
 				commitNote = note.CommitNote{}
 			}