Skip to content

Commit

Permalink
child_process: add env contents types in JSDoc
Browse files Browse the repository at this point in the history
Use JSDoc to indicate that the `env` object keys and values must be
strings.

Refs: nodejs#42489 (comment)

PR-URL: nodejs#42494
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
  • Loading branch information
Trott authored and xtx1130 committed Apr 25, 2022
1 parent 68b8b01 commit a925f31
Showing 1 changed file with 7 additions and 7 deletions.
14 changes: 7 additions & 7 deletions lib/child_process.js
Original file line number Diff line number Diff line change
Expand Up @@ -98,7 +98,7 @@ const MAX_BUFFER = 1024 * 1024;
* @param {{
* cwd?: string;
* detached?: boolean;
* env?: object;
* env?: Record<string, string>;
* execPath?: string;
* execArgv?: string[];
* gid?: number;
Expand Down Expand Up @@ -199,7 +199,7 @@ function normalizeExecArgs(command, options, callback) {
* @param {string} command
* @param {{
* cmd?: string;
* env?: object;
* env?: Record<string, string>;
* encoding?: string;
* shell?: string;
* signal?: AbortSignal;
Expand Down Expand Up @@ -253,7 +253,7 @@ ObjectDefineProperty(exec, promisify.custom, {
* @param {string[]} [args]
* @param {{
* cwd?: string;
* env?: object;
* env?: Record<string, string>;
* encoding?: string;
* timeout?: number;
* maxBuffer?: number;
Expand Down Expand Up @@ -662,7 +662,7 @@ function abortChildProcess(child, killSignal) {
* @param {string[]} [args]
* @param {{
* cwd?: string;
* env?: object;
* env?: Record<string, string>;
* argv0?: string;
* stdio?: Array | string;
* detached?: boolean;
Expand Down Expand Up @@ -735,7 +735,7 @@ function spawn(file, args, options) {
* input?: string | Buffer | TypedArray | DataView;
* argv0?: string;
* stdio?: string | Array;
* env?: object;
* env?: Record<string, string>;
* uid?: number;
* gid?: number;
* timeout?: number;
Expand Down Expand Up @@ -827,7 +827,7 @@ function checkExecSyncError(ret, args, cmd) {
* cwd?: string;
* input?: string | Buffer | TypedArray | DataView;
* stdio?: string | Array;
* env?: object;
* env?: Record<string, string>;
* uid?: number;
* gid?: number;
* timeout?: number;
Expand Down Expand Up @@ -864,7 +864,7 @@ function execFileSync(command, args, options) {
* cwd?: string;
* input?: string | Buffer | TypedArray | DataView;
* stdio?: string | Array;
* env?: object;
* env?: Record<string, string>;
* shell?: string;
* uid?: number;
* gid?: number;
Expand Down

0 comments on commit a925f31

Please sign in to comment.