Merge branch 'develop' for 2.1.1 release

.github/workflows/example-10.yml

@@ -82,8 +82,24 @@ jobs:
           mamba-version: ${{ matrix.mamba-version }}
           use-mamba: true
           python-version: ${{ matrix.python-version }}
+          clean-patched-environment-file: false
+
       - run: |
           mamba info
           mamba list
           python -VV
           printenv | sort
+      - name: verify unpatched environment.yml is reported and not cleaned
+        if: contains(matrix.environment-file, '.yml') && !matrix.python-version
+        run: |
+          set -eux
+          ls '${{ steps.setup-miniconda.outputs.environment-file }}'
+          ls '${{ matrix.environment-file }}'
+          diff -s '${{ steps.setup-miniconda.outputs.environment-file }}' '${{ matrix.environment-file }}' | grep 'are identical'
+      - name: verify patched output is reported, correct, and not cleaned
+        if: contains(matrix.environment-file, '.yml') && matrix.python-version
+        run: |
+          set -eux
+          if [ "$(diff '${{ steps.setup-miniconda.outputs.environment-file }}' '${{ matrix.environment-file }}' | grep -c 'python=${{ matrix.python-version }}')" -ge 1 ] ; then echo ok ; else exit 1 ; fi
+          python --version | grep "Python ${{ matrix.python-version }}"
+          awk '/- conda-forge/,/- defaults/' '${{ steps.setup-miniconda.outputs.environment-file }}'

CHANGELOG.md

@@ -1,5 +1,26 @@
 # CHANGELOG
 
