Merge pull request #8205 from davidwengier/FixDiagnostics176P1

dotnet · Feb 2, 2023 · 6c36b27 · 6c36b27
2 parents ad380b8 + 787d774
commit 6c36b27
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/....VisualStudio.LanguageServerClient.Razor/DefaultRazorLanguageServerCustomMessageTarget.cs b/....VisualStudio.LanguageServerClient.Razor/DefaultRazorLanguageServerCustomMessageTarget.cs
@@ -1209,7 +1209,9 @@ public override Task<ImplementationResult> ImplementationAsync(DelegatedPosition
         // servers.
         if (response?.Response is null or [{ Diagnostics: null }, ..])
         {
-            return null;
+            // Important that we send back an empty list here, because null would result it the above method throwing away any other
+            // diagnostics it receives from the other delegated server
+            return Array.Empty<VSInternalDiagnosticReport>();
         }
 
         return response.Response;