Fix #620 (#624)

* Fix #620 * Update Changelog.md
microsoft · Jul 12, 2024 · ba3d17a · ba3d17a
1 parent f3bfd4d
commit ba3d17a
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 2 deletions.
diff --git a/Changelog.md b/Changelog.md
@@ -4,7 +4,11 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.1.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
-## [1.0.39] - 20224-6-26
+## [1.0.40] - 2024-7-08
+## Fix
+Fixes extraneous printing of git errors when git ignore checking is enabled during analysis.
+
+## [1.0.39] - 2024-6-26
 ### Pipelines
 Pipeline maintenance.
 

diff --git a/DevSkim-DotNet/Microsoft.DevSkim.CLI/Commands/AnalyzeCommand.cs b/DevSkim-DotNet/Microsoft.DevSkim.CLI/Commands/AnalyzeCommand.cs
@@ -267,7 +267,8 @@ private static bool IsGitIgnored(string fp)
             {
                 Arguments = $"check-ignore {fp}",
                 WorkingDirectory = Directory.GetParent(fp)?.FullName,
-                RedirectStandardOutput = true
+                RedirectStandardOutput = true,
+                RedirectStandardError = true // Suppress git errors being printed - particularly when not in a git work tree
             });
             process?.WaitForExit();
             string? stdOut = process?.StandardOutput.ReadToEnd();