Refactor spawn_process fn to handle spawn error

This commit modifies the spawn_process function to explicitly return a Result type, handling the error of the spawn() method. This change improves the readability of the error handling and makes it more idiomatic in Rust. As a secondary correction, it fixes a typo in a print statement.

.github/workflows/release.yaml

@@ -18,9 +18,19 @@ jobs:
     steps:
       - uses: actions/checkout@master
       - name: Compile and release
+        id: compile
         uses: rust-build/rust-build.action@v1.4.4
         env:
           GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
         with:
           RUSTTARGET: ${{ matrix.target }}
           ARCHIVE_TYPES: ${{ matrix.archive }}
+          UPLOAD_MODE: 'none'
+      - name: Upload artifact
+        uses: actions/upload-artifact@v3
+        with:
+          name: Binary
+          path: |
+            ${{ steps.compile.outputs.BUILT_ARCHIVE }}
+            ${{ steps.compile.outputs.BUILT_CHECKSUM }}
+

Cargo.toml

@@ -1,6 +1,6 @@
 [package]
 name = "kr"
-version = "1.0.0"
+version = "1.0.1"
 edition = "2021"
 
 # See more keys and their definitions at https://doc.rust-lang.org/cargo/reference/manifest.html

src/main.rs

@@ -84,7 +84,7 @@ fn check_history(retry: &Retry) -> bool {
     return retry.history.len().lt(&usize::from(retry.max_retries))
 }
 
-fn spawn_process(retry: &Retry) -> std::io::Result<Child> {
+fn spawn_process(retry: &Retry) -> Result<Child, std::io::Error> {
     let mut command_parts: Vec<String> = retry
         .command
         .split_whitespace()
@@ -95,7 +95,10 @@ fn spawn_process(retry: &Retry) -> std::io::Result<Child> {
     command_parts.remove(0);
     command.args(command_parts);
 
-    return command.spawn();
+    match command.spawn() {
+        Ok(child) => Ok(child),
+        Err(err) => Err(err),
+    }
 }
 
 fn run_command(retry: &mut Retry) {
@@ -120,7 +123,7 @@ fn run_command(retry: &mut Retry) {
             println!("Restarting...");
             restart(retry);
         } else {
-            print!("The process has crashed more then {} times in the past {}, stop restarting\n", retry.max_retries, retry.restart_name);
+            println!("The process has crashed more than {} times in the past {}, stop restarting\n", retry.max_retries, retry.restart_name);
         }
     }
 }