+## [v2.1.1] (unreleased)
+
+### Features
+
+- [#163] leaves the patched `setup-miniconda-patched-{environment.yml}` in-place
+  if `clean-patched-environment-file: false` is given (otherwise cleans up after
+  itself)
+- [#163] adds action outputs `environment-file`, `environment-file-content` and
+  `environment-file-was-patched`
+
+### Fixes
+
+- [#161] restores proper ordering of `channels` when `environment-file` is
+  patched
+- [#163] if necessary, writes `setup-miniconda-patched-environment.yml` to the
+  same location to work with relative paths, e.g. `pip: ["-r requirements.txt"]`
+
+[v2.1.1]: https://github.com/conda-incubator/setup-miniconda/releases/tag/v2.1.0
+[#161]: https://github.com/conda-incubator/setup-miniconda/pull/161
+[#163]: https://github.com/conda-incubator/setup-miniconda/pull/163
+
 ## [v2.1.0] (2021-03-29)
 
 ### Features

README.md

@@ -16,12 +16,15 @@ A `conda-build-version` or `mamba-version` may be provided to install into
 The base `condabin/` folder is added to `$PATH` and shell integration is
 initialized across all platforms.
 
-By default, this action will then create, and activate an environment by one of:
+By default, this action will then create, and _activate_, an environment by one
+of:
 
 - creating a mostly-empty `test` environment, containing only the latest
   `python-version` and its dependencies
 - creating an `test` environment described in a given `environment-file`:
   - an `environment.yml`-like file (which can be patched with `python-version`)
+    - the patched environment will be cleaned up unless
+      `clean-patched-environment-file: false` is given
   - a [lockfile](#example-7-explicit-specification)
 
 This action correctly handles activation of environments and offers the
@@ -118,9 +121,9 @@ activate the `base` environment. This encourages the practice of not using the
 `base` environment to install packages used for the workflow and leave the
 `base` environment untouched, with only `conda` (and/or `mamba`) in it.
 
-## Inputs
+## Inputs and outputs
 
-For a full list of available inputs for this action see
+For a full list of available _inputs_ and _outputs_ for this action see
 [action.yml](action.yml).
 
 ### Use a different environment name or path

action.yml

@@ -1,6 +1,25 @@
 name: "Setup Miniconda"
 author: Gonzalo Peña-Castellanos (@goanpeca)
 description: "Set up Conda package and environment manager with Miniconda."
+
+runs:
+  using: "node12"
+  main: "dist/setup/index.js"
+  post: "dist/delete/index.js"
+  post-if: "success()"
+
+branding:
+  icon: "code"
+  color: "green"
+
+outputs:
+  environment-file:
+    description: "The full path to the environment-file used"
+  environment-file-content:
+    description: "The content of the environment-file used"
+  environment-file-was-patched:
+    description: "Whether a patched environment-file was made due to inputs"
+
 inputs:
   installer-url:
     description:
@@ -220,11 +239,8 @@ inputs:
       GitHub-hosted runners are "x86" and "x64". Default is "x64".'
     required: false
     default: "x64"
-runs:
-  using: "node12"
-  main: "dist/setup/index.js"
-  post: "dist/delete/index.js"
-  post-if: "success()"
-branding:
-  icon: "code"
-  color: "green"
+  clean-patched-environment-file:
+    description:
+      "Whether a patched environment-file (if created) should be cleaned"
+    required: false
+    default: "true"

dist/delete/index.js

@@ -1161,7 +1161,7 @@ var __importStar = (this && this.__importStar) || function (mod) {
     return result;
 };
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.PYTHON_SPEC = exports.WIN_PERMS_FOLDERS = exports.PROFILES = exports.ENV_VAR_CONDA_PKGS = exports.CONDA_CACHE_FOLDER = exports.CONDARC_PATH = exports.BOOTSTRAP_CONDARC = exports.FORCED_ERRORS = exports.IGNORED_WARNINGS = exports.MAMBA_SUBCOMMANDS = exports.KNOWN_EXTENSIONS = exports.BASE_ENV_NAMES = exports.MINIFORGE_DEFAULT_VERSION = exports.MINIFORGE_DEFAULT_VARIANT = exports.MINIFORGE_URL_PREFIX = exports.OS_NAMES = exports.MINIFORGE_ARCHITECTURES = exports.MINICONDA_ARCHITECTURES = exports.MINICONDA_BASE_URL = exports.IS_UNIX = exports.IS_LINUX = exports.IS_MAC = exports.IS_WINDOWS = exports.MINICONDA_DIR_PATH = void 0;
+exports.OUTPUT_ENV_FILE_WAS_PATCHED = exports.OUTPUT_ENV_FILE_CONTENT = exports.OUTPUT_ENV_FILE_PATH = exports.PYTHON_SPEC = exports.WIN_PERMS_FOLDERS = exports.PROFILES = exports.ENV_VAR_CONDA_PKGS = exports.CONDA_CACHE_FOLDER = exports.CONDARC_PATH = exports.BOOTSTRAP_CONDARC = exports.FORCED_ERRORS = exports.IGNORED_WARNINGS = exports.MAMBA_SUBCOMMANDS = exports.KNOWN_EXTENSIONS = exports.BASE_ENV_NAMES = exports.MINIFORGE_DEFAULT_VERSION = exports.MINIFORGE_DEFAULT_VARIANT = exports.MINIFORGE_URL_PREFIX = exports.OS_NAMES = exports.MINIFORGE_ARCHITECTURES = exports.MINICONDA_ARCHITECTURES = exports.MINICONDA_BASE_URL = exports.IS_UNIX = exports.IS_LINUX = exports.IS_MAC = exports.IS_WINDOWS = exports.MINICONDA_DIR_PATH = void 0;
 const os = __importStar(__webpack_require__(87));
 const path = __importStar(__webpack_require__(622));
 //-----------------------------------------------------------------------
@@ -1288,6 +1288,18 @@ exports.WIN_PERMS_FOLDERS = [
  * @see https://docs.conda.io/projects/conda-build/en/latest/resources/package-spec.html#package-match-specifications
  */
 exports.PYTHON_SPEC = /^(.*::)?python($|\s\=\<\>\!\|)/i;
+/**
+ * Output name for the effective environment-file path used.
+ */
+exports.OUTPUT_ENV_FILE_PATH = "environment-file";
+/**
+ * Output name for the effective environment-file file content used.
+ */
+exports.OUTPUT_ENV_FILE_CONTENT = "environment-file-content";
+/**
+ * Output name for whether the effective environment-file file was patched.
+ */
+exports.OUTPUT_ENV_FILE_WAS_PATCHED = "environment-file-was-patched";
 
 
 /***/ }),

dist/setup/index.js

@@ -9358,7 +9358,7 @@ var __importStar = (this && this.__importStar) || function (mod) {
     return result;
 };
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.PYTHON_SPEC = exports.WIN_PERMS_FOLDERS = exports.PROFILES = exports.ENV_VAR_CONDA_PKGS = exports.CONDA_CACHE_FOLDER = exports.CONDARC_PATH = exports.BOOTSTRAP_CONDARC = exports.FORCED_ERRORS = exports.IGNORED_WARNINGS = exports.MAMBA_SUBCOMMANDS = exports.KNOWN_EXTENSIONS = exports.BASE_ENV_NAMES = exports.MINIFORGE_DEFAULT_VERSION = exports.MINIFORGE_DEFAULT_VARIANT = exports.MINIFORGE_URL_PREFIX = exports.OS_NAMES = exports.MINIFORGE_ARCHITECTURES = exports.MINICONDA_ARCHITECTURES = exports.MINICONDA_BASE_URL = exports.IS_UNIX = exports.IS_LINUX = exports.IS_MAC = exports.IS_WINDOWS = exports.MINICONDA_DIR_PATH = void 0;
+exports.OUTPUT_ENV_FILE_WAS_PATCHED = exports.OUTPUT_ENV_FILE_CONTENT = exports.OUTPUT_ENV_FILE_PATH = exports.PYTHON_SPEC = exports.WIN_PERMS_FOLDERS = exports.PROFILES = exports.ENV_VAR_CONDA_PKGS = exports.CONDA_CACHE_FOLDER = exports.CONDARC_PATH = exports.BOOTSTRAP_CONDARC = exports.FORCED_ERRORS = exports.IGNORED_WARNINGS = exports.MAMBA_SUBCOMMANDS = exports.KNOWN_EXTENSIONS = exports.BASE_ENV_NAMES = exports.MINIFORGE_DEFAULT_VERSION = exports.MINIFORGE_DEFAULT_VARIANT = exports.MINIFORGE_URL_PREFIX = exports.OS_NAMES = exports.MINIFORGE_ARCHITECTURES = exports.MINICONDA_ARCHITECTURES = exports.MINICONDA_BASE_URL = exports.IS_UNIX = exports.IS_LINUX = exports.IS_MAC = exports.IS_WINDOWS = exports.MINICONDA_DIR_PATH = void 0;
 const os = __importStar(__webpack_require__(87));
 const path = __importStar(__webpack_require__(622));
 //-----------------------------------------------------------------------
@@ -9485,6 +9485,18 @@ exports.WIN_PERMS_FOLDERS = [
  * @see https://docs.conda.io/projects/conda-build/en/latest/resources/package-spec.html#package-match-specifications
  */
 exports.PYTHON_SPEC = /^(.*::)?python($|\s\=\<\>\!\|)/i;
+/**
+ * Output name for the effective environment-file path used.
+ */
+exports.OUTPUT_ENV_FILE_PATH = "environment-file";
+/**
+ * Output name for the effective environment-file file content used.
+ */
+exports.OUTPUT_ENV_FILE_CONTENT = "environment-file-content";
+/**
+ * Output name for whether the effective environment-file file was patched.
+ */
+exports.OUTPUT_ENV_FILE_WAS_PATCHED = "environment-file-was-patched";
 
 
 /***/ }),
@@ -13133,7 +13145,8 @@ function applyCondaConfiguration(inputs, options) {
             channels = options.envSpec.yaml.channels;
         }
         // LIFO: reverse order to preserve higher priority as listed in the option
-        for (const channel of channels.reverse()) {
+        // .slice ensures working against a copy
+        for (const channel of channels.slice().reverse()) {
             core.info(`Adding channel '${channel}'`);
             yield condaCommand(["config", "--add", "channels", channel], options);
         }
@@ -13632,6 +13645,7 @@ function parseInputs() {
                 always_yes: "true",
                 changeps1: "false",
             }),
+            cleanPatchedEnvironmentFile: core.getInput("clean-patched-environment-file"),
         });
         const errors = RULES.reduce((errors, rule) => {
             const msg = rule(inputs, inputs.condaConfig);
@@ -15870,6 +15884,7 @@ var __awaiter = (this && this.__awaiter) || function (thisArg, _arguments, P, ge
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.ensureExplicit = void 0;
 const conda = __importStar(__webpack_require__(259));
+const outputs = __importStar(__webpack_require__(405));
 /**
  * Install an environment from an explicit file generated `conda list --explicit`
  * or `conda-lock`
@@ -15878,9 +15893,13 @@ exports.ensureExplicit = {
     label: "conda create (from explicit)",
     provides: (inputs, options) => __awaiter(void 0, void 0, void 0, function* () { var _a, _b; return !!((_b = (_a = options.envSpec) === null || _a === void 0 ? void 0 : _a.explicit) === null || _b === void 0 ? void 0 : _b.length); }),
     condaArgs: (inputs, options) => __awaiter(void 0, void 0, void 0, function* () {
+        var _c;
         if (inputs.pythonVersion) {
             throw Error(`'python-version: ${inputs.pythonVersion}' is incompatible with an explicit 'environmentFile`);
         }
+        if ((_c = options.envSpec) === null || _c === void 0 ? void 0 : _c.explicit) {
+            outputs.setEnvironmentFileOutputs(inputs.environmentFile, options.envSpec.explicit);
+        }
         return [
             "create",
             ...conda.envCommandFlag(inputs),
@@ -18014,7 +18033,7 @@ var __awaiter = (this && this.__awaiter) || function (thisArg, _arguments, P, ge
     });
 };
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.setCacheVariable = exports.setPathVariables = void 0;
+exports.setEnvironmentFileOutputs = exports.setCacheVariable = exports.setPathVariables = void 0;
 /**
  * Modify environment variables and action outputs.
  */
@@ -18050,6 +18069,16 @@ function setCacheVariable(options) {
     });
 }
 exports.setCacheVariable = setCacheVariable;
+/**
+ * Export the effective environment-file path
+ */
+function setEnvironmentFileOutputs(envFile, envContent, patched = false) {
+    core.setOutput(constants.OUTPUT_ENV_FILE_PATH, path.resolve(envFile));
+    core.setOutput(constants.OUTPUT_ENV_FILE_CONTENT, envContent);
+    core.setOutput(constants.OUTPUT_ENV_FILE_WAS_PATCHED, patched ? "true" : "false");
+    core.saveState(constants.OUTPUT_ENV_FILE_WAS_PATCHED, patched);
+}
+exports.setEnvironmentFileOutputs = setEnvironmentFileOutputs;
 
 
 /***/ }),
@@ -20557,6 +20586,18 @@ function setupMiniconda(inputs) {
         if (inputs.activateEnvironment) {
             yield core.group("Ensuring environment...", () => env.ensureEnvironment(inputs, options));
         }
+        if (core.getState(constants.OUTPUT_ENV_FILE_WAS_PATCHED)) {
+            yield core.group("Maybe cleaning up patched environment-file...", () => __awaiter(this, void 0, void 0, function* () {
+                const patchedEnv = core.getState(constants.OUTPUT_ENV_FILE_PATH);
+                if (inputs.cleanPatchedEnvironmentFile === "true") {
+                    fs.unlinkSync(patchedEnv);
+                    core.info(`Cleaned ${patchedEnv}`);
+                }
+                else {
+                    core.info(`Leaving ${patchedEnv} in place`);
+                }
+            }));
+        }
         core.info("setup-miniconda ran successfully");
     });
 }
