From 9aa9bd94d8cbfc8d52f5f34dbdb2c4a2338034a0 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Jo=C3=ABl=20Stemmer?= Date: Sun, 13 Mar 2022 15:07:50 +0000 Subject: [PATCH] gtr: Remove unnecessary TODO There's no need (yet) to check if output is potentially a build error, as build errors are currently handled separately by the parser/gotest package. --- pkg/gtr/builder.go | 1 - 1 file changed, 1 deletion(-) diff --git a/pkg/gtr/builder.go b/pkg/gtr/builder.go index 142bf26..66a1448 100644 --- a/pkg/gtr/builder.go +++ b/pkg/gtr/builder.go @@ -218,7 +218,6 @@ func (b *ReportBuilder) Coverage(pct float64, packages []string) { // AppendOutput appends the given line to the currently active context. If no // active context exists, the output is assumed to belong to the package. func (b *ReportBuilder) AppendOutput(line string) { - // TODO(jstemmer): check if output is potentially a build error if b.lastId <= 0 { b.output = append(b.output, line) return