Don't require IBuildEngine10 in all build engine implementations

dotnet · May 24, 2021 · 600ece5 · 600ece5
1 parent 39e991b
commit 600ece5
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/src/Shared/TaskLoggingHelper.cs b/src/Shared/TaskLoggingHelper.cs
@@ -246,7 +246,8 @@ public virtual string GetResourceMessage(string resourceName)
         /// <returns>True if messages of the given importance should be logged, false if it's guaranteed that such messages would be ignored.</returns>
         public bool ShouldLogMessage(MessageImportance importance)
         {
-            return importance <= ((IBuildEngine10)BuildEngine).EngineInterface.MinimumRequiredMessageImportance;
+            return BuildEngine is not IBuildEngine10 buildEngine10
+                || importance <= buildEngine10.EngineInterface.MinimumRequiredMessageImportance;
         }
 
         /// <summary>