@@ -20616,13 +20657,13 @@ var __awaiter = (this && this.__awaiter) || function (thisArg, _arguments, P, ge
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.ensureYaml = void 0;
 const fs = __importStar(__webpack_require__(747));
-const os = __importStar(__webpack_require__(87));
 const path = __importStar(__webpack_require__(622));
 const yaml = __importStar(__webpack_require__(414));
 const core = __importStar(__webpack_require__(470));
 const constants = __importStar(__webpack_require__(211));
 const conda = __importStar(__webpack_require__(259));
 const utils = __importStar(__webpack_require__(163));
+const outputs = __importStar(__webpack_require__(405));
 /**
  * The current known providers of patches to `environment.yml`
  *
@@ -20689,13 +20730,17 @@ exports.ensureYaml = {
         }
         if (patchesApplied.length) {
             const patchedYaml = yaml.safeDump(Object.assign(Object.assign({}, yamlData), { dependencies }));
-            envFile = path.join(os.tmpdir(), "environment-patched.yml");
+            const origPath = path.resolve(inputs.environmentFile);
+            const origParent = path.dirname(origPath);
+            envFile = path.join(origParent, `setup-miniconda-patched-${path.basename(origPath)}`);
             core.info(`Making patched copy of 'environment-file: ${inputs.environmentFile}'`);
-            core.info(patchedYaml);
+            core.info(`Using: ${envFile}\n${patchedYaml}`);
             fs.writeFileSync(envFile, patchedYaml, "utf8");
+            outputs.setEnvironmentFileOutputs(envFile, yaml.safeDump(patchedYaml), true);
         }
         else {
             core.info(`Using 'environment-file: ${inputs.environmentFile}' as-is`);
+            outputs.setEnvironmentFileOutputs(envFile, fs.readFileSync(inputs.environmentFile, "utf-8"));
         }
         return [
             "env",
@@ -29955,7 +30000,7 @@ module.exports.safeLoad    = safeLoad;
 /* 771 */
 /***/ (function(module) {
 
-module.exports = {"_args":[["cheerio@1.0.0-rc.3","/Users/gpenacastellanos/develop/conda-incubator/setup-miniconda"]],"_from":"cheerio@1.0.0-rc.3","_id":"cheerio@1.0.0-rc.3","_inBundle":false,"_integrity":"sha512-0td5ijfUPuubwLUu0OBoe98gZj8C/AA+RW3v67GPlGOrvxWjZmBXiBCRU+I8VEiNyJzjth40POfHiz2RB3gImA==","_location":"/cheerio","_phantomChildren":{},"_requested":{"type":"version","registry":true,"raw":"cheerio@1.0.0-rc.3","name":"cheerio","escapedName":"cheerio","rawSpec":"1.0.0-rc.3","saveSpec":null,"fetchSpec":"1.0.0-rc.3"},"_requiredBy":["/get-hrefs"],"_resolved":"https://registry.npmjs.org/cheerio/-/cheerio-1.0.0-rc.3.tgz","_spec":"1.0.0-rc.3","_where":"/Users/gpenacastellanos/develop/conda-incubator/setup-miniconda","author":{"name":"Matt Mueller","email":"mattmuelle@gmail.com","url":"mat.io"},"bugs":{"url":"https://github.com/cheeriojs/cheerio/issues"},"dependencies":{"css-select":"~1.2.0","dom-serializer":"~0.1.1","entities":"~1.1.1","htmlparser2":"^3.9.1","lodash":"^4.15.0","parse5":"^3.0.1"},"description":"Tiny, fast, and elegant implementation of core jQuery designed specifically for the server","devDependencies":{"benchmark":"^2.1.0","coveralls":"^2.11.9","expect.js":"~0.3.1","istanbul":"^0.4.3","jquery":"^3.0.0","jsdom":"^9.2.1","jshint":"^2.9.2","mocha":"^3.1.2","xyz":"~1.1.0"},"engines":{"node":">= 0.6"},"files":["index.js","lib"],"homepage":"https://github.com/cheeriojs/cheerio#readme","keywords":["htmlparser","jquery","selector","scraper","parser","html"],"license":"MIT","main":"./index.js","name":"cheerio","repository":{"type":"git","url":"git://github.com/cheeriojs/cheerio.git"},"scripts":{"test":"make test"},"version":"1.0.0-rc.3"};
+module.exports = {"_args":[["cheerio@1.0.0-rc.3","/home/weg/projects/actions/setup-miniconda"]],"_from":"cheerio@1.0.0-rc.3","_id":"cheerio@1.0.0-rc.3","_inBundle":false,"_integrity":"sha512-0td5ijfUPuubwLUu0OBoe98gZj8C/AA+RW3v67GPlGOrvxWjZmBXiBCRU+I8VEiNyJzjth40POfHiz2RB3gImA==","_location":"/cheerio","_phantomChildren":{},"_requested":{"type":"version","registry":true,"raw":"cheerio@1.0.0-rc.3","name":"cheerio","escapedName":"cheerio","rawSpec":"1.0.0-rc.3","saveSpec":null,"fetchSpec":"1.0.0-rc.3"},"_requiredBy":["/get-hrefs"],"_resolved":"https://registry.npmjs.org/cheerio/-/cheerio-1.0.0-rc.3.tgz","_spec":"1.0.0-rc.3","_where":"/home/weg/projects/actions/setup-miniconda","author":{"name":"Matt Mueller","email":"mattmuelle@gmail.com","url":"mat.io"},"bugs":{"url":"https://github.com/cheeriojs/cheerio/issues"},"dependencies":{"css-select":"~1.2.0","dom-serializer":"~0.1.1","entities":"~1.1.1","htmlparser2":"^3.9.1","lodash":"^4.15.0","parse5":"^3.0.1"},"description":"Tiny, fast, and elegant implementation of core jQuery designed specifically for the server","devDependencies":{"benchmark":"^2.1.0","coveralls":"^2.11.9","expect.js":"~0.3.1","istanbul":"^0.4.3","jquery":"^3.0.0","jsdom":"^9.2.1","jshint":"^2.9.2","mocha":"^3.1.2","xyz":"~1.1.0"},"engines":{"node":">= 0.6"},"files":["index.js","lib"],"homepage":"https://github.com/cheeriojs/cheerio#readme","keywords":["htmlparser","jquery","selector","scraper","parser","html"],"license":"MIT","main":"./index.js","name":"cheerio","repository":{"type":"git","url":"git://github.com/cheeriojs/cheerio.git"},"scripts":{"test":"make test"},"version":"1.0.0-rc.3"};
 
 /***/ }),
 /* 772 */

etc/example-environment-no-name.yml

@@ -1,3 +1,6 @@
+channels:
+  - conda-forge
+  - defaults
 dependencies:
   - black
   - anaconda-client

etc/example-environment.yml

@@ -1,4 +1,9 @@
 name: anaconda-client-env
+channels:
+  - conda-forge
+  - defaults
 dependencies:
-  - black
   - anaconda-client
+  - pip
+  - pip:
+      - -r requirements.txt

etc/requirements.txt

@@ -0,0 +1 @@
+isort

src/conda.ts

@@ -126,7 +126,8 @@ export async function applyCondaConfiguration(
   }
 
   // LIFO: reverse order to preserve higher priority as listed in the option
-  for (const channel of channels.reverse()) {
+  // .slice ensures working against a copy
+  for (const channel of channels.slice().reverse()) {
     core.info(`Adding channel '${channel}'`);
     await condaCommand(["config", "--add", "channels", channel], options);
   }