Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

dev: Switch away from the now-deprecated provision-with-micromamba #34

Merged
merged 1 commit into from
Jun 1, 2023

Conversation

tsibley
Copy link
Member

@tsibley tsibley commented Jun 1, 2023

It was deprecated last week in favor of a new setup-micromamba action¹, but that doesn't put micromamba on PATH without extra faffing about.

Since we use the action merely to obtain a micromamba executable on the PATH—shell init, environment creation, etc. is all unnecessary for our purposes—switch to the simpler installer shell script.²

¹ https://github.com/mamba-org/provision-with-micromamba/releases/tag/v16
² https://mamba.readthedocs.io/en/latest/installation.html#install-script

Related issues

Supersedes #33.

Testing

  • Checks pass

@tsibley tsibley requested a review from a team June 1, 2023 20:09
It was deprecated last week in favor of a new setup-micromamba action¹,
but that doesn't put `micromamba` on PATH without extra faffing about.

Since we use the action merely to obtain a `micromamba` executable on the
PATH—shell init, environment creation, etc. is all unnecessary for our
purposes—switch to the simpler installer shell script.²

¹ <https://github.com/mamba-org/provision-with-micromamba/releases/tag/v16>
² <https://mamba.readthedocs.io/en/latest/installation.html#install-script>
@tsibley tsibley force-pushed the trs/micromamba-installation branch from 5716644 to 88e65b7 Compare June 1, 2023 20:24
Copy link
Contributor

@joverlee521 joverlee521 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Makes sense to switch to the simple install script 👍

@tsibley tsibley merged commit 19bf02d into main Jun 1, 2023
@tsibley tsibley deleted the trs/micromamba-installation branch June 1, 2023 22:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants