Merge pull request #9411 from drewnoakes/fix-warning-and-net8-code

Fix two compilation warnings in the solution

Author: drewnoakes

src/Microsoft.VisualStudio.ProjectSystem.Managed/ProjectSystem/Tree/Dependencies/Subscriptions/MSBuildDependencies/MSBuildDependencyFactoryBase.cs

@@ -179,6 +179,7 @@ protected internal virtual ProjectTreeFlags UpdateTreeFlags(string id, ProjectTr
     /// Gets the <see cref="DiagnosticLevel"/> to set for a dependency, based on its resolved state and item properties.
     /// </summary>
     /// <param name="isResolved"><see langword="true"/> if the dependency is resolved, <see langword="false"/> if it is unresolved, or <see langword="null"/> if the status is not yet determined.</param>
+    /// <param name="hasBuildError">Whether the MSBuild invocation that produced this data also encountered an error or not.</param>
     /// <param name="properties">The properties of the item.</param>
     /// <param name="defaultLevel">The diagnostic level to use when the property is either missing or empty. Intended to receive a dependency's current diagnostic level when an evaluation-only update is being processed.</param>
     /// <returns></returns>

tests/Common/ThrowingTraceListener.cs

@@ -32,7 +32,9 @@ public override void WriteLine(string message)
         {
         }
 
+#if !NET8_0_OR_GREATER
         [Serializable]
+#endif
         public class DebugAssertFailureException : Exception
         {
             public DebugAssertFailureException()
@@ -47,9 +49,11 @@ public DebugAssertFailureException(string message, Exception inner) : base(messa
             {
             }
 
+#if !NET8_0_OR_GREATER
             protected DebugAssertFailureException(SerializationInfo info, StreamingContext context) : base(info, context)
             {
             }
+#endif
         }
     }
 }