Tweaks

Author: FollowTheProcess

internal/colour/colour.go

@@ -29,8 +29,6 @@ func Red(text string) string {
 }
 
 // sprint returns a string with a given colour and the reset code.
-//
-// It handles checking for NO_COLOR and FORCE_COLOR.
 func sprint(code, text string) string {
 	return code + text + codeReset
 }

snapshot_test.go

@@ -162,13 +162,6 @@ func makeSnapshot(t *testing.T, shotter *snapshot.Shotter, content string) {
 
 	path := shotter.Path()
 
-	// If it already exists, no sense recreating it every time
-	_, err := os.Stat(path)
-	exists := err == nil
-	if exists {
-		return
-	}
-
 	// Because subtests insert a '/' i.e. TestSomething/subtest1, we need to make
 	// all directories along that path so find the last dir along the path
 	// and use that in the call to MkDirAll