Merge pull request #1463 from rirze/fix-rmdir-depreciation-warning

chore: remove depreciation warning for `fs.rmdir`
hashicorp · Jan 7, 2022 · 3981c69 · 3981c69
2 parents 88e7168 + 565238c
commit 3981c69
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 6 deletions.
diff --git a/packages/cdktf-cli/bin/cmds/helper/synth-stack.ts b/packages/cdktf-cli/bin/cmds/helper/synth-stack.ts
@@ -130,7 +130,7 @@ Command output on stdout:
     );
 
     for (const orphanedDirectory of orphanedDirectories) {
-      fs.rmdirSync(orphanedDirectory, { recursive: true });
+      fs.rmSync(orphanedDirectory, { recursive: true });
     }
 
     return stacks;

diff --git a/packages/cdktf-cli/bin/cmds/ui/models/terraform-cloud.ts b/packages/cdktf-cli/bin/cmds/ui/models/terraform-cloud.ts
@@ -455,10 +455,9 @@ export class TerraformCloud implements Terraform {
 
   private removeLocalTerraformDirectory() {
     try {
-      fs.rmdirSync(
-        path.resolve(this.stack.synthesizedStackPath, ".terraform"),
-        { recursive: true }
-      );
+      fs.rmSync(path.resolve(this.stack.synthesizedStackPath, ".terraform"), {
+        recursive: true,
+      });
     } catch (error) {
       logger.debug(`Could not remove .terraform folder`, error);
     }

diff --git a/packages/cdktf/lib/terraform-asset.ts b/packages/cdktf/lib/terraform-asset.ts
@@ -112,7 +112,7 @@ export class TerraformAsset extends Resource {
     // Cleanup existing assets
     const previousVersionsFolder = path.join(basePath, this.namedFolder);
     if (fs.existsSync(previousVersionsFolder)) {
-      fs.rmdirSync(previousVersionsFolder, { recursive: true });
+      fs.rmSync(previousVersionsFolder, { recursive: true });
     }
 
     const targetPath = path.join(basePath, this.path);