fix: fixed so we cleanup specific files for projects

strapi-community · Sep 14, 2022 · 0983455 · 0983455
1 parent 4e586ed
commit 0983455
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 4 deletions.
diff --git a/cli/commands/reset/reset.js b/cli/commands/reset/reset.js
@@ -9,8 +9,6 @@ const reset = async () => {
   const projectType = await detect.projectType();
 
   const detectedProviderName = await detect.provider();
-  console.log(detectedProviderName);
-
   const { environments, provider } = await askResetQuestions(
     detectedProviderName
   );

diff --git a/providers/heroku/heroku.js b/providers/heroku/heroku.js
@@ -5,6 +5,7 @@ const outputs = require(`../../cli/outputs`);
 const { generateHerokuServices } = require(`./generateHerokuServices`);
 const { destroyHerokuApp } = require(`./destroyHerokuApp`);
 const { herokuAuthenticate, netrcExists } = require(`./authentication`);
+const { file } = require(`../../core`);
 
 const herokuSetup = async ({ config, herokuConfig }) => {
   outputs.info(`Generating heroku configuration file`);
@@ -34,12 +35,15 @@ module.exports = {
       if (!netrcExists()) await herokuAuthenticate();
     },
     async build({ config, providerConfig }) {
-      await herokuSetup({ config, herokuConfig: providerConfig });
+      if (config.useDocker) {
+        await herokuSetup({ config, herokuConfig: providerConfig });
+      }
     },
     async postbuild({ config, providerConfig }) {
       await generateHerokuServices({ config, herokuConfig: providerConfig });
     },
     async destroy({ config, providerConfig }) {
+      await file.remove(providerConfig.outputFileName);
       await destroyHerokuApp({ config, herokuConfig: providerConfig });
     }
   }

diff --git a/providers/render/render.js b/providers/render/render.js
@@ -1,8 +1,8 @@
 const chalk = require(`chalk`);
 const { spinner } = require(`../../utils`);
-const { renderQuestions } = require(`../../core`);
 const outputs = require(`../../cli/outputs`);
 const { generateRenderTemplate } = require(`./renderFile`);
+const { file } = require(`../../core`);
 
 const renderSetup = async ({ config, renderConfig }) => {
   outputs.info(`Generating render configuration file`);
@@ -35,6 +35,9 @@ module.exports = {
   renderHooks: {
     async build({ config, providerConfig }) {
       await renderSetup({ config, renderConfig: providerConfig });
+    },
+    async destroy({ config, providerConfig }) {
+      await file.remove(providerConfig.outputFileName);
     }
   }
 };