Merge pull request #115 from atom/boolean-instead-of-number

Use Boolean instead of Number in `SpawnAsAdmin` return values
atom · Feb 7, 2022 · 0dcecb2 · 0dcecb2
2 parents e82121c + e215b58
commit 0dcecb2
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/src/main.cc b/src/main.cc
@@ -77,11 +77,11 @@ Napi::Value SpawnAsAdmin(const Napi::CallbackInfo& info) {
 
   void *child_process = StartChildProcess(command, args, test_mode);
   if (!child_process) {
-    return Napi::Number::New(env, false);
+    return Napi::Boolean::New(env, false);
   } else {
     auto worker = new Worker(info[3].As<Napi::Function>(), child_process, test_mode);
     worker->Queue();
-    return Napi::Number::New(env, true);
+    return Napi::Boolean::New(env, true);
   }
 }