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

Reference base environment specifically in Quick Start install #159

Merged
merged 2 commits into from
May 18, 2022

Conversation

srstsavage
Copy link
Contributor

A few users have been tripped up when trying to install dependencies from environment files with specific environment names, which haven't been created.

#136
#151

This tripped me up for a minute as well. My workflow is pretty much always to have an environment.yml file with a non-base name for local usage, but to install that environment as the base environment when creating a portable Docker image using micromamba. I suspect there are others out there doing the same.

Adding -n base to the quick start install instructions both clarifies that we're installing to the base environment, and also allows installing into the base environment from environment files with embedded names by overriding the name.

srstsavage and others added 2 commits May 17, 2022 13:11
A few users have been tripped up when trying to install dependencies from environment files with specific environment names, which haven't been created.

mamba-org#136
mamba-org#151

This tripped me up for a minute as well. My workflow is pretty much always to have an environment.yml file with a non-base `name` for local usage, but to install that environment as the `base` environment when creating a portable Docker image using micromamba. I suspect there are others out there doing the same.

Adding `-n base` to the quick start install instructions both clarifies that we're installing to the base environment, and also allows installing into the base environment from environment files with embedded names by overriding the name.
@wholtz
Copy link
Member

wholtz commented May 18, 2022

Hi @shane-axiom, thanks for the PR! As you correctly indicated, this has been a bit of a pain point, so thanks for the useful suggestion.

The examples in our README.md are part of our test suite, so I also updated the corresponding Dockerfile.

@wholtz wholtz merged commit f7cfe4b into mamba-org:main May 18, 2022
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