Skip to content

Commit 50b1cb3

Browse files
thefourtheyeTimothyGu
authored andcommitted
src: fix deprecation id for non-string env value
This is a follow-up of #18990. A new deprecation id was assigned in it, but it was not reflected in code and test. PR-URL: #19209 Reviewed-By: Tiancheng "Timothy" Gu <timothygu99@gmail.com> Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com>
1 parent 5826fe4 commit 50b1cb3

File tree

2 files changed

+2
-2
lines changed

2 files changed

+2
-2
lines changed

src/node.cc

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2667,7 +2667,7 @@ static void EnvSetter(Local<Name> property,
26672667
"Assigning any value other than a string, number, or boolean to a "
26682668
"process.env property is deprecated. Please make sure to convert the "
26692669
"value to a string before setting process.env with it.",
2670-
"DEP00XX").IsNothing())
2670+
"DEP0104").IsNothing())
26712671
return;
26722672
}
26732673
#ifdef __POSIX__

test/parallel/test-process-env-deprecation.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@ common.expectWarning(
99
'Assigning any value other than a string, number, or boolean to a ' +
1010
'process.env property is deprecated. Please make sure to convert the value ' +
1111
'to a string before setting process.env with it.',
12-
'DEP00XX'
12+
'DEP0104'
1313
);
1414

1515
process.env.ABC = undefined;

0 commit comments

Comments
 (0)