Skip to content
This repository has been archived by the owner on Mar 15, 2024. It is now read-only.

Commit

Permalink
fix: use all cap env variable name
Browse files Browse the repository at this point in the history
  • Loading branch information
lachrist committed Oct 22, 2022
1 parent 32fd562 commit b7f834f
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion components/path/node/win32.mjs
Original file line number Diff line number Diff line change
Expand Up @@ -56,7 +56,7 @@ export const sanitizePathFilename = (filename) => {
};

export const getShell = (env) =>
hasOwn(env, "comspec") ? env.comspec : "cmd.exe";
hasOwn(env, "COMSPEC") ? env.COMSPEC : "cmd.exe";

const ipc = "\\\\.\\pipe\\";

Expand Down
2 changes: 1 addition & 1 deletion components/path/node/win32.test.mjs
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@ import {

assertDeepEqual(getShell({}), "cmd.exe");

assertDeepEqual(getShell({ comspec: "powershell" }), "powershell");
assertDeepEqual(getShell({ COMSPEC: "powershell" }), "powershell");

// fromIpcPath && toIpcPath //

Expand Down

0 comments on commit b7f834f

Please sign in to comment.