address lint issues

cmd/commands/todos.go

@@ -97,7 +97,8 @@ var todosCmd = &cobra.Command{
 			handleError(err, s)
 
 		} else {
-			todos.WriteTodos(foundToDos, os.Stdout)
+			err := todos.WriteTodos(foundToDos, os.Stdout)
+			handleError(err, s)
 		}
 
 	},

pkg/comments/comments.go

@@ -116,7 +116,7 @@ func SearchCommit(commit *object.Commit, cb func(*Comment)) error {
 		return err
 	}
 	defer fileIter.Close()
-	fileIter.ForEach(func(file *object.File) error {
+	err = fileIter.ForEach(func(file *object.File) error {
 		if file.Mode.IsFile() {
 			wg.Add(1)
 			go func() {
@@ -138,6 +138,10 @@ func SearchCommit(commit *object.Commit, cb func(*Comment)) error {
 		return nil
 	})
 
+	if err != nil {
+		return err
+	}
+
 	wg.Wait()
 	return nil
 }

pkg/todos/todos.go

@@ -1,14 +1,12 @@
 package todos
 
 import (
-	"bufio"
 	"context"
 	"strings"
 
 	"github.com/augmentable-dev/tickgit/pkg/blame"
 	"github.com/augmentable-dev/tickgit/pkg/comments"
 	"github.com/dustin/go-humanize"
-	"gopkg.in/src-d/go-git.v4/plumbing/object"
 )
 
 // ToDo represents a ToDo item
@@ -100,33 +98,6 @@ func (t ToDos) CountWithCommits() (count int) {
 	return count
 }
 
-func (t *ToDo) existsInCommit(commit *object.Commit) (bool, error) {
-	f, err := commit.File(t.FilePath)
-	if err != nil {
-		if err == object.ErrFileNotFound {
-			return false, nil
-		}
-		return false, err
-	}
-	r, err := f.Reader()
-	if err != nil {
-		return false, err
-	}
-	defer r.Close()
-	s := bufio.NewScanner(r)
-	for s.Scan() {
-		line := s.Text()
-		if strings.Contains(line, t.Comment.String()) {
-			return true, nil
-		}
-	}
-	err = s.Err()
-	if err != nil {
-		return false, err
-	}
-	return false, nil
-}
-
 // FindBlame sets the blame information on each todo in a set of todos
 func (t *ToDos) FindBlame(ctx context.Context, dir string) error {
 	fileMap := make(map[string]ToDos)