fix: don't output coloured scopes when not writing to a tty

alecthomas · alecthomas · commit ba039ea0f52e · 2023-09-05T14:40:59.000+10:00
diff --git a/README.md b/README.md
@@ -1,5 +1,7 @@
 # Bit - A simple yet powerful build tool [![CI](https://github.com/alecthomas/bit/actions/workflows/ci.yml/badge.svg)](https://github.com/alecthomas/bit/actions/workflows/ci.yml)
 
+
+
 ![Build IT](bit.png)
 
 Bit's goal is to be a simple yet powerful local build system. It is inspired
diff --git a/engine/colours.go b/engine/colours.go
@@ -3,11 +3,17 @@ package engine
 import (
 	"fmt"
 	"hash/fnv"
+	"os"
 	"strconv"
+
+	"github.com/mattn/go-isatty"
 )
 
 var (
 	colours = func() []string {
+		if !isatty.IsTerminal(os.Stdout.Fd()) {
+			return []string{""}
+		}
 		rgb256Colours := []string{"000000", "008000", "808000", "000080", "800080", "008080", "c0c0c0",
 			"00ff00", "ffff00", "0000ff", "ff00ff", "00ffff", "ffffff", "000000", "00005f", "000087", "0000af",
 			"0000d7", "0000ff", "005f00", "005f5f", "005f87", "005faf", "005fd7", "005fff", "008700", "00875f",
diff --git a/engine/internal/glob.go b/engine/internal/glob.go
@@ -26,6 +26,7 @@ func NewGlobber(dir string, extraFiles func() []string) (*Globber, error) {
 	if err != nil && !errors.Is(err, fs.ErrNotExist) {
 		return nil, err
 	}
+	ignore = append(ignore, "**/.*")
 	for i, glob := range ignore {
 		if strings.HasPrefix(glob, "!") {
 			ignore[i] = glob[1:]

Original file line number	Diff line number	Diff line change
`@@ -26,6 +26,7 @@ func NewGlobber(dir string, extraFiles func() []string) (*Globber, error) {`
`26`	`26`	`if err != nil && !errors.Is(err, fs.ErrNotExist) {`
`27`	`27`	`return nil, err`
`28`	`28`	`}`
	`29`	`+ ignore = append(ignore, "*/.")`
`29`	`30`	`for i, glob := range ignore {`
`30`	`31`	`if strings.HasPrefix(glob, "!") {`
`31`	`32`	`ignore[i] = glob[1:]`