Wait for linting tasks to start before waiting (#158)

This fails sometimes when the waiting goroutine completes before the linting tasks. In such cases an empty result is returned. Signed-off-by: Charlie Egan <[email protected]>
StyraInc · Jun 8, 2023 · 562d430 · 562d430
1 parent 131bd2c
commit 562d430
Showing 1 changed file with 5 additions and 5 deletions.
diff --git a/pkg/linter/linter.go b/pkg/linter/linter.go
@@ -242,11 +242,6 @@ func (l Linter) lintWithRegoRules(ctx context.Context, input rules.Input) (repor
 	errCh := make(chan error)
 	doneCh := make(chan bool)
 
-	go func() {
-		wg.Wait()
-		doneCh <- true
-	}()
-
 	for _, name := range input.FileNames {
 		wg.Add(1)
 
@@ -280,6 +275,11 @@ func (l Linter) lintWithRegoRules(ctx context.Context, input rules.Input) (repor
 		}(name)
 	}
 
+	go func() {
+		wg.Wait()
+		doneCh <- true
+	}()
+
 	select {
 	case <-ctx.Done():
 		return report.Report{}, fmt.Errorf("context cancelled: %w", ctx.Err())