Can't error on warning if there is no warning. Checkmate msbuild (#727)

mmitche · web-flow · commit 35674fa72e19 · 2025-05-22T11:17:59.000-07:00
diff --git a/src/arcade/src/Microsoft.DotNet.SignTool/src/Configuration.cs b/src/arcade/src/Microsoft.DotNet.SignTool/src/Configuration.cs
@@ -528,7 +528,7 @@ private FileSignInfo ExtractSignInfo(
                         // Turn the else into a warning (and hoist into the if above) after issue is complete.
                         if (peInfo.IsManaged)
                         {
-                            LogWarning(code, warning);
+                            _log.LogMessage(MessageImportance.High, warning);
                         }
                         else
                         {

Original file line number	Diff line number	Diff line change
`@@ -528,7 +528,7 @@ private FileSignInfo ExtractSignInfo(`
`528`	`528`	`// Turn the else into a warning (and hoist into the if above) after issue is complete.`
`529`	`529`	`if (peInfo.IsManaged)`
`530`	`530`	`{`
`531`		`- LogWarning(code, warning);`
	`531`	`+ _log.LogMessage(MessageImportance.High, warning);`
`532`	`532`	`}`
`533`	`533`	`else`
`534`	`534`	`{`