Skip to content
This repository was archived by the owner on Dec 15, 2022. It is now read-only.

Commit 0dcecb2

Browse files
authored
Merge pull request #115 from atom/boolean-instead-of-number
Use Boolean instead of Number in `SpawnAsAdmin` return values
2 parents e82121c + e215b58 commit 0dcecb2

1 file changed

Lines changed: 2 additions & 2 deletions

File tree

src/main.cc

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -77,11 +77,11 @@ Napi::Value SpawnAsAdmin(const Napi::CallbackInfo& info) {
7777

7878
void *child_process = StartChildProcess(command, args, test_mode);
7979
if (!child_process) {
80-
return Napi::Number::New(env, false);
80+
return Napi::Boolean::New(env, false);
8181
} else {
8282
auto worker = new Worker(info[3].As<Napi::Function>(), child_process, test_mode);
8383
worker->Queue();
84-
return Napi::Number::New(env, true);
84+
return Napi::Boolean::New(env, true);
8585
}
8686
}
8787

0 commit comments

Comments
 (0)