Small Form eimprovments

Author: basicx-StrgV

src/WGet.NET/Components/Internal/ProcessOutputReader.cs

@@ -3,12 +3,12 @@
 // https://github.com/basicx-StrgV/                 //
 //--------------------------------------------------//
 using System;
-using System.Text;
 using System.Collections.Generic;
-using WGetNET.Models;
-using WGetNET.Helper;
+using System.Text;
 using WGetNET.Builder;
 using WGetNET.Extensions;
+using WGetNET.Helper;
+using WGetNET.Models;
 
 namespace WGetNET.Components.Internal
 {
@@ -165,10 +165,10 @@ private static List<WinGetPackage> CreatePackageListFromOutput(string[] output,
 #endif
                 }
                 else if ((action == PackageAction.SearchBySource || action == PackageAction.InstalledListBySource)
-                    && !string.IsNullOrWhiteSpace(sourceName) && sourceName != null)
+                    && sourceName != null && !string.IsNullOrWhiteSpace(sourceName))
                 {
                     // "sourceName" source name cant't be null here because of the following check "!string.IsNullOrWhiteSpace(sourceName)".
-                    // But .NET Standard 2.0 thinks it knows better (Or I'm stupid). Therefore a second null check comes after it.
+                    // But .NET Standard 2.0 thinks it knows better (Or I'm stupid). Therefore a second null check comes befor it.
                     builder.AddSourceName(sourceName);
                 }
 

src/WGet.NET/XmlDocumentation/WGet.NET.xml

@@ -2,12 +2,12 @@
 // Created by basicx-StrgV                          //
 // https://github.com/basicx-StrgV/                 //
 //--------------------------------------------------//
-using System.IO;
-using Serilog;
 using Nuke.Common;
 using Nuke.Common.IO;
 using Nuke.Common.ProjectModel;
 using Nuke.Common.Tools.DotNet;
+using Serilog;
+using System.IO;
 
 namespace BuildTool
 {
@@ -53,14 +53,14 @@ internal class Build : NukeBuild
             .Executes(() =>
             {
                 Log.Information("Loading the build config");
-                Log.Information("File: {configFile}", _configFile);
+                Log.Information("File: {ConfigFile}", _configFile);
 
                 // Check if config exist
                 if (!File.Exists(_configFile))
                 {
                     if (Version != null)
                     {
-                        Log.Warning("Config file not found. The provided version ({version}) will still be used", Version);
+                        Log.Warning("Config file not found. The provided version ({Version}) will still be used", Version);
                         _workingVersion = Version;
                     }
                     else
@@ -79,7 +79,7 @@ internal class Build : NukeBuild
                 {
                     if (Version != null)
                     {
-                        Log.Warning("Failed to load the config file. The provided version ({version}) will still be used", Version);
+                        Log.Warning("Failed to load the config file. The provided version ({Version}) will still be used", Version);
                         _workingVersion = Version;
                     }
                     else
@@ -98,12 +98,12 @@ internal class Build : NukeBuild
                 {
                     if (Version != null)
                     {
-                        Log.Warning("The version provided by the config file ({configVersion}) is invalid. The provided version ({version}) will still be used", version, Version);
+                        Log.Warning("The version provided by the config file ({ConfigVersion}) is invalid. The provided version ({Version}) will still be used", version, Version);
                         _workingVersion = Version;
                     }
                     else
                     {
-                        Log.Warning("The version provided by the config file ({configVersion}) is invalid. The version \"1.0.0\" will be used as a fallback", version);
+                        Log.Warning("The version provided by the config file ({ConfigVersion}) is invalid. The version \"1.0.0\" will be used as a fallback", version);
                         _workingVersion = "1.0.0";
                     }
 
@@ -152,8 +152,8 @@ internal class Build : NukeBuild
             .Executes(() =>
             {
                 Log.Information("Creating \"WGet.NET\" nuget package");
-                Log.Information("Version: {version}", _workingVersion);
-                Log.Information("Output directory: {dir}", _packages);
+                Log.Information("Version: {Version}", _workingVersion);
+                Log.Information("Output directory: {Dir}", _packages);
             });
 
         Target Restore => _ => _
@@ -263,7 +263,7 @@ internal class Build : NukeBuild
             .Executes(() =>
             {
                 Log.Information("Updating the build config");
-                Log.Information("File: {configFile}", _configFile);
+                Log.Information("File: {ConfigFile}", _configFile);
 
                 if (!File.Exists(_configFile))
                 {

src/build/Build.cs

@@ -4,8 +4,8 @@
 //--------------------------------------------------//
 using Serilog;
 using System;
-using System.IO;
 using System.Diagnostics;
+using System.IO;
 
 namespace BuildTool
 {
@@ -40,13 +40,13 @@ public bool GenerateDocs(string projectVersion)
 
                 if (exitCode != 0)
                 {
-                    Log.Error("The doxygen process failed with the exit code \"{exitCode}\"", exitCode);
+                    Log.Error("The doxygen process failed with the exit code \"{ExitCode}\"", exitCode);
                     return false;
                 }
             }
             catch (Exception e)
             {
-                Log.Error("Exception thrown on docs generation: {ex}", e);
+                Log.Error(e, "Exception thrown on docs generation");
                 return false;
             }