feat(resultdetail): show executor status by default

Makefile

@@ -12,7 +12,6 @@ all: build
 
 build:
 	$(foreach APP,$(APPS), go build -ldflags=$(FLAGS) -o $(BUILD_DIR)/$(APP) ./cmd/$(APP);)
-	cp ./build/repo-health-checker ./build/healthcheck
 
 clean:
 	rm -rf $(BUILD_DIR)/*

internal/parser/resultdetail/parser.go

@@ -7,13 +7,14 @@ import (
 )
 
 type Conf struct {
-	Score          int
+	Score              int
-	ShowExitStatus bool `default:"false"`
+	ShowExecutorStatus bool `default:"true"`
-	ShowError      bool `default:"false"`
+	ShowExitStatus     bool `default:"false"`
-	ShowTime       bool `default:"true"`
+	ShowError          bool `default:"false"`
-	ShowMemory     bool `default:"true"`
+	ShowTime           bool `default:"true"`
-	ShowRunTime    bool `default:"false"`
+	ShowMemory         bool `default:"true"`
-	ShowFiles      []string
+	ShowRunTime        bool `default:"false"`
+	ShowFiles          []string
 }
 
 type ResultDetail struct{}
@@ -29,6 +30,9 @@ func (*ResultDetail) Run(results []stage.ExecutorResult, confAny any) (
 	var res []stage.ParserResult
 	for _, result := range results {
 		comment := ""
+		if conf.ShowExecutorStatus {
+			comment += fmt.Sprintf("Executor Status: `%s`\n", result.Status.String())
+		}
 		if conf.ShowExitStatus {
 			comment += fmt.Sprintf("Exit Status: `%d`\n", result.ExitStatus)
 		}

pkg/healthcheck/nonascii.go

@@ -56,7 +56,7 @@ func getNonAscii(root string) ([]string, error) {
 				return err
 			}
 			ret, matched := matcher.Match(strings.Split(relPath, "/"), nil)
-			if matched && ret["text"].IsUnset() {
+			if matched && ret["text"].IsUnset() && !ret["text"].IsSet() {
 				return nil
 			}
 		}