try #2 on getting rid of deprecation warning

microsoft · Apr 25, 2019 · a012a02 · a012a02
1 parent dafe632
commit a012a02
Showing 1 changed file with 4 additions and 5 deletions.
diff --git a/packages/just-task/src/cli.ts b/packages/just-task/src/cli.ts
@@ -3,19 +3,18 @@ import { JustTaskRegistry } from './JustTaskRegistry';
 import yargs from 'yargs';
 
 const originalEmitWarning = process.emitWarning;
-process.emitWarning = function emitWarning(warning: string | Error, name?: string, ctor?: Function) {
-  if (name === 'DEP0097') {
+
+(process.emitWarning as any) = function emitWarning(this: any, warning: string, type: string, code: string, ctor?: Function) {
+  if (code === 'DEP0097') {
     // Undertaker uses a deprecated approach that causes NodeJS 10 to print
     // this warning to stderr:
     //
     // "Using a domain property in MakeCallback is deprecated. Use the  async_context
     // variant of MakeCallback or the AsyncResource class instead."
-
     // Suppress the warning!
     return;
   }
-
-  return originalEmitWarning(warning, name, ctor);
+  return originalEmitWarning.apply(this, arguments);
 };
 
 yargs