Fix packaging path and address feedback

Author: captainsafia

src/OpenApi/gen/XmlComments/XmlComment.cs

@@ -474,7 +474,7 @@ private static TNode Copy<TNode>(TNode node, bool copyAttributeAnnotations)
         // Documents can't be added to containers, so our usual copy trick won't work.
         if (node.NodeType == XmlNodeType.Document)
         {
-            copy = new XDocument(((XDocument)(object)node));
+            copy = new XDocument((XDocument)(object)node);
         }
         else
         {

src/OpenApi/sample/Endpoints/MapXmlEndpoints.cs

@@ -99,10 +99,12 @@ public class TodoWithDescription : ITodo
         /// The identifier of the todo, overridden.
         /// </summary>
         public int Id { get; set; }
+
         /// <value>
         /// The name of the todo, overridden.
         /// </value>
         public required string Name { get; set; }
+
         /// <summary>
         /// A description of the the todo.
         /// </summary>

src/OpenApi/src/Microsoft.AspNetCore.OpenApi.csproj

@@ -49,7 +49,10 @@
   <!-- Required to bundle source generator related assets into build. -->
   <ItemGroup>
     <ProjectReference Include="$(RepoRoot)/src/OpenApi/gen/Microsoft.AspNetCore.OpenApi.SourceGenerators.csproj" ReferenceOutputAssembly="false" />
-    <None Include="$(OutputPath)\Microsoft.AspNetCore.OpenApi.SourceGenerators.dll" Pack="true" PackagePath="analyzers/dotnet/cs" Visible="false" />
+    <None Include="$(ArtifactsBinDir)\Microsoft.AspNetCore.OpenApi.SourceGenerators\$(Configuration)\netstandard2.0\Microsoft.AspNetCore.OpenApi.SourceGenerators.dll"
+      Pack="true"
+      PackagePath="analyzers/dotnet/cs"
+      Visible="false" />
     <None Include="..\build\Microsoft.AspNetCore.OpenApi.targets" Pack="true" PackagePath="build" Visible="false" />
   </ItemGroup>
 

src/OpenApi/test/Microsoft.AspNetCore.OpenApi.SourceGenerators.Tests/Helpers/HostFactoryResolver.cs

@@ -65,25 +65,6 @@ public static Func<string[], object> ResolveHostFactory(Assembly assembly,
             return null;
         }
 
-        try
-        {
-            // Attempt to load hosting and check the version to make sure the events
-            // even have a chance of firing (they were added in .NET >= 6)
-            var hostingAssembly = Assembly.Load("Microsoft.Extensions.Hosting");
-            if (hostingAssembly.GetName().Version is Version version && version.Major < 6)
-            {
-                return null;
-            }
-
-            // We're using a version >= 6 so the events can fire. If they don't fire
-            // then it's because the application isn't using the hosting APIs
-        }
-        catch
-        {
-            // There was an error loading the extensions assembly, return null.
-            return null;
-        }
-
         return args => new HostingListener(args, assembly.EntryPoint, waitTimeout == default ? s_defaultWaitTimeout : waitTimeout, stopApplication, configureHostBuilder, entrypointCompleted).CreateHost();
     }