Error handling

Author: nefarius

Options.cs

@@ -14,7 +14,7 @@ public class Options
             HelpText = "A text file containing one line that gets parsed as the target version.")]
         public string TargetVersionFile { get; set; }
 
-        [Option("target-file",
+        [Option("target-file", Required = true,
             HelpText = "The target file to get parsed and modified.")]
         public string TargetFile { get; set; }
 

TextFileHelper.cs

@@ -1,41 +1,48 @@
-using System.IO;
+using System;
+using System.IO;
 using System.Text;
 using System.Text.RegularExpressions;
 using Microsoft.Build.Construction;
 
-namespace vpatch
+namespace vpatch;
+
+public static class TextFileHelper
 {
-    public static class TextFileHelper
+    public static void RegexReplace(string path, string pattern, string replacement, Encoding encoding)
     {
-        public static void RegexReplace(string path, string pattern, string replacement, Encoding encoding)
-        {
-            string content;
+        string content;
 
-            using (var reader = new StreamReader(path, encoding))
-            {
-                content = reader.ReadToEnd();
-            }
+        using (var reader = new StreamReader(path, encoding))
+        {
+            content = reader.ReadToEnd();
+        }
 
-            content = Regex.Replace(content, pattern, replacement, RegexOptions.Multiline);
+        content = Regex.Replace(content, pattern, replacement, RegexOptions.Multiline);
 
-            File.WriteAllText(path, content, encoding);
-        }
+        File.WriteAllText(path, content, encoding);
+    }
 
-        public static void RegexReplace(string path, string pattern, string replacement)
-        {
-            RegexReplace(path, pattern, replacement, Encoding.UTF8);
-        }
+    public static void RegexReplace(string path, string pattern, string replacement)
+    {
+        RegexReplace(path, pattern, replacement, Encoding.UTF8);
+    }
 
-        public static void PatchProjectFile(string path, string propertyName, string replacement)
-        {
-            var projectRootElement = ProjectRootElement.Open(path);
+    public static void PatchProjectFile(string projectFile, string propertyName, string replacement)
+    {
+        var projectRootElement = ProjectRootElement.Open(projectFile);
 
-            if (projectRootElement is null)
-                throw new IOException("Couldn't open project file for reading.");
+        if (projectRootElement is null)
+            throw new IOException("Couldn't open project file for reading.");
 
+        try
+        {
             projectRootElement.AddProperty(propertyName, replacement);
 
-            projectRootElement.Save();
+            projectRootElement.Save(projectFile);
+        }
+        catch (Exception ex)
+        {
+            throw new Exception("Failed to parse project file.", ex);
         }
     }
 }

vpatch.csproj

@@ -6,13 +6,16 @@
     <ApplicationIcon>favicon.ico</ApplicationIcon>
   </PropertyGroup>
 
-  <PropertyGroup>
+  <PropertyGroup Condition=" '$(Configuration)' == 'Release' ">
     <RuntimeIdentifier>win10-x64</RuntimeIdentifier>
     <PublishSingleFile>true</PublishSingleFile>
     <PublishTrimmed>true</PublishTrimmed>
     <IncludeNativeLibrariesForSelfExtract>true</IncludeNativeLibrariesForSelfExtract>
     <DebugType>None</DebugType>
     <DebugSymbols>false</DebugSymbols>
+  </PropertyGroup>
+
+  <PropertyGroup>
     <Authors>Benjamin Höglinger-Stelzer</Authors>
     <Company>Nefarius Software Solutions e.U.</Company>
     <Copyright>(C) 2020-2022 Benjamin Höglinger-Stelzer</Copyright>