Bind SonarCloud

jerone · Aug 18, 2024 · feb8e62 · feb8e62
1 parent 6796913
commit feb8e62
Show file tree

Hide file tree

Showing 4 changed files with 15 additions and 8 deletions.
diff --git a/.sonarlint/Jvw.DevToys.SemverCalculator.json b/.sonarlint/Jvw.DevToys.SemverCalculator.json
@@ -0,0 +1,4 @@
+{
+  "SonarCloudOrganization": "jerone",
+  "ProjectKey": "jerone_Jvw.DevToys.SemverCalculator"
+}
diff --git a/Jvw.DevToys.SemverCalculator/Components/CheatSheetComponent.cs b/Jvw.DevToys.SemverCalculator/Components/CheatSheetComponent.cs
@@ -7,7 +7,7 @@ namespace Jvw.DevToys.SemverCalculator.Components;
 /// <summary>
 /// Cheat sheet component.
 /// </summary>
-internal class CheatSheetComponent
+internal static class CheatSheetComponent
 {
     /// <summary>
     /// Columns for the cheat sheet.

diff --git a/Jvw.DevToys.SemverCalculator/Gui.cs b/Jvw.DevToys.SemverCalculator/Gui.cs
@@ -1,4 +1,5 @@
 using System.ComponentModel.Composition;
+using System.Diagnostics;
 using System.Net;
 using System.Text.Json;
 using DevToys.Api;
@@ -232,7 +233,7 @@ private List<IUIElement> MatchVersions()
 
         foreach (var version in versions)
         {
-            if (_includePreReleases == false && version.IsPrerelease)
+            if (!_includePreReleases && version.IsPrerelease)
             {
                 continue;
             }
@@ -258,7 +259,7 @@ public void OnDataReceived(string dataTypeName, object? parsedData)
 
     private async Task<PackageJson?> FetchPackage(string packageName)
     {
-        _logger.LogInformation($"Fetching package \"{packageName}\"...");
+        _logger.LogInformation("Fetching package \"{PackageName}\"...", packageName);
         try
         {
             var client = new HttpClient();
@@ -268,7 +269,7 @@ public void OnDataReceived(string dataTypeName, object? parsedData)
             var response = await client.GetAsync($"https://registry.npmjs.org/{packageName}/");
 
             response.EnsureSuccessStatusCode();
-            _logger.LogInformation($"Fetched packages \"{packageName}\".");
+            _logger.LogInformation("Fetched packages \"{PackageName}\".", packageName);
 
             var contentStream = await response.Content.ReadAsStreamAsync();
 
@@ -281,13 +282,15 @@ public void OnDataReceived(string dataTypeName, object? parsedData)
         }
         catch (HttpRequestException e) when (e.StatusCode == HttpStatusCode.NotFound)
         {
-            _logger.LogWarning($"Package \"{packageName}\" not found.");
-            Console.WriteLine(e.Message);
+#pragma warning disable S6667
+            _logger.LogWarning("Package \"{PackageName}\" not found.", packageName);
+#pragma warning restore S6667
+            Debug.WriteLine(e.Message);
             return null;
         }
         catch (Exception e)
         {
-            _logger.LogError(e, $"Failed to fetch package \"{packageName}\".");
+            _logger.LogError(e, "Failed to fetch package \"{PackageName}\".", packageName);
             Console.WriteLine(e.Message);
             return null;
         }

diff --git a/Jvw.DevToys.SemverCalculator/Models/Settings.cs b/Jvw.DevToys.SemverCalculator/Models/Settings.cs
@@ -10,6 +10,6 @@ internal static class Settings
     /// <summary>
     /// When user closes the HTTP agreement info-bar, this setting is set to true.
     /// </summary>
-    public static SettingDefinition<bool> HttpAgreementClosed =
+    public static readonly SettingDefinition<bool> HttpAgreementClosed =
         new(name: "Settings." + nameof(HttpAgreementClosed), defaultValue: false);
 }