Drop usage of Humanizer in descriptors ctor

kzu · kzu · commit 8ce2ffb3e833 · 2024-07-10T11:43:05.000-03:00
In most cases, we won't even need to load humanizer at all, so avoid that early load.
diff --git a/samples/dotnet/SponsorLink/DiagnosticsManager.cs b/samples/dotnet/SponsorLink/DiagnosticsManager.cs
@@ -8,7 +8,6 @@
 using System.IO;
 using System.Linq;
 using Humanizer;
-using Humanizer.Localisation;
 using Microsoft.CodeAnalysis;
 using Microsoft.CodeAnalysis.Diagnostics;
 using static Devlooped.Sponsors.SponsorLink;
@@ -195,7 +194,7 @@ SponsorStatus GetOrSetStatus(Func<ImmutableArray<AdditionalText>> getAdditionalF
         DiagnosticSeverity.Warning,
         isEnabledByDefault: true,
         description: string.Format(CultureInfo.CurrentCulture, Resources.Unknown_Description,
-            sponsorable.Humanize(x => $"https://github.com/sponsors/{x}"),
+            string.Join(", ", sponsorable.Select(x => $"https://github.com/sponsors/{x}")),
             string.Join(" ", sponsorable)),
         helpLinkUri: "https://github.com/devlooped#sponsorlink",
         WellKnownDiagnosticTags.NotConfigurable);
