Make pragmas take priority over file names (#21)

Co-authored-by: Tushar Sadhwani <[email protected]>
DeepSourceCorp · Mar 13, 2023 · 3588bc1 · 3588bc1
1 parent b234906
commit 3588bc1
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 1 deletion.
diff --git a/cmd/scatr/version.go b/cmd/scatr/version.go
@@ -7,7 +7,7 @@ import (
 	"github.com/spf13/cobra"
 )
 
-const version = "0.4.0"
+const version = "0.4.1"
 
 var versionCmd = &cobra.Command{
 	Use:   "version",

diff --git a/runner/diff.go b/runner/diff.go
@@ -37,6 +37,12 @@ func matchFileNameIssueCodes(files map[string]*pragma.File, analysisResult *Resu
 	}
 
 	for _, file := range files {
+		// If the file already has an ignore/check pragma, that takes priority.
+		// Don't check its file name in that case.
+		if file.CheckMode != pragma.CheckAll {
+			continue
+		}
+
 		_, exists := analysisIssueCodes[file.Name]
 		if !exists {
 			continue