Merge pull request #959 from appwrite/feat-removing-vars

feat(cli): pulling variable to .env only
appwrite · Aug 27, 2024 · 6ee8749 · 6ee8749
2 parents 9165e69 + dff294d
commit 6ee8749
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/templates/cli/lib/commands/pull.js.twig b/templates/cli/lib/commands/pull.js.twig
@@ -88,9 +88,9 @@ const pullFunctions = async ({ code, withVariables }) => {
         if (!localFunction['path']) {
             func['path'] = `functions/${func.name}`;
         }
-        if (!withVariables) {
-            delete func['vars'];
-        }
+        const holdingVars = func['vars'];
+        // We don't save var in to the config
+        delete func['vars'];
         localConfig.addFunction(func);
 
         if (!fs.existsSync(func['path'])) {
@@ -162,7 +162,7 @@ const pullFunctions = async ({ code, withVariables }) => {
             } catch {
             }
 
-            fs.writeFileSync(envFileLocation, func['vars'].map(r => `${r.key}=${r.value}\n`).join(''))
+            fs.writeFileSync(envFileLocation, holdingVars.map(r => `${r.key}=${r.value}\n`).join(''))
         }
     }