refactor: Return Result<> for getting NS version (#46)

as opposed to using an empty string to indicate an error.

1 files changed, 3 insertions, 6 deletions

diff --git a/src-tauri/src/main.rs b/src-tauri/src/main.rs
index 0210cabb..ad5842f4 100644
--- a/src-tauri/src/main.rs
+++ b/src-tauri/src/main.rs
@@ -146,13 +146,10 @@ async fn get_flightcore_version_number() -> String {
 }
 
 #[tauri::command]
-async fn get_northstar_version_number_caller(game_path: String) -> String {
+async fn get_northstar_version_number_caller(game_path: String) -> Result<String, String> {
     match get_northstar_version_number(game_path) {
-        Ok(version_number) => version_number,
-        Err(err) => {
-            println!("{}", err);
-            "".to_string()
-        }
+        Ok(version_number) => Ok(version_number),
+        Err(err) => Err(err.to_string()),
     }
 }