Remove support for DEBUG env var in favor of only PINEJS_DEBUG

Change-type: major
balena-io · May 17, 2023 · b2e1c66 · b2e1c66
1 parent c4bbef5
commit b2e1c66
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 6 deletions.
diff --git a/build/browser.ts b/build/browser.ts
@@ -18,7 +18,6 @@ config.plugins = config.plugins.concat(
 	new webpack.DefinePlugin({
 		'process.browser': true,
 		'process.env.CONFIG_LOADER_DISABLED': true,
-		'process.env.DEBUG': true,
 		'process.env.PINEJS_DEBUG': true,
 		'process.env.SBVR_SERVER_ENABLED': true,
 	}),

diff --git a/src/config-loader/env.ts b/src/config-loader/env.ts
@@ -1,12 +1,9 @@
-// TODO-MAJOR: Drop the support for the global `DEBUG` env var
-const { DEBUG: globalDebug, PINEJS_DEBUG } = process.env;
+const { PINEJS_DEBUG } = process.env;
 if (![undefined, '', '0', '1'].includes(PINEJS_DEBUG)) {
 	// TODO-MAJOR: Throw on invalid value
 	console.warn(`Invalid value for PINEJS_DEBUG '${PINEJS_DEBUG}'`);
 }
-// Setting PINEJS_DEBUG to explicitly '0' will disable debug even if global debug is truthy
-export const DEBUG =
-	PINEJS_DEBUG === '1' || (PINEJS_DEBUG !== '0' && !!globalDebug);
+export const DEBUG = PINEJS_DEBUG === '1';
 
 type CacheFnOpts<T extends (...args: any[]) => any> =
 	| {