Fix mamba-org#54

jonashaag · May 12, 2022 · d6bb4b0 · d6bb4b0
1 parent e36c711
commit d6bb4b0
Show file tree

Hide file tree

Showing 2 changed files with 24 additions and 16 deletions.
diff --git a/dist/main/index.js b/dist/main/index.js
diff --git a/index.js b/index.js
@@ -263,20 +263,16 @@ channel_priority: strict
 }
 
 async function installEnvironment (inputs, envFilePath, envYaml) {
+  core.warning([envFilePath, inputs.extraSpecs])
   if (!envFilePath && !inputs.extraSpecs) {
     // Nothing to install
     return
   }
 
   // Determine environment name
   let envName
-  if (inputs.envFile === 'false') {
-    if (inputs.envName) {
-      envName = inputs.envName
-    } else {
-      throw Error("Must provide 'environment-name' for 'environment-file: false'")
-    }
-  } else {
+  if (envFilePath) {
+    // Have environment.yml or .lock file
     if (envYaml) {
       if (inputs.envName) {
         envName = inputs.envName
@@ -287,13 +283,21 @@ async function installEnvironment (inputs, envFilePath, envYaml) {
           throw Error("Must provide 'environment-name' if environment.yml doesn't provide a 'name' attribute")
         }
       }
-    } else { // .lock file
+    } else {
+      // .lock file
       if (inputs.envName) {
         envName = inputs.envName
       } else {
         throw Error("Must provide 'environment-name' for .lock files")
       }
     }
+  } else {
+    // Have extra-specs only
+    if (inputs.envName) {
+      envName = inputs.envName
+    } else {
+      throw Error("Must provide 'environment-name' for 'environment-file: false'")
+    }
   }
 
   // Install env