Fix DeprecationWarning fs.rmdir (take 2) (#9044)

* Use fs.rm only if available Co-authored-by: Fraser Waters <fraser@pulumi.com>
pulumi · Feb 25, 2022 · 05a3eb7 · 05a3eb7
1 parent 94de6e6
commit 05a3eb7
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 2 deletions.
diff --git a/CHANGELOG_PENDING.md b/CHANGELOG_PENDING.md
@@ -1,3 +1,6 @@
 ### Improvements
 
-### Bug Fixes
+### Bug Fixes
+
+- [sdk/nodejs] - Fix Node `fs.rmdir` DeprecationWarning for Node JS 15.X+
+  [#9044](https://github.com/pulumi/pulumi/pull/9044)
diff --git a/sdk/nodejs/automation/stack.ts b/sdk/nodejs/automation/stack.ts
@@ -825,6 +825,12 @@ const cleanUp = async (logFile?: string, rl?: ReadlineResult) => {
     }
     if (logFile) {
         // remove the logfile
-        fs.rmdir(path.dirname(logFile), { recursive: true }, () => { return; });
+        if(fs.rm) {
+            // remove with Node JS 15.X+
+            fs.rm(path.dirname(logFile), { recursive: true }, () => { return; });
+        } else {
+            // remove with Node JS 14.X
+            fs.rmdir(path.dirname(logFile), { recursive: true }, () => { return; });
+        }
     }
 };