Skip to content

Commit c0e7671

Browse files
committed
feat(#135): adds deprecation warning when using parseFoo options in envVar directive
1 parent 7f13caf commit c0e7671

File tree

1 file changed

+19
-0
lines changed

1 file changed

+19
-0
lines changed

app-config-extensions/src/env-var-directive.ts

Lines changed: 19 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,7 @@ import {
77
defaultAliases,
88
EnvironmentAliases,
99
} from '@app-config/node';
10+
import { logger } from '@app-config/logging';
1011

1112
/** Substitutes environment variables */
1213
export function envVarDirective(
@@ -39,6 +40,24 @@ export function envVarDirective(
3940
parseFloat = !!(await parse(value.parseFloat)).toJSON();
4041
parseBool = !!(await parse(value.parseBool)).toJSON();
4142
name = resolved;
43+
44+
if (parseInt) {
45+
logger.warn(
46+
`Detected use of deprecated of 'parseInt' option in $envVar - use $parseInt directive instead`,
47+
);
48+
}
49+
50+
if (parseFloat) {
51+
logger.warn(
52+
`Detected use of deprecated of 'parseFloat' option in $envVar - use $parseFloat directive instead`,
53+
);
54+
}
55+
56+
if (parseBool) {
57+
logger.warn(
58+
`Detected use of deprecated of 'parseBool' option in $envVar - use $parseBool directive instead`,
59+
);
60+
}
4261
}
4362

4463
const parseValue = (strValue: string | null) => {

0 commit comments

Comments
 (0)